Convert track-webvtt-tc[008-010] tests to testharness.js

Convert track-webvtt-tc[008-010] tests to testharness.js

Cleaning up track-webvtt-tc[008-010] tests in media/track
to use testharness.js instead of video-test.js.
This will enable to upstream these tests to web-platform-tests.

BUG=588956
Committed: https://crrev.com/385c4876b0872dda62dc8e37511c5910e74216a8
Cr-Commit-Position: refs/heads/master@{#395584}

[Srirama, 2016-05-19 18:50:28]

Patchset #1 (id:1) has been deleted

[Srirama, 2016-05-19 18:51:48]

Description was changed from

==========
fix


fix

BUG=
==========

to

==========
Convert track-webvtt-tc[008-010] tests to testharness.js 

Cleaning up track-webvtt-tc[008-010] tests in media/track
to use testharness.js instead of video-test.js.
This will enable to upstream these tests to web-platform-tests. 

BUG=588956
==========

[Srirama, 2016-05-19 18:54:33]

srirama.m@samsung.com changed reviewers:
+ mlamouri@chromium.org

[Srirama, 2016-05-19 18:54:34]

PTAL

[mlamouri (slow - plz ping), 2016-05-20 12:44:26]

https://codereview.chromium.org/1962593002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc008-timings-no-hours.html
(right):

https://codereview.chromium.org/1962593002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc008-timings-no-hours.html:61:
assert_cues_equal(cues, expected);
Same as the other review, you can test the length in `assert_cues_equal`.

[Srirama, 2016-05-20 13:06:25]

Patchset #2 (id:40001) has been deleted

[Srirama, 2016-05-20 13:25:28]

Patchset #2 (id:60001) has been deleted

[Srirama, 2016-05-20 13:37:08]

https://codereview.chromium.org/1962593002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc008-timings-no-hours.html
(right):

https://codereview.chromium.org/1962593002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc008-timings-no-hours.html:61:
assert_cues_equal(cues, expected);
On 2016/05/20 12:44:26, Mounir Lamouri wrote:
> Same as the other review, you can test the length in `assert_cues_equal`.

Done.

[mlamouri (slow - plz ping), 2016-05-23 23:17:32]

lgtm

[Srirama, 2016-05-24 03:16:11]

The CQ bit was checked by srirama.m@samsung.com

[commit-bot: I haz the power, 2016-05-24 03:16:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1962593002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1962593002/80001

[commit-bot: I haz the power, 2016-05-24 04:21:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-24 04:21:55]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Srirama, 2016-05-24 07:15:52]

The CQ bit was checked by srirama.m@samsung.com

[Srirama, 2016-05-24 07:15:52]

The patchset sent to the CQ was uploaded after l-g-t-m from
mlamouri@chromium.org
Link to the patchset: https://codereview.chromium.org/1962593002/#ps100001
(title: "fix test failure")

[commit-bot: I haz the power, 2016-05-24 07:16:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1962593002/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1962593002/100001

[commit-bot: I haz the power, 2016-05-24 08:24:00]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-24 08:24:01]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Srirama, 2016-05-24 09:29:02]

Patchset #3 (id:100001) has been deleted

[Srirama, 2016-05-24 13:12:13]

The CQ bit was checked by srirama.m@samsung.com

[Srirama, 2016-05-24 13:12:13]

The patchset sent to the CQ was uploaded after l-g-t-m from
mlamouri@chromium.org
Link to the patchset: https://codereview.chromium.org/1962593002/#ps120001
(title: "fix test failure")

[commit-bot: I haz the power, 2016-05-24 13:12:29]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1962593002/120001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1962593002/120001

[commit-bot: I haz the power, 2016-05-24 13:16:15]

Description was changed from

==========
Convert track-webvtt-tc[008-010] tests to testharness.js 

Cleaning up track-webvtt-tc[008-010] tests in media/track
to use testharness.js instead of video-test.js.
This will enable to upstream these tests to web-platform-tests. 

BUG=588956
==========

to

==========
Convert track-webvtt-tc[008-010] tests to testharness.js 

Cleaning up track-webvtt-tc[008-010] tests in media/track
to use testharness.js instead of video-test.js.
This will enable to upstream these tests to web-platform-tests. 

BUG=588956
==========

[commit-bot: I haz the power, 2016-05-24 13:16:17]

Committed patchset #3 (id:120001)

[commit-bot: I haz the power, 2016-05-24 13:17:32]

Description was changed from

==========
Convert track-webvtt-tc[008-010] tests to testharness.js 

Cleaning up track-webvtt-tc[008-010] tests in media/track
to use testharness.js instead of video-test.js.
This will enable to upstream these tests to web-platform-tests. 

BUG=588956
==========

to

==========
Convert track-webvtt-tc[008-010] tests to testharness.js

Cleaning up track-webvtt-tc[008-010] tests in media/track
to use testharness.js instead of video-test.js.
This will enable to upstream these tests to web-platform-tests.

BUG=588956

Committed: https://crrev.com/385c4876b0872dda62dc8e37511c5910e74216a8
Cr-Commit-Position: refs/heads/master@{#395584}
==========

[commit-bot: I haz the power, 2016-05-24 13:17:33]

Patchset 3 (id:??) landed as
https://crrev.com/385c4876b0872dda62dc8e37511c5910e74216a8
Cr-Commit-Position: refs/heads/master@{#395584}