Convert video-poster* tests to testharness.js

Convert video-poster* tests to testharness.js

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

[Srirama, 2016-07-08 18:31:47]

Description was changed from

==========
fix

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

to

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

Cleaning up video-poster* 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-07-08 18:39:06]

Patchset #1 (id:1) has been deleted

[Srirama, 2016-07-08 18:39:36]

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

[Srirama, 2016-07-08 18:39:36]

PTAL

[fs, 2016-07-12 17:52:00]

lgtm

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

https://codereview.chromium.org/2130383003/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-poster.html:10: url: null,
reflectedUrl: "", width: 320, height: 240, setSize: true
Maybe 'width' / 'height' should be 'expected*'.

https://codereview.chromium.org/2130383003/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-poster.html:58: if
((image.src).indexOf("bogus") > 0)
Nit: these parenthesis (around image.src) shouldn't be needed.

https://codereview.chromium.org/2130383003/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-poster.html:68: var url =
(video.poster).substr((video.poster).lastIndexOf("/media/") + 7);
Ditto

[Srirama, 2016-07-13 10:47:08]

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

https://codereview.chromium.org/2130383003/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-poster.html:10: url: null,
reflectedUrl: "", width: 320, height: 240, setSize: true
On 2016/07/12 17:52:00, fs (ooo to July 18) wrote:
> Maybe 'width' / 'height' should be 'expected*'.

Done.

https://codereview.chromium.org/2130383003/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-poster.html:58: if
((image.src).indexOf("bogus") > 0)
On 2016/07/12 17:52:00, fs (ooo to July 18) wrote:
> Nit: these parenthesis (around image.src) shouldn't be needed.

Done.

https://codereview.chromium.org/2130383003/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/media/video-poster.html:68: var url =
(video.poster).substr((video.poster).lastIndexOf("/media/") + 7);
On 2016/07/12 17:52:00, fs (ooo to July 18) wrote:
> Ditto

Done.

[Srirama, 2016-07-13 10:47:13]

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

[Srirama, 2016-07-13 10:47:14]

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

[commit-bot: I haz the power, 2016-07-13 10:47:28]

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-07-13 11:58:39]

Description was changed from

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

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

Cleaning up video-poster* 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-07-13 11:58:41]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-07-13 11:58:45]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-13 12:00:10]

Description was changed from

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

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

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

[commit-bot: I haz the power, 2016-07-13 12:00:11]

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