Convert controls-* tests to testharness.js

Convert controls-* tests to testharness.js

Cleaning up controls-* 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/d06818a0702f013dd29b468a669c812755435273
Cr-Commit-Position: refs/heads/master@{#396703}

[Srirama, 2016-05-29 04:47:53]

Description was changed from

==========
f


fix

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

to

==========
Convert controls-* tests to testharness.js

Cleaning up controls-* 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-05-29 04:49:44]

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

[Srirama, 2016-05-29 04:49:44]

PTAL

[fs, 2016-05-29 14:52:43]

lgtm

https://codereview.chromium.org/2021693002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/media/controls-timeline.html (right):

https://codereview.chromium.org/2021693002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/controls-timeline.html:23: var time =
video.currentTime / video.duration;
Nit: Maybe 'normalizedTime'?

https://codereview.chromium.org/2021693002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/media/controls-volume-slider-keynav.html
(right):

https://codereview.chromium.org/2021693002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/controls-volume-slider-keynav.html:17:
button.focus();
Nit: These two lines could be merged.

[Srirama, 2016-05-30 07:55:14]

https://codereview.chromium.org/2021693002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/media/controls-timeline.html (right):

https://codereview.chromium.org/2021693002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/controls-timeline.html:23: var time =
video.currentTime / video.duration;
On 2016/05/29 14:52:43, fs wrote:
> Nit: Maybe 'normalizedTime'?

Done.

https://codereview.chromium.org/2021693002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/media/controls-volume-slider-keynav.html
(right):

https://codereview.chromium.org/2021693002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/controls-volume-slider-keynav.html:17:
button.focus();
On 2016/05/29 14:52:43, fs wrote:
> Nit: These two lines could be merged.

Done.

[Srirama, 2016-05-30 07:55:22]

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

[Srirama, 2016-05-30 07:55:22]

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

[commit-bot: I haz the power, 2016-05-30 07:55:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2021693002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2021693002/20001

[commit-bot: I haz the power, 2016-05-30 09:39:32]

Description was changed from

==========
Convert controls-* tests to testharness.js

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

Cleaning up controls-* 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-05-30 09:39:34]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-30 09:41:16]

Description was changed from

==========
Convert controls-* tests to testharness.js

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

Cleaning up controls-* 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/d06818a0702f013dd29b468a669c812755435273
Cr-Commit-Position: refs/heads/master@{#396703}
==========

[commit-bot: I haz the power, 2016-05-30 09:41:17]

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