[webkit-reviews] review granted: [Bug 59378] WebKit2: Should be able to inspect Web Inspector in Debug Builds : [Attachment 91011] [PATCH] Fix v2
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Apr 25 16:37:46 PDT 2011
Adam Roben (:aroben) <aroben at apple.com> has granted Brian Weinstein
<bweinstein at apple.com>'s request for review:
Bug 59378: WebKit2: Should be able to inspect Web Inspector in Debug Builds
https://bugs.webkit.org/show_bug.cgi?id=59378
Attachment 91011: [PATCH] Fix v2
https://bugs.webkit.org/attachment.cgi?id=91011&action=review
------- Additional Comments from Adam Roben (:aroben) <aroben at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=91011&action=review
> Source/WebKit2/ChangeLog:9
> + (WebKit::WebInspectorProxy::createInspectorPageGroup): Creates an
inspector page group and enables
The "WebInspectorProxy::" part of this line is incorrect.
> Source/WebKit2/UIProcess/WebInspectorProxy.cpp:50
> +static WebPageGroup* createInspectorPageGroup()
> +{
> + WebPageGroup* pageGroup =
WebPageGroup::create("__WebInspectorPageGroup__", false, false).leakRef();
You should make this function return a PassRefPtr and put the leakRef at the
call site.
More information about the webkit-reviews
mailing list