RESOLVED FIXED 59320
Web Inspector: introduce InspectorWorkerAgent 
https://bugs.webkit.org/show_bug.cgi?id=59320
Summary Web Inspector: introduce InspectorWorkerAgent
Yury Semikhatsky
Reported 2011-04-25 04:33:24 PDT
We need an agent that would communicate with workers inside the inspected page and allow connecting to the inspector of the corresponding worker context.
Attachments
Patch (37.56 KB, patch)
2011-04-25 06:01 PDT, Yury Semikhatsky 		pfeldman: review+
DetailsFormatted DiffDiff
Patch for landing (36.07 KB, patch)
2011-04-25 06:24 PDT, Yury Semikhatsky 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Yury Semikhatsky
Comment 1 2011-04-25 06:01:16 PDT
Created attachment 90914 [details] Patch
Pavel Feldman
Comment 2 2011-04-25 06:08:43 PDT
Comment on attachment 90914 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=90914&action=review > Source/WebCore/inspector/Inspector.json:1683 > + "name": "dispatchMessageOnWorkerInspector", Which way is what? sendMessageToWorker > Source/WebCore/inspector/Inspector.json:1698 > + "name": "dispatchMessageFromWorker", dispatchMessageFromWorker > Source/WebCore/inspector/Inspector.json:1703 > + }, Please remove trailing coma.
Yury Semikhatsky
Comment 3 2011-04-25 06:24:06 PDT
Created attachment 90915 [details] Patch for landing
Yury Semikhatsky
Comment 4 2011-04-25 06:24:33 PDT
(In reply to comment #2) > (From update of attachment 90914 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=90914&action=review > > > Source/WebCore/inspector/Inspector.json:1683 > > + "name": "dispatchMessageOnWorkerInspector", > > Which way is what? sendMessageToWorker > Done. > > Source/WebCore/inspector/Inspector.json:1698 > > + "name": "dispatchMessageFromWorker", > > dispatchMessageFromWorker > Done. > > Source/WebCore/inspector/Inspector.json:1703 > > + }, > > Please remove trailing coma. Done.
Yury Semikhatsky
Comment 5 2011-04-25 06:28:17 PDT
Committed r84773: <http://trac.webkit.org/changeset/84773>
Note You need to log in before you can comment on or make changes to this bug.