Convert video-dom* tests to testharness.js.

Convert video-dom* tests to testharness.js.

Cleaning up video-dom* 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/b8425e4d5f1fb3237de1ca382cef56e67410f1ba
Cr-Commit-Position: refs/heads/master@{#400657}

[Srirama, 2016-06-19 13:55:45]

Patchset #1 (id:1) has been deleted

[Srirama, 2016-06-19 13:57:06]

Description was changed from

==========
fix

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

to

==========
Convert video-dom* tests to testharness.js.

Cleaning up video-dom* 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-19 14:04:13]

Patchset #1 (id:20001) has been deleted

[Srirama, 2016-06-19 14:04:31]

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

[Srirama, 2016-06-19 14:04:31]

PTAL

[fs, 2016-06-19 17:53:01]

LGTM w/ nits

https://codereview.chromium.org/2079203002/diff/40001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/media/video-dom-preload.html (right):

https://codereview.chromium.org/2079203002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-dom-preload.html:23: // Set via DOM
s/DOM/IDL attribute/

https://codereview.chromium.org/2079203002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-dom-preload.html:27: // - and via
attribute.
s/attribute/content attribute/

https://codereview.chromium.org/2079203002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-dom-preload.html:34: function
setPreload(value, expected) {
Maybe call this "checkPreloadValue" or so instead?

https://codereview.chromium.org/2079203002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-dom-preload.html:35: // Set via DOM
As above

https://codereview.chromium.org/2079203002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-dom-preload.html:40: // - and via
attribute.
Ditto

[Srirama, 2016-06-20 05:31:43]

https://codereview.chromium.org/2079203002/diff/40001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/media/video-dom-preload.html (right):

https://codereview.chromium.org/2079203002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-dom-preload.html:23: // Set via DOM
On 2016/06/19 17:53:01, fs wrote:
> s/DOM/IDL attribute/

Done.

https://codereview.chromium.org/2079203002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-dom-preload.html:27: // - and via
attribute.
On 2016/06/19 17:53:00, fs wrote:
> s/attribute/content attribute/

Done.

https://codereview.chromium.org/2079203002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-dom-preload.html:34: function
setPreload(value, expected) {
On 2016/06/19 17:53:00, fs wrote:
> Maybe call this "checkPreloadValue" or so instead?

Done.

https://codereview.chromium.org/2079203002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-dom-preload.html:35: // Set via DOM
On 2016/06/19 17:53:01, fs wrote:
> As above

Done.

https://codereview.chromium.org/2079203002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-dom-preload.html:40: // - and via
attribute.
On 2016/06/19 17:53:01, fs wrote:
> Ditto

Done.

[Srirama, 2016-06-20 05:31:51]

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

[Srirama, 2016-06-20 05:31:51]

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/2079203002/#ps60001
(title: "address nits")

[commit-bot: I haz the power, 2016-06-20 05:31:55]

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

[commit-bot: I haz the power, 2016-06-20 07:49:15]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-20 07:49:16]

Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Srirama, 2016-06-20 12:09:29]

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

[commit-bot: I haz the power, 2016-06-20 12:09:39]

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

[commit-bot: I haz the power, 2016-06-20 12:13:28]

Description was changed from

==========
Convert video-dom* tests to testharness.js.

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

Cleaning up video-dom* 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-20 12:13:29]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2016-06-20 12:15:51]

Description was changed from

==========
Convert video-dom* tests to testharness.js.

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

Cleaning up video-dom* 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/b8425e4d5f1fb3237de1ca382cef56e67410f1ba
Cr-Commit-Position: refs/heads/master@{#400657}
==========

[commit-bot: I haz the power, 2016-06-20 12:15:52]

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