Changeset 197035 in webkit

Timestamp:

Feb 24, 2016, 9:56:11 AM (9 years ago)

Author:

timothy@apple.com

Message:

Follow up fix for the TimelineRuler "select all" mode to fix zeroTime.

​https://bugs.webkit.org/show_bug.cgi?id=154561
rdar://problem/24779872

• UserInterface/Views/TimelineRuler.js:

(WebInspector.TimelineRuler.prototype.set zeroTime): Change selectionStartTime
before _zeroTime so the check for entireRangeSelected still works.

Location:

trunk/Source/WebInspectorUI

Files:

• ChangeLog (modified) (1 diff)
• UserInterface/Views/TimelineRuler.js (modified) (1 diff)

trunk/Source/WebInspectorUI/ChangeLog

@@ +1 @@
+2016-02-24  Timothy Hatcher  <timothy@apple.com>
+
+        Follow up fix for the TimelineRuler "select all" mode to fix zeroTime.
+
+        https://bugs.webkit.org/show_bug.cgi?id=154561
+        rdar://problem/24779872
+
+        * UserInterface/Views/TimelineRuler.js:
+        (WebInspector.TimelineRuler.prototype.set zeroTime): Change selectionStartTime
+        before _zeroTime so the check for entireRangeSelected still works.
+
 2016-02-24  Matt Baker  <mattbaker@apple.com>
 

trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRuler.js

@@ -175 +175 @@
             return;
 
+        if (this.entireRangeSelected)
+            this.selectionStartTime = x;
+
         this._zeroTime = x;
-        if (this.entireRangeSelected)
-            this.selectionStartTime = this._zeroTime;
 
         this.needsLayout();