RESOLVED FIXED 195471
Web Inspector: CPU Usage Timeline - Enable by default
https://bugs.webkit.org/show_bug.cgi?id=195471
Summary Web Inspector: CPU Usage Timeline - Enable by default
Joseph Pecoraro
Reported 2019-03-08 10:49:20 PST
CPU Usage Timeline - Enable by default - And remove Legacy
Attachments
[PATCH] Proposed Fix (21.45 KB, patch)
2019-03-08 10:50 PST, Joseph Pecoraro
hi: review+
Joseph Pecoraro
Comment 1 2019-03-08 10:50:32 PST
Created attachment 364034 [details] [PATCH] Proposed Fix
Devin Rousso
Comment 2 2019-03-08 17:56:34 PST
Comment on attachment 364034 [details] [PATCH] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=364034&action=review r=me > Source/WebInspectorUI/UserInterface/Controllers/TimelineManager.js:113 > + if (WI.CPUInstrument.supported()) > + defaultTypes.push(WI.TimelineRecord.Type.CPU); Should we put this above the FPS instrument, seeing as it's shown in the same grouping as the other default types (e.g. the rendering frame timeline is a separate "view"/"tab" from Network/Layout/Script)?
Joseph Pecoraro
Comment 3 2019-03-11 14:31:08 PDT
Radar WebKit Bug Importer
Comment 4 2019-03-11 14:32:22 PDT
Note You need to log in before you can comment on or make changes to this bug.