Chromium Code Reviews| Index: Source/devtools/front_end/TimelineOverviewPane.js |
| diff --git a/Source/devtools/front_end/TimelineOverviewPane.js b/Source/devtools/front_end/TimelineOverviewPane.js |
| index 2e6072958165d9453796eff76c8b2ea30e192bd6..307c973982abe4802132ef7361ece5726a68616b 100644 |
| --- a/Source/devtools/front_end/TimelineOverviewPane.js |
| +++ b/Source/devtools/front_end/TimelineOverviewPane.js |
| @@ -97,6 +97,7 @@ WebInspector.TimelineOverviewPane.prototype = { |
| this._overviewControl.update(); |
| this._overviewGrid.updateDividers(this._overviewCalculator); |
| this._updateEventDividers(); |
| + this._updateWindow(); |
|
pfeldman
2014/03/04 06:26:52
This is strange since window has not changed. We s
|
| }, |
| _updateEventDividers: function() |
| @@ -152,6 +153,8 @@ WebInspector.TimelineOverviewPane.prototype = { |
| if (this._muteOnWindowChanged) |
| return; |
| var windowTimes = this._overviewControl.windowTimes(this._overviewGrid.windowLeft(), this._overviewGrid.windowRight()); |
| + this._windowStartTime = windowTimes.startTime; |
|
pfeldman
2014/03/04 06:26:52
These three lines should be extracted (they are us
|
| + this._windowEndTime = windowTimes.endTime; |
| this.dispatchEventToListeners(WebInspector.TimelineOverviewPane.Events.WindowChanged, windowTimes); |
| }, |
| @@ -165,13 +168,17 @@ WebInspector.TimelineOverviewPane.prototype = { |
| return; |
| this._windowStartTime = startTime; |
| this._windowEndTime = endTime; |
| - var windowBoundaries = this._overviewControl.windowBoundaries(startTime, endTime); |
| + this._updateWindow(); |
| + this.dispatchEventToListeners(WebInspector.TimelineOverviewPane.Events.WindowChanged, { startTime: startTime, endTime: endTime }); |
| + }, |
| + |
| + _updateWindow: function() |
| + { |
| + var windowBoundaries = this._overviewControl.windowBoundaries(this._windowStartTime, this._windowEndTime); |
| this._muteOnWindowChanged = true; |
| this._overviewGrid.setWindow(windowBoundaries.left, windowBoundaries.right); |
| this._overviewGrid.setResizeEnabled(!!this._model.records().length); |
| this._muteOnWindowChanged = false; |
| - |
| - this.dispatchEventToListeners(WebInspector.TimelineOverviewPane.Events.WindowChanged, { startTime: startTime, endTime: endTime }); |
| }, |
| _scheduleRefresh: function() |