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

Unified Diff: third_party/WebKit/LayoutTests/media/track/regions-webvtt/vtt-region-display.html

Issue 2682333002: Implement VTTCue.region and sync the VTTRegion interface (Closed)
Patch Set: Add DCHECK Created 3 years, 10 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/media/track/regions-webvtt/vtt-region-display.html
diff --git a/third_party/WebKit/LayoutTests/media/track/regions-webvtt/vtt-region-display.html b/third_party/WebKit/LayoutTests/media/track/regions-webvtt/vtt-region-display.html
index aaa884b941c4f67ffbc91604d554dac90a0e944b..a6eba0cc04a97040ae236213977c3e642cc7dc04 100644
--- a/third_party/WebKit/LayoutTests/media/track/regions-webvtt/vtt-region-display.html
+++ b/third_party/WebKit/LayoutTests/media/track/regions-webvtt/vtt-region-display.html
@@ -49,10 +49,10 @@ async_test(function() {
for (var i = 0; i < container.children.length; ++i) {
var cue = container.children[i];
- totalVisibleLines += countVisibleLines(cue);;
+ totalVisibleLines += countVisibleLines(cue);
}
- assert_less_than_equal(totalVisibleLines, track.regions[0].height);
+ assert_less_than_equal(totalVisibleLines, track.regions[0].lines);
if (seekTimeIndex == seekTimes.length)
this.done();

Powered by Google App Engine
This is Rietveld 408576698