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

Unified Diff: LayoutTests/media/track/track-cue-container-rendering-position.html

Issue 533073003: Force layout to make a track rendering test non-flaky (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 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
« no previous file with comments | « LayoutTests/TestExpectations ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/media/track/track-cue-container-rendering-position.html
diff --git a/LayoutTests/media/track/track-cue-container-rendering-position.html b/LayoutTests/media/track/track-cue-container-rendering-position.html
index 2167967d1c8bfb56b10e114d26b8cc8ff179c6ab..30c02df8b71e307f58f18907aecb2de8bf8f3ce5 100644
--- a/LayoutTests/media/track/track-cue-container-rendering-position.html
+++ b/LayoutTests/media/track/track-cue-container-rendering-position.html
@@ -20,6 +20,7 @@
consoleWrite("");
cueDisplayElement = textTrackDisplayElement(video, 'display', 0);
+ document.body.offsetTop; // Force layout.
testExpected("cueDisplayElement.offsetTop > (video.videoHeight * .75)", true);
fs 2014/09/03 10:33:02 Any idea why reading cueDisplayElement.offsetTop i
philipj_slow 2014/09/03 11:31:55 While debugging I replaced it with a constant and
endTest();
}
« no previous file with comments | « LayoutTests/TestExpectations ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698