Title: [236957] trunk/LayoutTests
Revision
236957
Author
[email protected]
Date
2018-10-08 19:37:24 -0700 (Mon, 08 Oct 2018)

Log Message

Fix inspector/canvas/recording.html after r236952

Unreviewed. Test gardening.

* inspector/canvas/recording-expected.txt:
* inspector/canvas/recording.html:
Recordings default to only capturing a single frame, so add a parameter to ensure that we
are capturing multiple frames.

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (236956 => 236957)


--- trunk/LayoutTests/ChangeLog	2018-10-09 02:17:42 UTC (rev 236956)
+++ trunk/LayoutTests/ChangeLog	2018-10-09 02:37:24 UTC (rev 236957)
@@ -1,3 +1,14 @@
+2018-10-08  Devin Rousso  <[email protected]>
+
+        Fix inspector/canvas/recording.html after r236952
+
+        Unreviewed. Test gardening.
+
+        * inspector/canvas/recording-expected.txt:
+        * inspector/canvas/recording.html:
+        Recordings default to only capturing a single frame, so add a parameter to ensure that we
+        are capturing multiple frames.
+
 2018-10-08  Wenson Hsieh  <[email protected]>
 
         [Cocoa] [WK2] Add support for text alignment and text lists in font attributes

Modified: trunk/LayoutTests/inspector/canvas/recording-expected.txt (236956 => 236957)


--- trunk/LayoutTests/inspector/canvas/recording-expected.txt	2018-10-09 02:17:42 UTC (rev 236956)
+++ trunk/LayoutTests/inspector/canvas/recording-expected.txt	2018-10-09 02:37:24 UTC (rev 236957)
@@ -15,8 +15,6 @@
 PASS: There should be a recording for canvas 2.
 
 -- Running test case: Canvas.startRecording.InvalidCanvasId
-ERROR: No active recording for canvas
-ERROR: No active recording for canvas
 PASS: Should produce an error.
 Error: No canvas for given identifier.
 

Modified: trunk/LayoutTests/inspector/canvas/recording.html (236956 => 236957)


--- trunk/LayoutTests/inspector/canvas/recording.html	2018-10-09 02:17:42 UTC (rev 236956)
+++ trunk/LayoutTests/inspector/canvas/recording.html	2018-10-09 02:37:24 UTC (rev 236957)
@@ -20,6 +20,8 @@
         name: "Canvas.multipleRecording",
         description: "Check that multiple recordings are able to be started/stopped at the same time.",
         test(resolve, reject) {
+            const singleFrame = false;
+
             let canvases = WI.canvasManager.canvases;
             InspectorTest.assert(canvases.length === 2, "There should be two canvas contexts.");
 
@@ -58,11 +60,11 @@
                 InspectorTest.expectThat(canvases[0].recordingActive, "Recording started of canvas 1");
 
                 InspectorTest.log("Starting a recording of canvas 2...");
-                canvases[1].startRecording();
+                canvases[1].startRecording(singleFrame);
             });
 
             InspectorTest.log("Starting a recording of canvas 1...");
-            canvases[0].startRecording();
+            canvases[0].startRecording(singleFrame);
         },
     });
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to