RESOLVED FIXED 165701
Web Inspector: Move MainTarget and WorkerTarget to their own files
https://bugs.webkit.org/show_bug.cgi?id=165701
Summary Web Inspector: Move MainTarget and WorkerTarget to their own files
Joseph Pecoraro
Reported 2016-12-09 16:22:59 PST
Summary: Move MainTarget and WorkerTarget to their own files This is cleanup that I had in another patch that is stalled. Separating it out into its own patch.
Attachments
[PATCH] Proposed Fix (14.29 KB, patch)
2016-12-09 16:27 PST, Joseph Pecoraro
no flags
Joseph Pecoraro
Comment 1 2016-12-09 16:27:14 PST
Created attachment 296737 [details] [PATCH] Proposed Fix
Blaze Burg
Comment 2 2016-12-11 15:12:30 PST
Comment on attachment 296737 [details] [PATCH] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=296737&action=review rs=me > Source/WebInspectorUI/UserInterface/Protocol/Target.js:-147 > - initialize() Why did this construction ever exist instead of putting in just one subclass constructor?
WebKit Commit Bot
Comment 3 2016-12-11 15:37:43 PST
Comment on attachment 296737 [details] [PATCH] Proposed Fix Clearing flags on attachment: 296737 Committed r209686: <http://trac.webkit.org/changeset/209686>
WebKit Commit Bot
Comment 4 2016-12-11 15:37:48 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.