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

Unified Diff: third_party/WebKit/LayoutTests/media/track/track-webvtt-tc018-align-text-line-position.html

Issue 2683633006: [WebVTT] Rename Middle to Center (Closed)
Patch Set: fix tests 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/track-webvtt-tc018-align-text-line-position.html
diff --git a/third_party/WebKit/LayoutTests/media/track/track-webvtt-tc018-align-text-line-position.html b/third_party/WebKit/LayoutTests/media/track/track-webvtt-tc018-align-text-line-position.html
index b4250fbeecc04e3e7c46fef333cfe249a7ecf4f2..18500e43567c495beb77aacbca510d9d8d51c967 100644
--- a/third_party/WebKit/LayoutTests/media/track/track-webvtt-tc018-align-text-line-position.html
+++ b/third_party/WebKit/LayoutTests/media/track/track-webvtt-tc018-align-text-line-position.html
@@ -33,9 +33,9 @@ async_test(function(t) {
var expected = [
{ align : "start", position : 10, line : 0, snapToLines : false },
{ align : "start", position : "auto", line : 0, snapToLines : true },
- { align : "middle", position : 80, line : 80, snapToLines : false },
+ { align : "center", position : 80, line : 80, snapToLines : false },
{ align : "end", position : 30, line : 5, snapToLines : true },
- { align : "middle", position : 60, line : -3, snapToLines : true }
+ { align : "center", position : 60, line : -3, snapToLines : true }
];
assert_cues_match(video.textTracks[index].cues, expected);
@@ -43,12 +43,12 @@ async_test(function(t) {
function testTrackError(index) {
var expected = [
- { align : "middle", position : "auto", line : "auto", snapToLines : true },
+ { align : "center", position : "auto", line : "auto", snapToLines : true },
{ align : "end", position : 0, line : "auto", snapToLines : true },
- { align : "middle", position : 60, line : -3, snapToLines : true }
+ { align : "center", position : 60, line : -3, snapToLines : true }
];
assert_cues_match(video.textTracks[index].cues, expected);
}
});
-</script>
+</script>

Powered by Google App Engine
This is Rietveld 408576698