Convert track-language* and track-large* tests to testharness.js

Convert track-language* and track-large* tests to testharness.js

Cleaning up track-language* and track-large* 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/f43cd3e41312f5edbcdcd703fcc24f5ea63a223a
Cr-Commit-Position: refs/heads/master@{#391588}

[Srirama, 2016-04-29 19:07:00]

Description was changed from

==========
fix

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

to

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

Cleaning up track-language* and track-large* 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-29 19:08:30]

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

[Srirama, 2016-04-29 19:08:31]

PTAL

[ddorwin, 2016-04-29 21:47:48]

OOC, is there a reason this isn't combined with
https://codereview.chromium.org/1937483002/? Please at least indicate the
difference in the first line of the commit message. (They currently appear to be
the same change.)

[Srirama, 2016-04-30 03:24:16]

On 2016/04/29 21:47:48, ddorwin wrote:
> OOC, is there a reason this isn't combined with
> https://codereview.chromium.org/1937483002/ Please at least indicate the
> difference in the first line of the commit message. (They currently appear to
be
> the same change.)

There is no reason other than making it easier for Mounir to review. I thought
first line of commit msg becomes too long to add differentiation. I think i can
add a few more words to differentiate them as you said. In case if you have time
to review these changes please let me know what is a good number of test cases
per CL. Still there are many test cases in media/ and track/ folders.

[ddorwin, 2016-04-30 18:55:22]

On 2016/04/30 03:24:16, Srirama wrote:
> On 2016/04/29 21:47:48, ddorwin wrote:
> > OOC, is there a reason this isn't combined with
> > https://codereview.chromium.org/1937483002/ Please at least indicate the
> > difference in the first line of the commit message. (They currently appear
to
> be
> > the same change.)
> 
> There is no reason other than making it easier for Mounir to review. I thought
> first line of commit msg becomes too long to add differentiation. I think i
can
> add a few more words to differentiate them as you said. In case if you have
time
> to review these changes please let me know what is a good number of test cases
> per CL. Still there are many test cases in media/ and track/ folders.

I'll leave the size preference up to the reviewer.

You can probably provide enough detail by changing the message to something
like:
Convert track-language* and track-large* tests to testharness.js.

[Srirama, 2016-05-01 11:26:08]

On 2016/04/30 18:55:22, ddorwin wrote:
> On 2016/04/30 03:24:16, Srirama wrote:
> > On 2016/04/29 21:47:48, ddorwin wrote:
> > > OOC, is there a reason this isn't combined with
> > > https://codereview.chromium.org/1937483002/ Please at least indicate the
> > > difference in the first line of the commit message. (They currently appear
> to
> > be
> > > the same change.)
> > 
> > There is no reason other than making it easier for Mounir to review. I
thought
> > first line of commit msg becomes too long to add differentiation. I think i
> can
> > add a few more words to differentiate them as you said. In case if you have
> time
> > to review these changes please let me know what is a good number of test
cases
> > per CL. Still there are many test cases in media/ and track/ folders.
> 
> I'll leave the size preference up to the reviewer.
> 
> You can probably provide enough detail by changing the message to something
> like:
> Convert track-language* and track-large* tests to testharness.js.

Thanks for the suggestion, that looks much better, i will change it.

[Srirama, 2016-05-01 11:26:55]

Description was changed from

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

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

Cleaning up track-language* and track-large* 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
==========

[mlamouri (slow - plz ping), 2016-05-04 14:11:41]

lgtm

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

https://codereview.chromium.org/1938533002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-language-preference.html:37:
expectedLanguage : "es-MX"
Interesting.

https://codereview.chromium.org/1938533002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-language-preference.html:80:
track = event.target;
`var track`

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

https://codereview.chromium.org/1938533002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-large-timestamp.html:17:
assert_equals(cue.endTime / (60 * 60), 1234567890);
nit: you could also do " / 3600". I think readers should understand what that
meant.

[Srirama, 2016-05-04 17:56:48]

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

[Srirama, 2016-05-04 17:56:48]

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

[commit-bot: I haz the power, 2016-05-04 17:57:19]

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

[commit-bot: I haz the power, 2016-05-04 19:01:33]

Description was changed from

==========
Convert track-language* and track-large* tests to testharness.js

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

Cleaning up track-language* and track-large* 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 19:01:34]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-04 19:03:58]

Description was changed from

==========
Convert track-language* and track-large* tests to testharness.js

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

Cleaning up track-language* and track-large* 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/f43cd3e41312f5edbcdcd703fcc24f5ea63a223a
Cr-Commit-Position: refs/heads/master@{#391588}
==========

[commit-bot: I haz the power, 2016-05-04 19:03:59]

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