RESOLVED FIXED 58668
Web Inspector: introduce WorkerInspectorController 
https://bugs.webkit.org/show_bug.cgi?id=58668
Summary Web Inspector: introduce WorkerInspectorController
Yury Semikhatsky
Reported 2011-04-15 08:42:02 PDT
As described in bug 58541 to support worker inspection we will need a class that would hold all inspector agents for the inspected WorkerContext.
Attachments
Patch (25.29 KB, patch)
2011-04-15 09:34 PDT, Yury Semikhatsky 		pfeldman: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Yury Semikhatsky
Comment 1 2011-04-15 09:34:11 PDT
Created attachment 89796 [details] Patch
Pavel Feldman
Comment 2 2011-04-18 02:10:50 PDT
Comment on attachment 89796 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=89796&action=review > Source/WebCore/inspector/WorkerInspectorController.cpp:107 > + 0, What does it mean? :) Could you comment on agent names?
Yury Semikhatsky
Comment 3 2011-04-18 02:21:18 PDT
(In reply to comment #2) > (From update of attachment 89796 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=89796&action=review > > > Source/WebCore/inspector/WorkerInspectorController.cpp:107 > > + 0, > > What does it mean? :) Could you comment on agent names? Done.
Yury Semikhatsky
Comment 4 2011-04-18 02:28:29 PDT
Committed r84125: <http://trac.webkit.org/changeset/84125>
WebKit Review Bot
Comment 5 2011-04-18 04:05:49 PDT
http://trac.webkit.org/changeset/84125 might have broken GTK Linux 64-bit Debug
Note You need to log in before you can comment on or make changes to this bug.