Convert media-element*, track-cue* and track-remove* tests to testharness.js

Convert media-element*, track-cue* and track-remove* tests to testharness.js

Cleaning up media-element*, track-cue* and 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/2e0d99c0e86f5dc6624817f27ba76884f5cbbf83
Cr-Commit-Position: refs/heads/master@{#400393}

[Srirama, 2016-06-16 13:28:13]

Description was changed from

==========
fix

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

to

==========
Convert media-element*, track-cue* and track-remove* tests to testharness.js

Cleaning up media-element*, track-cue* and 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-06-16 13:30:44]

srirama.m@samsung.com changed reviewers:
+ foolip@chromium.org, fs@opera.com

[Srirama, 2016-06-16 13:30:45]

PTAL

https://codereview.chromium.org/2078443003/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/media/track/track-cue-gc-wrapper.html
(right):

https://codereview.chromium.org/2078443003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-cue-gc-wrapper.html:26:
cue.addEventListener("enter", checkNativeProperty);
I am not sure why using cue.onenter isn't working here (custom property check is
failing). Any idea?

[fs, 2016-06-16 22:58:50]

lgtm

https://codereview.chromium.org/2078443003/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/media/track/track-cue-gc-wrapper.html
(right):

https://codereview.chromium.org/2078443003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-cue-gc-wrapper.html:26:
cue.addEventListener("enter", checkNativeProperty);
On 2016/06/16 at 13:30:45, Srirama wrote:
> I am not sure why using cue.onenter isn't working here (custom property check
is failing). Any idea?

Sounds odd. Is it failing because 'currentCue' (i.e transitively 'this') is not
the/a cue, or because it's lacking the property?

[Srirama, 2016-06-17 06:30:13]

https://codereview.chromium.org/2078443003/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/media/track/track-cue-gc-wrapper.html
(right):

https://codereview.chromium.org/2078443003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-cue-gc-wrapper.html:26:
cue.addEventListener("enter", checkNativeProperty);
On 2016/06/16 22:58:50, fs wrote:
> On 2016/06/16 at 13:30:45, Srirama wrote:
> > I am not sure why using cue.onenter isn't working here (custom property
check
> is failing). Any idea?
> 
> Sounds odd. Is it failing because 'currentCue' (i.e transitively 'this') is
not
> the/a cue, or because it's lacking the property?

Problem is with "this", as instanceof is failing. Worked fine when i added
"event" argument and used event.target.

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

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

[Srirama, 2016-06-17 06:40:06]

The patchset sent to the CQ was uploaded after l-g-t-m from fs@opera.com
Link to the patchset: https://codereview.chromium.org/2078443003/#ps20001
(title: "Use "onenter" event attribute")

[commit-bot: I haz the power, 2016-06-17 06:40:23]

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

[commit-bot: I haz the power, 2016-06-17 07:53:56]

Description was changed from

==========
Convert media-element*, track-cue* and track-remove* tests to testharness.js

Cleaning up media-element*, track-cue* and 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 media-element*, track-cue* and track-remove* tests to testharness.js

Cleaning up media-element*, track-cue* and 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-06-17 07:53:57]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-17 07:55:33]

Description was changed from

==========
Convert media-element*, track-cue* and track-remove* tests to testharness.js

Cleaning up media-element*, track-cue* and 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 media-element*, track-cue* and track-remove* tests to testharness.js

Cleaning up media-element*, track-cue* and 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/2e0d99c0e86f5dc6624817f27ba76884f5cbbf83
Cr-Commit-Position: refs/heads/master@{#400393}
==========

[commit-bot: I haz the power, 2016-06-17 07:55:35]

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