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

Unified Diff: LayoutTests/media/track/track-cue-rendering-after-controls-added.html

Issue 1156993013: New media playback UI. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: manual rebaseline. Created 5 years, 5 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: LayoutTests/media/track/track-cue-rendering-after-controls-added.html
diff --git a/LayoutTests/media/track/track-cue-rendering-after-controls-added.html b/LayoutTests/media/track/track-cue-rendering-after-controls-added.html
index c465d0d2809465dc0519f692159ed853a2cf4fd9..4ca57d079c02113e39ee0f5e26f98a287bd16b84 100644
--- a/LayoutTests/media/track/track-cue-rendering-after-controls-added.html
+++ b/LayoutTests/media/track/track-cue-rendering-after-controls-added.html
@@ -22,9 +22,14 @@ track.mode = "showing";
video.onloadeddata = function() {
testRunner.layoutAndPaintAsyncThen(function() {
- // Add the controls. The cue should move to avoid overlap.
+ // Add the controls. The cue should move to avoid overlap. Defer this
philipj_slow 2015/07/08 10:31:51 Was this test flaky before as well, or what has ch
liberato (no reviews please) 2015/07/09 12:10:53 this wasn't flaky. this change gives the schedule
+ // until the controls resize.
video.controls = true;
- testRunner.notifyDone();
+ // Wait for the relayout event to happen.
+ setTimeout(function()
+ {
+ testRunner.notifyDone();
+ }, 0);
});
};
</script>

Powered by Google App Engine
This is Rietveld 408576698