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

Unified Diff: third_party/WebKit/LayoutTests/paint/invalidation/canvas-composite-repaint-by-all-imagesource.html

Issue 2499883002: Media Controls: handle 'timeupdate', 'play' and 'pause' via an EventListener. (Closed)
Patch Set: fix repaint test Created 4 years 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/paint/invalidation/canvas-composite-repaint-by-all-imagesource.html
diff --git a/third_party/WebKit/LayoutTests/paint/invalidation/canvas-composite-repaint-by-all-imagesource.html b/third_party/WebKit/LayoutTests/paint/invalidation/canvas-composite-repaint-by-all-imagesource.html
index a00c00d5192761eec9162a60e966cc73916a045b..3545c6e1689ee1e20e338a5d1c9ae25d39ecc819 100644
--- a/third_party/WebKit/LayoutTests/paint/invalidation/canvas-composite-repaint-by-all-imagesource.html
+++ b/third_party/WebKit/LayoutTests/paint/invalidation/canvas-composite-repaint-by-all-imagesource.html
@@ -156,13 +156,15 @@
// we can start this test after the video can be played.
function startTest() {
- video.removeEventListener("playing", startTest, true);
prepareRepaintTest();
runRepaintTest();
}
var video = document.getElementById("video");
- video.addEventListener("playing", startTest, true);
+ video.addEventListener("playing", _ => {
+ // Video controls will do a re-paint when it starts playing.
+ setTimeout(startTest);
+ }, { passive: true, once: true });
video.play();
var imageElement = document.getElementById('image');

Powered by Google App Engine
This is Rietveld 408576698