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

Unified Diff: content/test/data/gpu/pixel_offscreenCanvas_2D_commit_main.html

Issue 2358833002: Update pixel test expectation and add new worker tests (Closed)
Patch Set: minor change in test description Created 4 years, 3 months 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 side-by-side diff with in-line comments
Download patch
Index: content/test/data/gpu/pixel_offscreenCanvas_2D_commit_main.html
diff --git a/content/test/data/gpu/pixel_offscreenCanvas_webgl_commit_main.html b/content/test/data/gpu/pixel_offscreenCanvas_2D_commit_main.html
similarity index 83%
copy from content/test/data/gpu/pixel_offscreenCanvas_webgl_commit_main.html
copy to content/test/data/gpu/pixel_offscreenCanvas_2D_commit_main.html
index 3ae0e56c24d2c1864f35277dbaa6b75ff33191d1..56231da74b822e721c9071d7a4b9b65e401687f7 100644
--- a/content/test/data/gpu/pixel_offscreenCanvas_webgl_commit_main.html
+++ b/content/test/data/gpu/pixel_offscreenCanvas_2D_commit_main.html
@@ -8,7 +8,7 @@ that the baseline images are regenerated on the next run.
<html>
<head>
-<title>OffscreenCanvas commit flow on main thread: green square on white background.</title>
+<title>OffscreenCanvas 2D commit flow on main thread: green square on white background.</title>
<style type="text/css">
.nomargin {
margin: 0px auto;
@@ -27,10 +27,8 @@ function draw()
{
var canvas = document.getElementById("c");
var offscreenCanvas = canvas.transferControlToOffscreen();
- var gl = offscreenCanvas.getContext("webgl");
- gl.clearColor(0.0, 1.0, 0.0, 1.0);
- gl.clear(gl.COLOR_BUFFER_BIT);
- gl.commit();
+ var offscreen2d = offscreenCanvas.getContext("2d");
+ offscreen2d.commit();
Ken Russell (switch to Gerrit) 2016/09/22 00:50:36 This doesn't seem to do anything with the context
xidachen 2016/09/22 00:58:15 Sorry that this is confusing. I forgot to clarify:
Ken Russell (switch to Gerrit) 2016/09/22 01:12:09 Yes, please either: 1) Remove the 2D canvas tests
}
function waitForFinish()

Powered by Google App Engine
This is Rietveld 408576698