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-default* and track-disabled* 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/056826d0477d09a94adfaec69a8c2500dc12a9f8
Cr-Commit-Position: refs/heads/master@{#390643}

[Srirama, 2016-04-28 18:27:20]

Description was changed from

==========
fix

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

to

==========
Convert track tests from video-test.js to testharness.js based
Cleaning up track-default* and track-delete* 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-28 18:27:53]

Description was changed from

==========
Convert track tests from video-test.js to testharness.js based
Cleaning up track-default* and track-delete* 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-default* and track-delete* 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-28 18:28:40]

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

[Srirama, 2016-04-28 18:28:40]

PTAL

[Srirama, 2016-04-28 18:29:11]

Description was changed from

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

Cleaning up track-default* and track-delete* 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-default* and track-disabled* 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-04-29 12:39:12]

lgtm

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

https://codereview.chromium.org/1925243002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-default-attribute.html:18:
assert_equals(event.target.default, true);
assert_true()

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

https://codereview.chromium.org/1925243002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-disabled-addcue.html:21: if
(e.target.currentTime < 2 * cueDuration)
nit: can you add parenthesis?

[Srirama, 2016-04-29 13:17:08]

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

https://codereview.chromium.org/1925243002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-default-attribute.html:18:
assert_equals(event.target.default, true);
On 2016/04/29 12:39:12, Mounir Lamouri wrote:
> assert_true()

Done.

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

https://codereview.chromium.org/1925243002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-disabled-addcue.html:21: if
(e.target.currentTime < 2 * cueDuration)
On 2016/04/29 12:39:12, Mounir Lamouri wrote:
> nit: can you add parenthesis?

Done.

[Srirama, 2016-04-29 13:17:14]

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

[Srirama, 2016-04-29 13:17:14]

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

[commit-bot: I haz the power, 2016-04-29 13:17:28]

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

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

Description was changed from

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

Cleaning up track-default* and track-disabled* 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-default* and track-disabled* 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-29 14:21:40]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-30 17:25:42]

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

[commit-bot: I haz the power, 2016-04-30 18:00:33]

Description was changed from

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

Cleaning up track-default* and track-disabled* 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-default* and track-disabled* 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/056826d0477d09a94adfaec69a8c2500dc12a9f8
Cr-Commit-Position: refs/heads/master@{#390643}
==========