Title: [197035] trunk/Source/WebInspectorUI
- Revision
- 197035
- Author
- timo...@apple.com
- Date
- 2016-02-24 09:56:11 -0800 (Wed, 24 Feb 2016)
Log 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.
Modified Paths
Diff
Modified: trunk/Source/WebInspectorUI/ChangeLog (197034 => 197035)
--- trunk/Source/WebInspectorUI/ChangeLog 2016-02-24 17:41:42 UTC (rev 197034)
+++ trunk/Source/WebInspectorUI/ChangeLog 2016-02-24 17:56:11 UTC (rev 197035)
@@ -1,3 +1,14 @@
+2016-02-24 Timothy Hatcher <timo...@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 <mattba...@apple.com>
Web Inspector: TimelineRuler should have a "select all" mode
Modified: trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRuler.js (197034 => 197035)
--- trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRuler.js 2016-02-24 17:41:42 UTC (rev 197034)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/TimelineRuler.js 2016-02-24 17:56:11 UTC (rev 197035)
@@ -174,10 +174,11 @@
if (this._zeroTime === x)
return;
- this._zeroTime = x;
if (this.entireRangeSelected)
- this.selectionStartTime = this._zeroTime;
+ this.selectionStartTime = x;
+ this._zeroTime = x;
+
this.needsLayout();
}
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes