[webkit-reviews] review granted: [Bug 116225] [GTK] Remote Web Inspector sometimes won't serve any content : [Attachment 202438] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue May 21 10:15:22 PDT 2013


Gustavo Noronha (kov) <gns at gnome.org> has granted Anton Obzhirov
<a.obzhirov at samsung.com>'s request for review:
Bug 116225: [GTK] Remote Web Inspector sometimes won't serve any content
https://bugs.webkit.org/show_bug.cgi?id=116225

Attachment 202438: Patch
https://bugs.webkit.org/attachment.cgi?id=202438&action=review

------- Additional Comments from Gustavo Noronha (kov) <gns at gnome.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=202438&action=review


> Source/WebKit2/UIProcess/InspectorServer/WebInspectorServer.h:75
> +    const char* inspectorServerFilesPath();

I think it doesn't make sense to use a char pointer just here, just return the
CString? In fact, you can just use WTFString and avoid the other include IMO.


More information about the webkit-reviews mailing list