Convert video-seek* tests to testharness.js

Convert video-seek* tests to testharness.js

Cleaning up video-seek* 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/278bd74563e00bf0abbae6594c34b1680a7ed428
Cr-Commit-Position: refs/heads/master@{#402169}

[Srirama, 2016-06-27 11:56:46]

Description was changed from

==========
fix

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

to

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

Cleaning up video-seek* 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-27 11:57:46]

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

[Srirama, 2016-06-27 11:57:47]

PTAL

[fs, 2016-06-27 12:09:54]

lgtm

https://codereview.chromium.org/2100703003/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/media/video-seek-past-end-playing.html
(right):

https://codereview.chromium.org/2100703003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/video-seek-past-end-playing.html:13:
video.oncanplaythrough = null;
Nit: indentation

https://codereview.chromium.org/2100703003/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/media/video-seek-to-duration-with-playbackrate-zero.html
(right):

https://codereview.chromium.org/2100703003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/video-seek-to-duration-with-playbackrate-zero.html:21:
// Seeking to the middle of the video");
Nit: stray "); at the end (replace with .)

https://codereview.chromium.org/2100703003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/video-seek-to-duration-with-playbackrate-zero.html:24:
video.onseeked = null;
Nit: this will be set again below

https://codereview.chromium.org/2100703003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/video-seek-to-duration-with-playbackrate-zero.html:29:
video.onseeked = null;
Nit: ditto

[Srirama, 2016-06-27 13:10:38]

https://codereview.chromium.org/2100703003/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/media/video-seek-past-end-playing.html
(right):

https://codereview.chromium.org/2100703003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/video-seek-past-end-playing.html:13:
video.oncanplaythrough = null;
On 2016/06/27 12:09:54, fs wrote:
> Nit: indentation

Done.

https://codereview.chromium.org/2100703003/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/media/video-seek-to-duration-with-playbackrate-zero.html
(right):

https://codereview.chromium.org/2100703003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/video-seek-to-duration-with-playbackrate-zero.html:21:
// Seeking to the middle of the video");
On 2016/06/27 12:09:54, fs wrote:
> Nit: stray "); at the end (replace with .)

Done.

https://codereview.chromium.org/2100703003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/video-seek-to-duration-with-playbackrate-zero.html:24:
video.onseeked = null;
On 2016/06/27 12:09:54, fs wrote:
> Nit: this will be set again below

Done.

https://codereview.chromium.org/2100703003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/video-seek-to-duration-with-playbackrate-zero.html:29:
video.onseeked = null;
On 2016/06/27 12:09:54, fs wrote:
> Nit: ditto

Done.

[Srirama, 2016-06-27 13:10:47]

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

[Srirama, 2016-06-27 13:10:47]

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

[commit-bot: I haz the power, 2016-06-27 13:10:55]

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-27 14:21:12]

Description was changed from

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

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

Cleaning up video-seek* 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-27 14:21:14]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-27 14:22:28]

Description was changed from

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

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

Cleaning up video-seek* 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/278bd74563e00bf0abbae6594c34b1680a7ed428
Cr-Commit-Position: refs/heads/master@{#402169}
==========

[commit-bot: I haz the power, 2016-06-27 14:22:30]

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