Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 function enableAllTextTracks(textTracks) { | 1 function enableAllTextTracks(textTracks) { |
| 2 for (var i = 0; i < textTracks.length; i++) { | 2 for (var i = 0; i < textTracks.length; i++) { |
| 3 var track = textTracks[i]; | 3 var track = textTracks[i]; |
| 4 if (track.mode == "disabled") | 4 if (track.mode == "disabled") |
| 5 track.mode = "hidden"; | 5 track.mode = "hidden"; |
| 6 } | 6 } |
| 7 } | 7 } |
| 8 | 8 |
| 9 function assert_cues_equal(cues, expected) { | 9 function assert_cues_equal(cues, expected) { |
| 10 assert_equals(cues.length, expected.length); | 10 assert_equals(cues.length, expected.length); |
| 11 for (var i = 0; i < cues.length; i++) { | 11 for (var i = 0; i < cues.length; i++) { |
| 12 assert_equals(cues[i].id, expected[i].id); | 12 assert_equals(cues[i].id, expected[i].id); |
| 13 assert_equals(cues[i].startTime, expected[i].startTime); | 13 assert_equals(cues[i].startTime, expected[i].startTime); |
| 14 assert_equals(cues[i].endTime, expected[i].endTime); | 14 assert_equals(cues[i].endTime, expected[i].endTime); |
| 15 assert_equals(cues[i].text, expected[i].text); | 15 assert_equals(cues[i].text, expected[i].text); |
| 16 } | 16 } |
| 17 } | |
| 18 | |
| 19 function assert_cues_match(cues, expected, properties) { | |
| 20 assert_equals(cues.length, expected.length); | |
| 21 for (var i = 0; i < cues.length; i++) | |
| 22 for (var property of properties) | |
| 23 assert_equals(cues[property], expected[property]); | |
|
fs
2016/05/26 12:18:22
This needs to index in cues too. I'd suggest loadi
Srirama
2016/05/26 13:34:50
Done.
| |
| 17 } | 24 } |
| OLD | NEW |