Convert track-webvtt-tc[006,007,011] tests to testharness.js

Convert track-webvtt-tc[006,007,011] tests to testharness.js

Cleaning up track-webvtt-tc[006,007,011] tests in
media/track to use testharness.js instead of video-test.js.
This will enable to upstream these tests to web-platform-tests.

Updated track-webvtt-tc006-cue-identifiers.html and
track-webvtt-tc007-cue-no-id.html to reflect that "-->"
is now allowed in cue identifiers.
Updated track-webvtt-tc011-blank-lines.html to reflect that a
timing line always starts a new cue in absence of a separating line.

BUG=588956
Committed: https://crrev.com/b4e200f61f06f94474979d4d2733c30a02785360
Cr-Commit-Position: refs/heads/master@{#397954}

[Srirama, 2016-06-05 09:09:01]

Patchset #1 (id:1) has been deleted

[Srirama, 2016-06-05 09:10:37]

Description was changed from

==========
fix

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

to

==========
Convert track-webvtt-tc[006,007,011] tests to testharness.js

Cleaning up track-webvtt-tc[006,007,011] tests in
media/track 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-05 09:11:10]

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

[Srirama, 2016-06-05 09:11:10]

PTAL

[fs, 2016-06-05 13:04:58]

LGTM w/ nit, and requests to mention some things in the description.

https://codereview.chromium.org/2042613002/diff/2/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc006-cue-identifiers-expected.txt
(left):

https://codereview.chromium.org/2042613002/diff/2/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc006-cue-identifiers-expected.txt:24:
EXPECTED (cues.length == '0'), OBSERVED '3' FAIL
Please mention this in the description: "track-webvtt-tc006-cue-identifiers.html
updated to reflect that --> is now allowed in cue identifiers" or something like
that.

https://codereview.chromium.org/2042613002/diff/2/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc007-cue-no-id-expected.txt
(left):

https://codereview.chromium.org/2042613002/diff/2/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc007-cue-no-id-expected.txt:20:
EXPECTED (cues.length == '0'), OBSERVED '3' FAIL
Ditto.

https://codereview.chromium.org/2042613002/diff/2/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc011-blank-lines-expected.txt
(left):

https://codereview.chromium.org/2042613002/diff/2/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc011-blank-lines-expected.txt:20:
EXPECTED (cues.length == '1'), OBSERVED '3' FAIL
And one more. (But this one is about timing lines always starting a new cue.)

https://codereview.chromium.org/2042613002/diff/2/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc011-blank-lines.html
(right):

https://codereview.chromium.org/2042613002/diff/2/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc011-blank-lines.html:2:
<title>Tests that cues are affected neither by multiple newlines \n, \r, and
\r\n nor by the absense of seperation line.</title>
Nit: "separation" (or, I believe better: "a separating")

[Srirama, 2016-06-05 21:05:30]

Description was changed from

==========
Convert track-webvtt-tc[006,007,011] tests to testharness.js

Cleaning up track-webvtt-tc[006,007,011] tests in
media/track to use testharness.js instead of video-test.js.
This will enable to upstream these tests to web-platform-tests. 

BUG=588956
==========

to

==========
Convert track-webvtt-tc[006,007,011] tests to testharness.js

Cleaning up track-webvtt-tc[006,007,011] tests in
media/track to use testharness.js instead of video-test.js.
This will enable to upstream these tests to web-platform-tests. 

Updated track-webvtt-tc006-cue-identifiers.html and
track-webvtt-tc007-cue-no-id.html to reflect that "-->"
is now allowed in cue identifiers.
Updated track-webvtt-tc011-blank-lines.html to reflect that a
timing line always starts a new cue in absense of a separating line.

BUG=588956
==========

[Srirama, 2016-06-05 21:10:29]

Description was changed from

==========
Convert track-webvtt-tc[006,007,011] tests to testharness.js

Cleaning up track-webvtt-tc[006,007,011] tests in
media/track to use testharness.js instead of video-test.js.
This will enable to upstream these tests to web-platform-tests. 

Updated track-webvtt-tc006-cue-identifiers.html and
track-webvtt-tc007-cue-no-id.html to reflect that "-->"
is now allowed in cue identifiers.
Updated track-webvtt-tc011-blank-lines.html to reflect that a
timing line always starts a new cue in absense of a separating line.

BUG=588956
==========

to

==========
Convert track-webvtt-tc[006,007,011] tests to testharness.js

Cleaning up track-webvtt-tc[006,007,011] tests in
media/track to use testharness.js instead of video-test.js.
This will enable to upstream these tests to web-platform-tests. 

Updated track-webvtt-tc006-cue-identifiers.html and
track-webvtt-tc007-cue-no-id.html to reflect that "-->"
is now allowed in cue identifiers.
Updated track-webvtt-tc011-blank-lines.html to reflect that a
timing line always starts a new cue in absence of a separating line.

BUG=588956
==========

[Srirama, 2016-06-05 21:12:41]

https://codereview.chromium.org/2042613002/diff/2/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc006-cue-identifiers-expected.txt
(left):

https://codereview.chromium.org/2042613002/diff/2/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc006-cue-identifiers-expected.txt:24:
EXPECTED (cues.length == '0'), OBSERVED '3' FAIL
On 2016/06/05 13:04:58, fs wrote:
> Please mention this in the description:
"track-webvtt-tc006-cue-identifiers.html
> updated to reflect that --> is now allowed in cue identifiers" or something
like
> that.

Done.

https://codereview.chromium.org/2042613002/diff/2/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc007-cue-no-id-expected.txt
(left):

https://codereview.chromium.org/2042613002/diff/2/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc007-cue-no-id-expected.txt:20:
EXPECTED (cues.length == '0'), OBSERVED '3' FAIL
On 2016/06/05 13:04:58, fs wrote:
> Ditto.

Done.

https://codereview.chromium.org/2042613002/diff/2/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc011-blank-lines-expected.txt
(left):

https://codereview.chromium.org/2042613002/diff/2/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc011-blank-lines-expected.txt:20:
EXPECTED (cues.length == '1'), OBSERVED '3' FAIL
On 2016/06/05 13:04:58, fs wrote:
> And one more. (But this one is about timing lines always starting a new cue.)

Done.

https://codereview.chromium.org/2042613002/diff/2/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc011-blank-lines.html
(right):

https://codereview.chromium.org/2042613002/diff/2/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/media/track/track-webvtt-tc011-blank-lines.html:2:
<title>Tests that cues are affected neither by multiple newlines \n, \r, and
\r\n nor by the absense of seperation line.</title>
On 2016/06/05 13:04:58, fs wrote:
> Nit: "separation" (or, I believe better: "a separating")

Done.

[Srirama, 2016-06-05 21:12:54]

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

[Srirama, 2016-06-05 21:12:55]

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/2042613002/#ps30001
(title: "address nit and add more info in the description")

[commit-bot: I haz the power, 2016-06-05 21:13:01]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2042613002/30001

[commit-bot: I haz the power, 2016-06-05 23:48:02]

Description was changed from

==========
Convert track-webvtt-tc[006,007,011] tests to testharness.js

Cleaning up track-webvtt-tc[006,007,011] tests in
media/track to use testharness.js instead of video-test.js.
This will enable to upstream these tests to web-platform-tests. 

Updated track-webvtt-tc006-cue-identifiers.html and
track-webvtt-tc007-cue-no-id.html to reflect that "-->"
is now allowed in cue identifiers.
Updated track-webvtt-tc011-blank-lines.html to reflect that a
timing line always starts a new cue in absence of a separating line.

BUG=588956
==========

to

==========
Convert track-webvtt-tc[006,007,011] tests to testharness.js

Cleaning up track-webvtt-tc[006,007,011] tests in
media/track to use testharness.js instead of video-test.js.
This will enable to upstream these tests to web-platform-tests. 

Updated track-webvtt-tc006-cue-identifiers.html and
track-webvtt-tc007-cue-no-id.html to reflect that "-->"
is now allowed in cue identifiers.
Updated track-webvtt-tc011-blank-lines.html to reflect that a
timing line always starts a new cue in absence of a separating line.

BUG=588956
==========

[commit-bot: I haz the power, 2016-06-05 23:48:04]

Committed patchset #2 (id:30001)

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

Description was changed from

==========
Convert track-webvtt-tc[006,007,011] tests to testharness.js

Cleaning up track-webvtt-tc[006,007,011] tests in
media/track to use testharness.js instead of video-test.js.
This will enable to upstream these tests to web-platform-tests. 

Updated track-webvtt-tc006-cue-identifiers.html and
track-webvtt-tc007-cue-no-id.html to reflect that "-->"
is now allowed in cue identifiers.
Updated track-webvtt-tc011-blank-lines.html to reflect that a
timing line always starts a new cue in absence of a separating line.

BUG=588956
==========

to

==========
Convert track-webvtt-tc[006,007,011] tests to testharness.js

Cleaning up track-webvtt-tc[006,007,011] tests in
media/track to use testharness.js instead of video-test.js.
This will enable to upstream these tests to web-platform-tests.

Updated track-webvtt-tc006-cue-identifiers.html and
track-webvtt-tc007-cue-no-id.html to reflect that "-->"
is now allowed in cue identifiers.
Updated track-webvtt-tc011-blank-lines.html to reflect that a
timing line always starts a new cue in absence of a separating line.

BUG=588956

Committed: https://crrev.com/b4e200f61f06f94474979d4d2733c30a02785360
Cr-Commit-Position: refs/heads/master@{#397954}
==========

[commit-bot: I haz the power, 2016-06-05 23:50:03]

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