This removed all inspector notification methods in favor of using events.
Created attachment 116107 [details] Patch
Comment on attachment 116107 [details] Patch Attachment 116107 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/10393435 New failing tests: inspector/extensions/extensions-events.html
Comment on attachment 116107 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=116107&action=review > Source/WebCore/inspector/front-end/ExtensionServer.js:526 > + this._registerAutosubscriptionHandler("panel-objectSelected-elements", Could you introduce the constants for these? > Source/WebCore/inspector/front-end/Resource.js:865 > + WebInspector.Resource.notifications.dispatchEventToListeners(WebInspector.Resource.Events.ResourceContentCommitted, { resource: this, content: newContent }); Could you please dispatch this event on the resource tree model instead?
Created attachment 116204 [details] Patch
- use constants for command and event names - dispatch ResourceContentCommitted on WebInspector.resourceTreeModel, not on WebInspector.Resource.notifications()
Comment on attachment 116204 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=116204&action=review > Source/WebCore/inspector/front-end/Resource.js:255 > + ResourceContentCommitted: "resource-content-committed" You should move this to the tree model event list.
Committed r100994: <http://trac.webkit.org/changeset/100994>