Title: [240489] trunk/Source/WebInspectorUI
Revision
240489
Author
[email protected]
Date
2019-01-25 11:39:38 -0800 (Fri, 25 Jan 2019)

Log Message

Web Inspector: REGRESSION (r237808): offscreen path warning doesn't work
https://bugs.webkit.org/show_bug.cgi?id=193830

Reviewed by Matt Baker.

* UserInterface/Models/RecordingAction.js:
(WI.RecordingAction.prototype.process):

Modified Paths

Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (240488 => 240489)


--- trunk/Source/WebInspectorUI/ChangeLog	2019-01-25 19:37:38 UTC (rev 240488)
+++ trunk/Source/WebInspectorUI/ChangeLog	2019-01-25 19:39:38 UTC (rev 240489)
@@ -1,3 +1,13 @@
+2019-01-25  Devin Rousso  <[email protected]>
+
+        Web Inspector: REGRESSION (r237808): offscreen path warning doesn't work
+        https://bugs.webkit.org/show_bug.cgi?id=193830
+
+        Reviewed by Matt Baker.
+
+        * UserInterface/Models/RecordingAction.js:
+        (WI.RecordingAction.prototype.process):
+
 2019-01-25  Joseph Pecoraro  <[email protected]>
 
         Web Inspector: Add another Protocol Version (iOS 12.2)

Modified: trunk/Source/WebInspectorUI/UserInterface/Models/RecordingAction.js (240488 => 240489)


--- trunk/Source/WebInspectorUI/UserInterface/Models/RecordingAction.js	2019-01-25 19:37:38 UTC (rev 240488)
+++ trunk/Source/WebInspectorUI/UserInterface/Models/RecordingAction.js	2019-01-25 19:39:38 UTC (rev 240489)
@@ -260,11 +260,11 @@
             }
 
             if (WI.ImageUtilities.supportsCanvasPathDebugging()) {
-                let currentX = currentState.currentX;
-                let invalidX = (currentX < 0 || currentX >= context.canvas.width) && (!lastState || currentX !== lastState.currentX);
+                let currentX = currentState.get("currentX");
+                let invalidX = (currentX < 0 || currentX >= context.canvas.width) && (!lastState || currentX !== lastState.get("currentX"));
 
-                let currentY = currentState.currentY;
-                let invalidY = (currentY < 0 || currentY >= context.canvas.height) && (!lastState || currentY !== lastState.currentY);
+                let currentY = currentState.get("currentY");
+                let invalidY = (currentY < 0 || currentY >= context.canvas.height) && (!lastState || currentY !== lastState.get("currentY"));
 
                 if (invalidX || invalidY)
                     this._warning = WI.UIString("This action moves the path outside the visible area");
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to