The chromedevtools plugin (http://code.google.com/p/chromedevtools/) requires a specific json format to connect to the webkit inspector. The current format is missing some required fields that make the connection with the plugin fails. The format expected by the plugin is as follows: { "title": "Foo", "url": "http://foo", "devtoolsFrontendUrl": "/inspector.html?ws=localhost:9222/devtools/page/1", "webSocketDebuggerUrl": "ws://localhost:9222/devtools/page/1" } The current format is missing the entries for "devtoolsFrontendUrl" and "webSocketDebuggerUrl". The plugin also sends a GET request to http://host:port/json instead of http://host:port/pagelist.json when retrieving the pages list. Patch to follow.
<rdar://problem/14958022>
Created attachment 211244 [details] Patch This patch requires the patch from bug #121123 to work properly.
Created attachment 211586 [details] Updated patch against upstream master Updated patch after http://trac.webkit.org/changeset/155714 - to use Main.html instead of inspector.html on the "devtoolsFrontendUrl" entry on results to /json requests. Tested chromedevtools 0.3.8 plugin with master at 6fa836 + this updated patch and everything is working fine.
Comment on attachment 211586 [details] Updated patch against upstream master View in context: https://bugs.webkit.org/attachment.cgi?id=211586&action=review > Source/WebKit2/ChangeLog:12 > + fails. s/fails/fail/ > Source/WebKit2/ChangeLog:28 > + This patch fix the issues by adding support for the missing fields and accepting requests s/fix/fixes/ > Source/WebKit2/UIProcess/InspectorServer/gtk/WebInspectorServerGtk.cpp:128 > + > + // chromedevtools (http://code.google.com/p/chromedevtools) 0.3.8 expected JSON format: > + // { > + // "title": "Foo", > + // "url": "http://foo", > + // "devtoolsFrontendUrl": "/Main.html?ws=localhost:9222/devtools/page/1", > + // "webSocketDebuggerUrl": "ws://localhost:9222/devtools/page/1" > + // }, > + This comment might be more useful if it comes before the response is built.
Created attachment 212812 [details] Patch with review suggestions applied Patch with review suggestions applied.
Comment on attachment 212812 [details] Patch with review suggestions applied Will just wait on the EWS to cq+
Comment on attachment 212812 [details] Patch with review suggestions applied Clearing flags on attachment: 212812 Committed r156555: <http://trac.webkit.org/changeset/156555>
All reviewed patches have been landed. Closing bug.