RESOLVED FIXED 127655
Web Inspector: Prefix existing Profiler UI classes with "Legacy" 
https://bugs.webkit.org/show_bug.cgi?id=127655
Summary Web Inspector: Prefix existing Profiler UI classes with "Legacy"
Timothy Hatcher
Reported 2014-01-26 12:08:25 PST
We need to keep the old profiler classes around to support iOS 6 & 7. Also remove the unfinished Canvas profiler classes, they will need redone for the Timeline view. These didn't exist on iOS 6 & 7.
Attachments
Patch (185.26 KB, patch)
2014-01-26 12:16 PST, Timothy Hatcher 		no flags
DetailsFormatted DiffDiff
Better Patch (67.08 KB, patch)
2014-01-26 12:17 PST, Timothy Hatcher 		joepeck: review+
timothy: commit-queue-
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2014-01-26 12:08:39 PST
<rdar://problem/15911205>
Timothy Hatcher
Comment 2 2014-01-26 12:16:48 PST
Created attachment 222280 [details] Patch
Timothy Hatcher
Comment 3 2014-01-26 12:17:53 PST
Created attachment 222281 [details] Better Patch
Joseph Pecoraro
Comment 4 2014-01-28 16:06:38 PST
Comment on attachment 222281 [details] Better Patch View in context: https://bugs.webkit.org/attachment.cgi?id=222281&action=review r=me > Source/WebInspectorUI/UserInterface/Main.js:-58 > - if (InspectorBackend.registerCanvasDispatcher) > - InspectorBackend.registerCanvasDispatcher(new WebInspector.CanvasObserver); Do you want to also delete the Canvas protocol/agent at this time?
Timothy Hatcher
Comment 5 2014-01-30 17:30:58 PST
http://trac.webkit.org/changeset/163138
Note You need to log in before you can comment on or make changes to this bug.