Cleaning up comments and unused portions of a webgl test, as per review comments.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 1 Dec 2011 20:02:31 +0000 (20:02 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 1 Dec 2011 20:02:31 +0000 (20:02 +0000)
https://bugs.webkit.org/show_bug.cgi?id=73033

Patch by Jeff Timanus <twiz@chromium.org> on 2011-12-01
Reviewed by Stephen White.

* fast/canvas/webgl/webgl-texture-binding-preserved.html:

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

LayoutTests/ChangeLog
LayoutTests/fast/canvas/webgl/webgl-texture-binding-preserved.html

index b6a096a041a7a0a8fca6dce59187696746940f7c..7bbc266cb8ed159ba78857926d7ad55be933e1a5 100644 (file)
@@ -1,3 +1,12 @@
+2011-12-01  Jeff Timanus  <twiz@chromium.org>
+
+        Cleaning up comments and unused portions of a webgl test, as per review comments.
+        https://bugs.webkit.org/show_bug.cgi?id=73033
+
+        Reviewed by Stephen White.
+
+        * fast/canvas/webgl/webgl-texture-binding-preserved.html:
+
 2011-12-01  Ilya Tikhonovsky  <loislo@chromium.org>
 
         Web Inspector: chromium: move and adapt Inspector's performance tests for running with run-inspector-perf-tests.py.
 2011-12-01  Ilya Tikhonovsky  <loislo@chromium.org>
 
         Web Inspector: chromium: move and adapt Inspector's performance tests for running with run-inspector-perf-tests.py.
index 06a1b7b00eee1dc776088e5807caf5e01031ac07..b71cd8db63f2b6030ffc0bb92c439262ae6673e5 100755 (executable)
@@ -19,17 +19,16 @@ if (window.layoutTestController) {
 
 var canvas;
 var context;
 
 var canvas;
 var context;
-var requestAnimationFrame;
 
 function draw() {
   // Draw a blue quad to the screen.  The compositor should have executed
 
 function draw() {
   // Draw a blue quad to the screen.  The compositor should have executed
-  // before reaching this point.  If the texture state was corruted, then 
+  // before reaching this point.  If the texture state was corrupted, then
   // a red quad will be drawn.  See crbug.com/105045.
   wtu.drawQuad(context);
 
   wtu.checkCanvasRect(context, 0, 0, 1, 1, [0, 0, 255, 255], "Should have rendered blue.", 1);
   if (window.layoutTestController)
   // a red quad will be drawn.  See crbug.com/105045.
   wtu.drawQuad(context);
 
   wtu.checkCanvasRect(context, 0, 0, 1, 1, [0, 0, 255, 255], "Should have rendered blue.", 1);
   if (window.layoutTestController)
-      layoutTestController.notifyDone();
+    layoutTestController.notifyDone();
 }
 
 window.onload = function()
 }
 
 window.onload = function()
@@ -39,7 +38,6 @@ window.onload = function()
   }
 
   canvas = document.getElementById("webgl-canvas");
   }
 
   canvas = document.getElementById("webgl-canvas");
-  canvas.width = 50; canvas.height = 50;
   context = create3DContext(canvas);
 
   requestAnimationFrame = window.webkitRequestAnimationFrame
   context = create3DContext(canvas);
 
   requestAnimationFrame = window.webkitRequestAnimationFrame
@@ -59,7 +57,9 @@ window.onload = function()
   context.clear(context.COLOR_BUFFER_BIT | context.DEPTH_BUFFER_BIT);
 
   // We need to ensure that the compositor has run before the drawing
   context.clear(context.COLOR_BUFFER_BIT | context.DEPTH_BUFFER_BIT);
 
   // We need to ensure that the compositor has run before the drawing
-  // takes place.
+  // takes place.  requestAnimationFrame was tried for this purpose,
+  // but it did not produce the failing behaviour in crbug.com/105045.
+  // A timeout of 100 ms was found to consistently reproduce the problem.
   setTimeout(draw, 100);
 }
 </script>
   setTimeout(draw, 100);
 }
 </script>