Add TODOs to convert from video-test.js to testharness.js

Add TODOs to convert from video-test.js to testharness.js

BUG=588956
R=mlamouri@chromium.org

Committed: https://crrev.com/41c5954800077d4a5213b8b400f85230f7dc9d0d
Cr-Commit-Position: refs/heads/master@{#376938}

[philipj_slow, 2016-02-22 04:52:06]

The CQ bit was checked by philipj@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-02-22 04:52:20]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1715303002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1715303002/1

[commit-bot: I haz the power, 2016-02-22 06:05:49]

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

[commit-bot: I haz the power, 2016-02-22 06:05:50]

Dry run: This issue passed the CQ dry run.

[philipj_slow, 2016-02-22 06:28:07]

philipj@opera.com changed reviewers:
+ mlamouri@chromium.org

[philipj_slow, 2016-02-22 06:28:09]

PTAL. If you approve I'll create a tracking bug. If you like I could reference
that in all the comments as well.

Note that this also removes trailing whitespace as a drive-by fix, because I've
told my editor to.

[mlamouri (slow - plz ping), 2016-02-22 15:31:48]

lgtm but it would be great to have a tracking bug and mention the bug number in
the TODO.

BTW, I started doing that for media-play-promise in
https://codereview.chromium.org/1710933003

[philipj_slow, 2016-02-23 06:16:30]

Description was changed from

==========
Add TODOs to convert from video-test.js to testharness.js

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

to

==========
Add TODOs to convert from video-test.js to testharness.js

BUG=588956
R=mlamouri@chromium.org
==========

[philipj_slow, 2016-02-23 06:17:42]

add bug ref

[philipj_slow, 2016-02-23 06:18:17]

On 2016/02/22 15:31:48, Mounir Lamouri (slow) wrote:
> lgtm but it would be great to have a tracking bug and mention the bug number
in
> the TODO.

Added that.

> BTW, I started doing that for media-play-promise in
> https://codereview.chromium.org/1710933003

\o/

[philipj_slow, 2016-02-23 06:18:30]

The CQ bit was checked by philipj@opera.com

[philipj_slow, 2016-02-23 06:18:31]

The patchset sent to the CQ was uploaded after l-g-t-m from
mlamouri@chromium.org
Link to the patchset: https://codereview.chromium.org/1715303002/#ps20001
(title: "add bug ref")

[commit-bot: I haz the power, 2016-02-23 06:19:00]

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

[commit-bot: I haz the power, 2016-02-23 07:22:19]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-23 07:23:33]

Description was changed from

==========
Add TODOs to convert from video-test.js to testharness.js

BUG=588956
R=mlamouri@chromium.org
==========

to

==========
Add TODOs to convert from video-test.js to testharness.js

BUG=588956
R=mlamouri@chromium.org

Committed: https://crrev.com/41c5954800077d4a5213b8b400f85230f7dc9d0d
Cr-Commit-Position: refs/heads/master@{#376938}
==========

[commit-bot: I haz the power, 2016-02-23 07:23:36]

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