[Webkit-unassigned] [Bug 98705] [EFL][WK2] Add Remote Web Inspector

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Dec 6 17:06:33 PST 2012


https://bugs.webkit.org/show_bug.cgi?id=98705





--- Comment #15 from Gyuyoung Kim <gyuyoung.kim at samsung.com>  2012-12-06 17:08:59 PST ---
(From update of attachment 177992)
View in context: https://bugs.webkit.org/attachment.cgi?id=177992&action=review

> Source/WebKit2/UIProcess/InspectorServer/efl/WebInspectorServerEfl.cpp:53
> +    String localPath = inspectorResourcePath() + ((path == "/") ? "/inspectorPageIndex.html" : path);

At least, it looks you can use ASCIILiteral("/inspectorPageIndex.html") instead of "/inspectorPageIndex.html".

> Source/WebKit2/UIProcess/efl/WebContextEfl.cpp:46
> +

It would be good if you add a comment why *initialized* should be true.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list