Now that the 3 modes implement a common interface, remove all custom logic in overview pane itself and handle them in a consistent manner.
Created attachment 195984 [details] Patch
Comment on attachment 195984 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=195984&action=review > Source/WebCore/inspector/front-end/TimelineOverviewPane.js:148 > + return null; Why not just throw if the mode is invalid, you would anyway fail on the first attempt to call a method on the controll at line 131?
Comment on attachment 195984 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=195984&action=review > Source/WebCore/inspector/front-end/TimelineOverviewPane.js:410 > + getWindowTimes: function(windowLeft, windowRight) windowTimes (since you touch it...)
Comment on attachment 195984 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=195984&action=review >> Source/WebCore/inspector/front-end/TimelineOverviewPane.js:148 >> + return null; > > Why not just throw if the mode is invalid, you would anyway fail on the first attempt to call a method on the controll at line 131? Fixed! >> Source/WebCore/inspector/front-end/TimelineOverviewPane.js:410 >> + getWindowTimes: function(windowLeft, windowRight) > > windowTimes (since you touch it...) Fixed as well.
Committed r147431: <http://trac.webkit.org/changeset/147431>