Convert video-src-invalid* and video-src-none* tests to testharness.js

Convert video-src-invalid* and video-src-none* tests to testharness.js

Cleaning up video-src-invalid* and video-src-none* 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/b63b47f4b9753783c34028662858b1e7788c3010
Cr-Commit-Position: refs/heads/master@{#402727}

[Srirama, 2016-06-25 13:52:02]

Description was changed from

==========
fix

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

to

==========
Convert video-src-invalid* and video-src-none* tests to testharness.js

Cleaning up video-src-invalid* and video-src-none* 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-25 13:56:33]

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

[Srirama, 2016-06-25 13:56:33]

PTAL

[Srirama, 2016-06-25 13:59:03]

Patchset #1 (id:1) has been deleted

[ilovebobby906, 2016-06-25 14:10:45]

On 2016/06/25 13:52:02, Srirama wrote:
> Description was changed from
> 
> ==========
> fix
> 
> BUG=
> ==========
> 
> to
> 
> ==========
> Convert video-src-invalid* and video-src-none* tests to testharness.js
> 
> Cleaning up video-src-invalid* and video-src-none* 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
> ==========

Cm

[fs, 2016-06-25 16:38:40]

https://codereview.chromium.org/2096223002/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/media/video-src-invalid-poster.html (right):

https://codereview.chromium.org/2096223002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-src-invalid-poster.html:11:
setTimeout(t.step_func_done(function() {
Can we guarantee that the poster image has finished loading in the "task"
following the task firing 'loadstart'? I guess it's roughly as gratuitous as
20ms though... (One option might be to add an <img> that loads the same source
and use the load event of that - and assume that they reference the same
resource internally...)

[Srirama, 2016-06-28 17:02:51]

https://codereview.chromium.org/2096223002/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/media/video-src-invalid-poster.html (right):

https://codereview.chromium.org/2096223002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-src-invalid-poster.html:11:
setTimeout(t.step_func_done(function() {
On 2016/06/25 16:38:39, fs wrote:
> Can we guarantee that the poster image has finished loading in the "task"
> following the task firing 'loadstart'? I guess it's roughly as gratuitous as
> 20ms though... (One option might be to add an <img> that loads the same source
> and use the load event of that - and assume that they reference the same
> resource internally...) 

Done.

[fs, 2016-06-28 17:24:03]

lgtm

https://codereview.chromium.org/2096223002/diff/40001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/media/video-src-invalid-poster.html (right):

https://codereview.chromium.org/2096223002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-src-invalid-poster.html:13: // With
this we can be sure that the poster is loaded.
s/sure/be reasonably sure/ I think - to stay on the conservative side =)

[Srirama, 2016-06-28 17:39:26]

https://codereview.chromium.org/2096223002/diff/40001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/media/video-src-invalid-poster.html (right):

https://codereview.chromium.org/2096223002/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-src-invalid-poster.html:13: // With
this we can be sure that the poster is loaded.
On 2016/06/28 17:24:03, fs wrote:
> s/sure/be reasonably sure/ I think - to stay on the conservative side =)

Done.

[Srirama, 2016-06-28 17:39:35]

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

[Srirama, 2016-06-28 17:39:35]

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

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

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-28 20:40:55]

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

[commit-bot: I haz the power, 2016-06-28 20:40:56]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Srirama, 2016-06-29 03:45:04]

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

[commit-bot: I haz the power, 2016-06-29 03:45:37]

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-29 05:08:59]

Description was changed from

==========
Convert video-src-invalid* and video-src-none* tests to testharness.js

Cleaning up video-src-invalid* and video-src-none* 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-src-invalid* and video-src-none* tests to testharness.js

Cleaning up video-src-invalid* and video-src-none* 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-29 05:09:00]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-06-29 05:10:35]

Description was changed from

==========
Convert video-src-invalid* and video-src-none* tests to testharness.js

Cleaning up video-src-invalid* and video-src-none* 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-src-invalid* and video-src-none* tests to testharness.js

Cleaning up video-src-invalid* and video-src-none* 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/b63b47f4b9753783c34028662858b1e7788c3010
Cr-Commit-Position: refs/heads/master@{#402727}
==========

[commit-bot: I haz the power, 2016-06-29 05:10:36]

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