Convert track-remove* tests to testharness.js

Convert track-remove* tests to testharness.js

Cleaning up track-remove* 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/4e8d672b8e5d2c1e3f81eb9a14d7cb6199d73526
Cr-Commit-Position: refs/heads/master@{#392060}

[Srirama, 2016-05-05 14:02:07]

Description was changed from

==========
fix


fix


fix

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

to

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

Cleaning up track-remove* 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-05 14:03:31]

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

[Srirama, 2016-05-05 14:03:31]

PTAL

[mlamouri (slow - plz ping), 2016-05-06 12:58:28]

lgtm

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

https://codereview.chromium.org/1950283002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-removal-crash.html:19:
setTimeout(CFcrash, 10);
step_func?

https://codereview.chromium.org/1950283002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-removal-crash.html:27:
setTimeout(runGCAndEndTest, 0);
step_func_done

https://codereview.chromium.org/1950283002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/media/track/track-remove-by-setting-innerHTML.html
(right):

https://codereview.chromium.org/1950283002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-remove-by-setting-innerHTML.html:21:
video.innerHTML = 'This is a test';
`video.innerHTML = '';`

I think passing the "This is a test" string is more confusing than anything.

[Srirama, 2016-05-06 14:17:08]

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

https://codereview.chromium.org/1950283002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-removal-crash.html:19:
setTimeout(CFcrash, 10);
On 2016/05/06 12:58:28, Mounir Lamouri wrote:
> step_func?

Done.

https://codereview.chromium.org/1950283002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-removal-crash.html:27:
setTimeout(runGCAndEndTest, 0);
On 2016/05/06 12:58:28, Mounir Lamouri wrote:
> step_func_done

Done.

https://codereview.chromium.org/1950283002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/media/track/track-remove-by-setting-innerHTML.html
(right):

https://codereview.chromium.org/1950283002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-remove-by-setting-innerHTML.html:21:
video.innerHTML = 'This is a test';
On 2016/05/06 12:58:28, Mounir Lamouri wrote:
> `video.innerHTML = '';`
> 
> I think passing the "This is a test" string is more confusing than anything.

Done.

[Srirama, 2016-05-06 14:17:17]

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

[Srirama, 2016-05-06 14:17:17]

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

[commit-bot: I haz the power, 2016-05-06 14:17:30]

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

[commit-bot: I haz the power, 2016-05-06 15:20:56]

Description was changed from

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

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

Cleaning up track-remove* 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-06 15:20:57]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-06 15:23:11]

Description was changed from

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

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

Cleaning up track-remove* 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/4e8d672b8e5d2c1e3f81eb9a14d7cb6199d73526
Cr-Commit-Position: refs/heads/master@{#392060}
==========

[commit-bot: I haz the power, 2016-05-06 15:23:12]

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