[webkit-reviews] review requested: [Bug 91116] WebContext::getWebCoreStatistics() causes crash if no m_process : [Attachment 151999] Proposed patch which checks for m_process before sending message.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jul 12 10:54:33 PDT 2012


Josh Hawn <jhawn at apple.com> has asked  for review:
Bug 91116: WebContext::getWebCoreStatistics() causes crash if no m_process
https://bugs.webkit.org/show_bug.cgi?id=91116

Attachment 151999: Proposed patch which checks for m_process before sending
message.
https://bugs.webkit.org/attachment.cgi?id=151999&action=review

------- Additional Comments from Josh Hawn <jhawn at apple.com>
Return value has also been changed from void to bool. Because send() returns a
bool, it is probably a good idea to return this result up to the user of the
function. This way the user will know if the action succeeded or not.


More information about the webkit-reviews mailing list