Web Inspector: rendering frames timeline "60fps" horizontal guide is mispositioned...
authormattbaker@apple.com <mattbaker@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 10 Jun 2016 19:59:20 +0000 (19:59 +0000)
committermattbaker@apple.com <mattbaker@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 10 Jun 2016 19:59:20 +0000 (19:59 +0000)
https://bugs.webkit.org/show_bug.cgi?id=158587
<rdar://problem/26728174>

Reviewed by Timothy Hatcher.

When the view mode changes we iterate over all overview graphs, updating
their visibility as needed. Graphs being shown perform layout, even though
the overview's state is in transition: its offset height may still change.

A simple fix is to use TimelineOverviewGraph.height instead of the graph
element's offset height when performing layout.

* UserInterface/Views/RenderingFrameTimelineOverviewGraph.js:
(WebInspector.RenderingFrameTimelineOverviewGraph.prototype.layout):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@201937 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebInspectorUI/ChangeLog
Source/WebInspectorUI/UserInterface/Views/RenderingFrameTimelineOverviewGraph.js

index 074e06b..70f4902 100644 (file)
@@ -1,5 +1,23 @@
 2016-06-10  Matt Baker  <mattbaker@apple.com>
 
+        Web Inspector: rendering frames timeline "60fps" horizontal guide is mispositioned after switching timeline modes
+        https://bugs.webkit.org/show_bug.cgi?id=158587
+        <rdar://problem/26728174>
+
+        Reviewed by Timothy Hatcher.
+
+        When the view mode changes we iterate over all overview graphs, updating
+        their visibility as needed. Graphs being shown perform layout, even though
+        the overview's state is in transition: its offset height may still change.
+
+        A simple fix is to use TimelineOverviewGraph.height instead of the graph
+        element's offset height when performing layout.
+
+        * UserInterface/Views/RenderingFrameTimelineOverviewGraph.js:
+        (WebInspector.RenderingFrameTimelineOverviewGraph.prototype.layout):
+
+2016-06-10  Matt Baker  <mattbaker@apple.com>
+
         REGRESSION (r201538): Web Inspector: 1px gap above timeline progress view
         https://bugs.webkit.org/show_bug.cgi?id=158626
 
index d189d6b..5a74257 100644 (file)
@@ -178,7 +178,7 @@ WebInspector.RenderingFrameTimelineOverviewGraph = class RenderingFrameTimelineO
         if (this.graphHeightSeconds === 0)
             return;
 
-        var overviewGraphHeight = this.element.offsetHeight;
+        let overviewGraphHeight = this.height;
 
         function createDividerAtPosition(framesPerSecond)
         {