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

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

Issue 1018593004: Implement <video controls> dodging for text track layout (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: braces Created 5 years, 9 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-removed-expected.html
diff --git a/LayoutTests/media/track/track-webvtt-two-cue-layout-after-first-end-expected.html b/LayoutTests/media/track/track-cue-rendering-after-controls-removed-expected.html
similarity index 57%
copy from LayoutTests/media/track/track-webvtt-two-cue-layout-after-first-end-expected.html
copy to LayoutTests/media/track/track-cue-rendering-after-controls-removed-expected.html
index 2f009352d10deb666b242599a4e1c4ef7e73c02d..953c4584ada7c30c52d1e5c9c0eee635a7dec122 100644
--- a/LayoutTests/media/track/track-webvtt-two-cue-layout-after-first-end-expected.html
+++ b/LayoutTests/media/track/track-cue-rendering-after-controls-removed-expected.html
@@ -1,15 +1,21 @@
<!DOCTYPE html>
-<title>WebVTT two-cue layout after the first cue has ended (reference)</title>
+<title>Text track cue layout after controls are removed (reference)</title>
+<style>
+::cue {
+ font-size: 50px;
+}
+</style>
<video style="border:1px solid gray">
<source src="opera/track/webvtt/rendering/reftest/media/white.webm" type="video/webm">
<source src="opera/track/webvtt/rendering/reftest/media/white.mp4" type="video/mp4">
</video>
<script>
-// Add a single cue at line -2, where it would be if there was a first
+// Add a single cue at line -2, where it would be if there were controls visible
+// at the bottom. (This assumes that those controls are less than 50px high.)
// cue at line -1.
var video = document.querySelector("video");
var track = video.addTextTrack("captions");
-var cue = new VTTCue(0, 3, "cue 2");
+var cue = new VTTCue(0, 1, "text");
cue.line = -2;
track.addCue(cue);
track.mode = "showing";
« no previous file with comments | « LayoutTests/media/track/track-cue-rendering-after-controls-removed.html ('k') | Source/core/html/HTMLMediaElement.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698