Title: [223171] trunk/Source/WebInspectorUI
Revision
223171
Author
[email protected]
Date
2017-10-11 00:59:43 -0700 (Wed, 11 Oct 2017)

Log Message

Web Inspector: Avoid "100.00ms" vs "100.0ms" jitter in TimelineRuler labels
https://bugs.webkit.org/show_bug.cgi?id=178143

Reviewed by Brian Burg.

* UserInterface/Base/Utilities.js:
Allow a small slop factor that wouldn't have changed the display value's
significant figures anyways. This addresses cases where 100.0000000000001
were being treated slightly differently from 100.0.

Modified Paths

Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (223170 => 223171)


--- trunk/Source/WebInspectorUI/ChangeLog	2017-10-11 07:55:24 UTC (rev 223170)
+++ trunk/Source/WebInspectorUI/ChangeLog	2017-10-11 07:59:43 UTC (rev 223171)
@@ -1,5 +1,17 @@
 2017-10-11  Joseph Pecoraro  <[email protected]>
 
+        Web Inspector: Avoid "100.00ms" vs "100.0ms" jitter in TimelineRuler labels
+        https://bugs.webkit.org/show_bug.cgi?id=178143
+
+        Reviewed by Brian Burg.
+
+        * UserInterface/Base/Utilities.js:
+        Allow a small slop factor that wouldn't have changed the display value's
+        significant figures anyways. This addresses cases where 100.0000000000001
+        were being treated slightly differently from 100.0.
+
+2017-10-11  Joseph Pecoraro  <[email protected]>
+
         Web Inspector: Network Tab - Show initially loaded resources even if network info not logged
         https://bugs.webkit.org/show_bug.cgi?id=178098
         <rdar://problem/34073529>

Modified: trunk/Source/WebInspectorUI/UserInterface/Base/Utilities.js (223170 => 223171)


--- trunk/Source/WebInspectorUI/UserInterface/Base/Utilities.js	2017-10-11 07:55:24 UTC (rev 223170)
+++ trunk/Source/WebInspectorUI/UserInterface/Base/Utilities.js	2017-10-11 07:59:43 UTC (rev 223171)
@@ -1089,19 +1089,21 @@
         if (!ms)
             return WI.UIString("%.0fms").format(0);
 
-        if (Math.abs(ms) < 10) {
+        const epsilon = 0.0001;
+
+        if (Math.abs(ms) < (10 + epsilon)) {
             if (higherResolution)
                 return WI.UIString("%.3fms").format(ms);
             return WI.UIString("%.2fms").format(ms);
         }
 
-        if (Math.abs(ms) < 100) {
+        if (Math.abs(ms) < (100 + epsilon)) {
             if (higherResolution)
                 return WI.UIString("%.2fms").format(ms);
             return WI.UIString("%.1fms").format(ms);
         }
 
-        if (Math.abs(ms) < 1000) {
+        if (Math.abs(ms) < (1000 + epsilon)) {
             if (higherResolution)
                 return WI.UIString("%.1fms").format(ms);
             return WI.UIString("%.0fms").format(ms);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to