as far as each call from inspector front-end to inspector back-end has its own seq property by default we can remove explicit callId arguments from all backend functions. And now it is possible to call any InspectorBackend function with a callback as additional argument. as example InspectorBackend.enableResourceTracking(); or InspectorBackend.enableResourceTracking(someCallbackFunction);
Created attachment 65462 [details] [patch] initial version.
Comment on attachment 65462 [details] [patch] initial version. WebCore/inspector/CodeGeneratorInspector.pm:357 + push(@function, " responseMessage->setString(\"type\", \"response\");"); Why do we specify type for something that has seq? WebCore/inspector/CodeGeneratorInspector.pm: + reportProtocolError(callId, "dispatch", "Error: Invalid message format. The type of 'command' property should be string."); I think this is time to write a test that would issue valid and invalid requests against InspectorBackend and test the response. WebCore/inspector/front-end/CSSStyleModel.js:94 + InspectorBackend.setStyleText(styleId, cssText); Is this case handled properly? A test would be great.
Committed r66087 M WebKit/chromium/ChangeLog M WebKit/chromium/src/js/ProfilerAgent.js M WebCore/ChangeLog M WebCore/inspector/CodeGeneratorInspector.pm M WebCore/inspector/Inspector.idl M WebCore/inspector/front-end/MetricsSidebarPane.js M WebCore/inspector/front-end/ResourcesPanel.js M WebCore/inspector/front-end/ProfileView.js M WebCore/inspector/front-end/ElementsTreeOutline.js M WebCore/inspector/front-end/Settings.js M WebCore/inspector/front-end/ScriptsPanel.js M WebCore/inspector/front-end/DOMAgent.js M WebCore/inspector/front-end/inspector.js M WebCore/inspector/front-end/CSSStyleModel.js M WebCore/inspector/front-end/ElementsPanel.js M WebCore/inspector/front-end/ScriptView.js M WebCore/inspector/front-end/ProfilesPanel.js M WebCore/inspector/front-end/DOMStorage.js M WebCore/inspector/front-end/Database.js M WebCore/inspector/front-end/AuditRules.js M WebCore/inspector/front-end/BreakpointManager.js M WebCore/inspector/front-end/InjectedScriptAccess.js M LayoutTests/http/tests/inspector/inspect-iframe-from-different-domain.html M LayoutTests/inspector/styles-source-offsets.html M LayoutTests/inspector/elements-panel-search.html M LayoutTests/inspector/styles-iframe.html M LayoutTests/inspector/styles-source-lines.html M LayoutTests/ChangeLog r66087 = dbd6bd336843e7a0ebbf6b16a6682700ebd6f782 (refs/remotes/trunk)