Convert track-mode* tests to testharness.js

Convert track-mode* tests to testharness.js

Cleaning up track-mode* 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/ee51763e46f6fa5c8d9ce7213f38a4b27df0685f
Cr-Commit-Position: refs/heads/master@{#391580}

[Srirama, 2016-05-01 11:54:50]

Description was changed from

==========
fix

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

to

==========
Convert track-mode* tests to testharness.js

Cleaning up track-mode* 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-01 11:58:12]

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

[Srirama, 2016-05-01 11:58:13]

PTAL

[mlamouri (slow - plz ping), 2016-05-04 13:55:25]

lgtm with comments applied.

https://codereview.chromium.org/1942633002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/media/track/track-mode-triggers-loading.html
(right):

https://codereview.chromium.org/1942633002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-mode-triggers-loading.html:23:
// Check metadata track state.
I think you can remove the comment.

https://codereview.chromium.org/1942633002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-mode-triggers-loading.html:31:
// Change metadata track mode so it loads.
Could you move the comment next to where you actually do it?

https://codereview.chromium.org/1942633002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/media/track/track-mode.html (right):

https://codereview.chromium.org/1942633002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-mode.html:48: function
setMode(value) {
can you call this function 'setModeAndCheck'. It's checking stuff which isn't
obvious when reading the callers' code.

[Srirama, 2016-05-04 17:35:05]

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

[Srirama, 2016-05-04 17:35:05]

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/1942633002/#ps20001
(title: "address comments")

[commit-bot: I haz the power, 2016-05-04 17:35:29]

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

[commit-bot: I haz the power, 2016-05-04 18:39:55]

Description was changed from

==========
Convert track-mode* tests to testharness.js

Cleaning up track-mode* 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-mode* tests to testharness.js

Cleaning up track-mode* 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-04 18:39:56]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-04 18:41:31]

Description was changed from

==========
Convert track-mode* tests to testharness.js

Cleaning up track-mode* 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-mode* tests to testharness.js

Cleaning up track-mode* 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/ee51763e46f6fa5c8d9ce7213f38a4b27df0685f
Cr-Commit-Position: refs/heads/master@{#391580}
==========

[commit-bot: I haz the power, 2016-05-04 18:41:32]

Patchset 2 (id:??) landed as
https://crrev.com/ee51763e46f6fa5c8d9ce7213f38a4b27df0685f
Cr-Commit-Position: refs/heads/master@{#391580}