Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(108)

Side by Side Diff: content/test/data/gpu/pixel_canvas_display_linear-rgb.html

Issue 2457113002: Fixing timeout in pixel_canvas_display_linear-rgb browser pixel test (Closed)
Patch Set: Still timeout. Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <!DOCTYPE HTML> 1 <!DOCTYPE HTML>
2 <html> 2 <html>
3 <head> 3 <head>
4 <style type="text/css">
5 .nomargin {
6 margin: 0px auto;
7 }
8 </style>
9 <script> 4 <script>
10 // This browser pixel test checks if the 2D canvas is properly displayed 5 // This browser pixel test checks if the 2D canvas is properly displayed
11 // when the color space is set to linear-rgb. 6 // when the color space is set to linear-rgb.
12 7
13 var g_swapsBeforeAck = 15; 8 var g_swapsBeforeAck = 15;
14 9
15 function main() 10 function main()
16 { 11 {
17 draw(); 12 draw();
18 waitForFinish(); 13 waitForFinish();
(...skipping 16 matching lines...) Expand all
35 { 30 {
36 if (g_swapsBeforeAck == 0) { 31 if (g_swapsBeforeAck == 0) {
37 domAutomationController.setAutomationId(1); 32 domAutomationController.setAutomationId(1);
38 domAutomationController.send("SUCCESS"); 33 domAutomationController.send("SUCCESS");
39 } else { 34 } else {
40 g_swapsBeforeAck--; 35 g_swapsBeforeAck--;
41 document.getElementById('container').style.zIndex = g_swapsBeforeAck + 1; 36 document.getElementById('container').style.zIndex = g_swapsBeforeAck + 1;
42 window.webkitRequestAnimationFrame(waitForFinish); 37 window.webkitRequestAnimationFrame(waitForFinish);
43 } 38 }
44 } 39 }
40
41 main();
xidachen 2016/11/02 15:48:26 if we have a body onload="main()", do we still nee
zakerinasab 2016/11/02 15:54:55 Fixed.
45 </script> 42 </script>
46 </head> 43 </head>
47 <body onload="main()"> 44 <body onload="main()">»
xidachen 2016/11/02 15:48:26 why is there a small symbol here after the '>'?
zakerinasab 2016/11/02 15:54:55 Hm. It couldn't be seen in the editor. Fixed.
48 <div style="position:relative; width:300px; height:300px; background-color:white ">
49 </div>
50 <div id="container" style="position:absolute; top:0px; left:0px"> 45 <div id="container" style="position:absolute; top:0px; left:0px">
51 <canvas id="c" width="200" height="200" class="nomargin"></canvas> 46 <canvas id="c" width="140" height="140"></canvas>
52 </div> 47 </div>
53 </body> 48 </body>
54 </html> 49 </html>
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698