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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/security/offscreencanvas-placeholder-read-blocked-no-crossorigin.html

Issue 2738773004: No longer clamp setTimeout(..., 0) to 1ms. (Closed)
Patch Set: update webgl2 conformance expectations as well Created 3 years, 8 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: third_party/WebKit/LayoutTests/http/tests/security/offscreencanvas-placeholder-read-blocked-no-crossorigin.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/security/offscreencanvas-placeholder-read-blocked-no-crossorigin.html b/third_party/WebKit/LayoutTests/http/tests/security/offscreencanvas-placeholder-read-blocked-no-crossorigin.html
index dfb964311826dabfa86a82f173fa464bf0176f55..c33418b50181eab789941d97f2c8c94f05e347f0 100644
--- a/third_party/WebKit/LayoutTests/http/tests/security/offscreencanvas-placeholder-read-blocked-no-crossorigin.html
+++ b/third_party/WebKit/LayoutTests/http/tests/security/offscreencanvas-placeholder-read-blocked-no-crossorigin.html
@@ -16,20 +16,28 @@ async_test(t => {
t.step(function() {
canvas.toDataURL(); // Succeeds by not throwing
});
- setTimeout(function() { // sync barrier for commit() propagation
- t.step(function() {
- assert_throws("SecurityError", function() {
- canvas.toDataURL();
- }, "Check toDataURL blocked.");
- });
- ctx.commit(); // Second frame does not reset origin-clean flag.
- setTimeout(function() { // sync barrier for commit() propagation
+ // TODO(junov): Use the Promise returned by commit to schedule after the
+ // commit. (crbug.com/709484)
+ setTimeout(function() {
+ setTimeout(function() {
t.step(function() {
assert_throws("SecurityError", function() {
canvas.toDataURL();
- });
+ }, "Check toDataURL blocked.");
});
- t.done();
+ ctx.commit(); // Second frame does not reset origin-clean flag.
+ // TODO(junov): Use the Promise returned by commit to schedule after the
+ // commit. (crbug.com/709484)
+ setTimeout(function() {
+ setTimeout(function() {
+ t.step(function() {
+ assert_throws("SecurityError", function() {
+ canvas.toDataURL();
+ });
+ });
+ t.done();
+ }, 0);
+ }, 0);
}, 0);
}, 0);
}

Powered by Google App Engine
This is Rietveld 408576698