Convert track tests from video-test.js to testharness.js based

Convert track tests from video-test.js to testharness.js based

Cleaning up track-cue-nothing* and track-cue-rendering* 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/e302d364c9bb07162b7902d88386c06629a91e39
Cr-Commit-Position: refs/heads/master@{#388761}

[Srirama, 2016-04-14 07:23:49]

Description was changed from

==========
fix


fix

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

to

==========
Convert track tests from video-test.js to testharness.js based

Cleaning up track-cue-nothing* and track-cue-rendering* 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-04-14 07:26:26]

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

[Srirama, 2016-04-14 07:26:27]

PTAL

[mlamouri (slow - plz ping), 2016-04-19 15:16:29]

https://codereview.chromium.org/1887943002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/media/track/track-cue-nothing-to-render.html
(right):

https://codereview.chromium.org/1887943002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-cue-nothing-to-render.html:37:
assert_equals(textTrackDisplayElement(video, "display").innerText, cueText);
It seems that this could be outside of the if/else block.

[Srirama, 2016-04-19 16:31:02]

https://codereview.chromium.org/1887943002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/media/track/track-cue-nothing-to-render.html
(right):

https://codereview.chromium.org/1887943002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-cue-nothing-to-render.html:37:
assert_equals(textTrackDisplayElement(video, "display").innerText, cueText);
On 2016/04/19 15:16:29, Mounir Lamouri wrote:
> It seems that this could be outside of the if/else block.

As i mentioned in my previous CL, this is also same.
When we call "textTrackDisplayElement(video, 'display')" it throws 
"No text track cue with display id '-webkit-media-text-track-display' is
currently visible" error as there is no active cue (captions-gaps.vtt doesn't
have cues matching for 0.5, 2.5, 0.6 and 4.4). So i moved it under the if
condition to avoid printing this error in the output).

[mlamouri (slow - plz ping), 2016-04-21 12:27:52]

lgtm

[philipj_slow, 2016-04-21 12:46:16]

https://codereview.chromium.org/1887943002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/media/track/track-cue-nothing-to-render.html
(right):

https://codereview.chromium.org/1887943002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-cue-nothing-to-render.html:48:
video.currentTime = 0.5;
Use info[0] here?

[philipj_slow, 2016-04-21 12:48:17]

lgtm % nits

https://codereview.chromium.org/1887943002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/media/track/track-cue-rendering-on-resize.html
(right):

https://codereview.chromium.org/1887943002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-cue-rendering-on-resize.html:2:
<title>Test that Line Height of cue element isn't overridden by author
CSS.</title>
No need to capitalize Line Height

[Srirama, 2016-04-21 13:02:36]

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

[Srirama, 2016-04-21 13:02:36]

The patchset sent to the CQ was uploaded after l-g-t-m from philipj@opera.com,
mlamouri@chromium.org
Link to the patchset: https://codereview.chromium.org/1887943002/#ps20001
(title: "address nits")

[commit-bot: I haz the power, 2016-04-21 13:02:45]

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

[commit-bot: I haz the power, 2016-04-21 14:00:29]

Description was changed from

==========
Convert track tests from video-test.js to testharness.js based

Cleaning up track-cue-nothing* and track-cue-rendering* 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 tests from video-test.js to testharness.js based

Cleaning up track-cue-nothing* and track-cue-rendering* 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-04-21 14:00:30]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-22 19:34:18]

Description was changed from

==========
Convert track tests from video-test.js to testharness.js based

Cleaning up track-cue-nothing* and track-cue-rendering* 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 tests from video-test.js to testharness.js based

Cleaning up track-cue-nothing* and track-cue-rendering* 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/e302d364c9bb07162b7902d88386c06629a91e39
Cr-Commit-Position: refs/heads/master@{#388761}
==========

[commit-bot: I haz the power, 2016-04-22 19:34:19]

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