Convert video-volume*, video-width* and viewport* tests to testharness.js

Convert video-volume*, video-width* and viewport* tests to testharness.js

Cleaning up video-volume*, video-width* and viewport* tests
in media/ 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/ba1f666447669a025c4a3aeadadd995c25f6f475
Cr-Commit-Position: refs/heads/master@{#402065}

[Srirama, 2016-06-24 12:39:19]

Description was changed from

==========
fix

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

to

==========
Convert video-volume*, video-width* and viewport* tests to testharness.js

Cleaning up video-volume*, video-width* and viewport* tests
in media/ 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-24 12:44:26]

Patchset #1 (id:1) has been deleted

[Srirama, 2016-06-24 12:45:27]

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

[Srirama, 2016-06-24 12:45:27]

PTAL

[fs, 2016-06-24 15:43:33]

lgtm

https://codereview.chromium.org/2094993002/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/media/video-volume.html (right):

https://codereview.chromium.org/2094993002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-volume.html:23:
video.oncanplaythrough = t.step_func_done(function () {
Nit: stray space after "function"

https://codereview.chromium.org/2094993002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/media/viewport-in-standalone-media-document.html
(right):

https://codereview.chromium.org/2094993002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/viewport-in-standalone-media-document.html:12:
iframe.onload = t.step_func_done(function() {
Because we're skipping/ignoring the first event, shouldn't this be a step_func
with an explicit t.done()?

[Srirama, 2016-06-25 14:19:58]

Patchset #2 (id:40001) has been deleted

[Srirama, 2016-06-25 14:34:44]

Patchset #2 (id:60001) has been deleted

[Srirama, 2016-06-25 14:39:26]

https://codereview.chromium.org/2094993002/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/media/video-volume.html (right):

https://codereview.chromium.org/2094993002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-volume.html:23:
video.oncanplaythrough = t.step_func_done(function () {
On 2016/06/24 15:43:32, fs wrote:
> Nit: stray space after "function"

Done.

https://codereview.chromium.org/2094993002/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/media/viewport-in-standalone-media-document.html
(right):

https://codereview.chromium.org/2094993002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/viewport-in-standalone-media-document.html:12:
iframe.onload = t.step_func_done(function() {
On 2016/06/24 15:43:32, fs wrote:
> Because we're skipping/ignoring the first event, shouldn't this be a step_func
> with an explicit t.done()?

Done. Now the event is fired only once as we set onload event a little late.

[Srirama, 2016-06-25 14:39:34]

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

[Srirama, 2016-06-25 14:39:34]

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

[commit-bot: I haz the power, 2016-06-25 14:39:42]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-25 15:49:50]

Description was changed from

==========
Convert video-volume*, video-width* and viewport* tests to testharness.js

Cleaning up video-volume*, video-width* and viewport* tests
in media/ 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-volume*, video-width* and viewport* tests to testharness.js

Cleaning up video-volume*, video-width* and viewport* tests
in media/ 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-25 15:49:51]

Committed patchset #2 (id:80001)

[commit-bot: I haz the power, 2016-06-25 15:51:16]

Description was changed from

==========
Convert video-volume*, video-width* and viewport* tests to testharness.js

Cleaning up video-volume*, video-width* and viewport* tests
in media/ 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-volume*, video-width* and viewport* tests to testharness.js

Cleaning up video-volume*, video-width* and viewport* tests
in media/ 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/ba1f666447669a025c4a3aeadadd995c25f6f475
Cr-Commit-Position: refs/heads/master@{#402065}
==========

[commit-bot: I haz the power, 2016-06-25 15:51:17]

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