Unskip mediastream-capture WPT tests

Unskip mediastream-capture WPT tests

Many of these are passing now.  For those that aren't, add -expected.txt
files so people notice when causing any new sub-test failures.

Renamed MediaStreamTrack-end.https to be a manual test because it relies on user interaction to revoke the permission.

Fixed timeout in MediaStream-MediaElement-preload-none.https.html by adding an error handler.

Left MediaStreamTrack-MediaElement-disabled-audio-is-silence.https.html skipped due to time-out I couldn't figure out by the end of our WPT hackathon (tracked by http://crbug.com/688613).

BUG=412381, 688613
Review-Url: https://codereview.chromium.org/2677763002
Cr-Commit-Position: refs/heads/master@{#452840}
Committed: https://chromium.googlesource.com/chromium/src/+/747265bacb740e3f91170b6e271efcc72737c5b8

[Rick Byers, 2017-02-04 01:43:27]

The CQ bit was checked by rbyers@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-04 01:43:47]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Rick Byers, 2017-02-04 01:48:02]

Description was changed from

==========
Unskip mediastream-capture WPT tests

Many of these are passing now.  For those that aren't, add -expected.txt
files so people notice when causing any new sub-test failures.
Fix a couple test timeouts, and leave one I couldn't figure out unfixed
(tracked by http://crbug.com/688613).

BUG=412381,688613
==========

to

==========
Unskip mediastream-capture WPT tests

Many of these are passing now.  For those that aren't, add -expected.txt
files so people notice when causing any new sub-test failures.

Renamed MediaStreamTrack-end.https to be a manual test because it relies on user
interaction to revoke the permission.

Fixed timeout in MediaStream-MediaElement-preload-none.https.html by adding an
error handler.

Left MediaStreamTrack-MediaElement-disabled-audio-is-silence.https.html skipped
due to time-out I couldn't figure out by the end of our WPT hackathon (tracked
by http://crbug.com/688613).

BUG=412381,688613
==========

[Rick Byers, 2017-02-04 01:48:02]

rbyers@chromium.org changed reviewers:
+ hta@chromium.org

[Rick Byers, 2017-02-04 01:49:55]

Hey Harald,
We had a web-platform-test hackathon today and I worked to remove some 'Skipped'
tests from TestExpecations.  PTAL.

[commit-bot: I haz the power, 2017-02-04 05:34:24]

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

[commit-bot: I haz the power, 2017-02-04 05:34:25]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_TIMED_OUT, no
build URL)

[hta - Chromium, 2017-02-06 01:41:26]

lgtm

thanks for doing this!

Where are we supposed to record the crbug numbers for those tests that are left
failing via an "expected" file?

[foolip, 2017-02-06 02:20:51]

On 2017/02/06 01:41:26, hta - Chromium wrote:
> lgtm
> 
> thanks for doing this!
> 
> Where are we supposed to record the crbug numbers for those tests that are
left
> failing via an "expected" file?

I'm afraid we don't have a process for this, inside external/wpt or elsewhere. I
think triaging all failures in web-platform-tests is important, but am not sure
how best to do it. Because one would have to single out failing subtests,
there's some connection to
https://bugs.chromium.org/p/chromium/issues/detail?id=679742 so I'll comment on
that bug.

[hta - Chromium, 2017-02-07 10:28:01]

On 2017/02/06 02:20:51, foolip wrote:
> On 2017/02/06 01:41:26, hta - Chromium wrote:
> > lgtm
> > 
> > thanks for doing this!
> > 
> > Where are we supposed to record the crbug numbers for those tests that are
> left
> > failing via an "expected" file?
> 
> I'm afraid we don't have a process for this, inside external/wpt or elsewhere.
I
> think triaging all failures in web-platform-tests is important, but am not
sure
> how best to do it. Because one would have to single out failing subtests,
> there's some connection to
> https://bugs.chromium.org/p/chromium/issues/detail?id=679742 so I'll comment
on
> that bug.

Thanks, commented on that bug. Feel free to submit this CL!

[Rick Byers, 2017-02-08 14:37:22]

The CQ bit was checked by rbyers@chromium.org

[Rick Byers, 2017-02-08 14:37:23]

The patchset sent to the CQ was uploaded after l-g-t-m from hta@chromium.org
Link to the patchset: https://codereview.chromium.org/2677763002/#ps20001
(title: "Rebase and update manifest")

[commit-bot: I haz the power, 2017-02-08 14:37:37]

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, 2017-02-08 15:38:18]

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

[commit-bot: I haz the power, 2017-02-08 15:38:19]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Rick Byers, 2017-02-08 22:23:05]

The CQ bit was checked by rbyers@chromium.org

[Rick Byers, 2017-02-08 22:23:06]

The patchset sent to the CQ was uploaded after l-g-t-m from hta@chromium.org
Link to the patchset: https://codereview.chromium.org/2677763002/#ps40001
(title: "Use correct manual test naming pattern")

[commit-bot: I haz the power, 2017-02-08 22:23:37]

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, 2017-02-08 22:27:00]

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

[commit-bot: I haz the power, 2017-02-08 22:27:01]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[Rick Byers, 2017-02-24 15:04:04]

The CQ bit was checked by rbyers@chromium.org

[Rick Byers, 2017-02-24 15:04:05]

The patchset sent to the CQ was uploaded after l-g-t-m from hta@chromium.org
Link to the patchset: https://codereview.chromium.org/2677763002/#ps50001
(title: "rebase, fix test name")

[commit-bot: I haz the power, 2017-02-24 15:04:18]

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, 2017-02-24 16:21:20]

CQ is committing da patch.

Bot data: {"patchset_id": 50001, "attempt_start_ts": 1487948644620600,
"parent_rev": "2c4fd527db90b99fabaf9008216c95a6db521c58", "commit_rev":
"747265bacb740e3f91170b6e271efcc72737c5b8"}

[commit-bot: I haz the power, 2017-02-24 16:22:03]

Description was changed from

==========
Unskip mediastream-capture WPT tests

Many of these are passing now.  For those that aren't, add -expected.txt
files so people notice when causing any new sub-test failures.

Renamed MediaStreamTrack-end.https to be a manual test because it relies on user
interaction to revoke the permission.

Fixed timeout in MediaStream-MediaElement-preload-none.https.html by adding an
error handler.

Left MediaStreamTrack-MediaElement-disabled-audio-is-silence.https.html skipped
due to time-out I couldn't figure out by the end of our WPT hackathon (tracked
by http://crbug.com/688613).

BUG=412381,688613
==========

to

==========
Unskip mediastream-capture WPT tests

Many of these are passing now.  For those that aren't, add -expected.txt
files so people notice when causing any new sub-test failures.

Renamed MediaStreamTrack-end.https to be a manual test because it relies on user
interaction to revoke the permission.

Fixed timeout in MediaStream-MediaElement-preload-none.https.html by adding an
error handler.

Left MediaStreamTrack-MediaElement-disabled-audio-is-silence.https.html skipped
due to time-out I couldn't figure out by the end of our WPT hackathon (tracked
by http://crbug.com/688613).

BUG=412381,688613

Review-Url: https://codereview.chromium.org/2677763002
Cr-Commit-Position: refs/heads/master@{#452840}
Committed:
https://chromium.googlesource.com/chromium/src/+/747265bacb740e3f91170b6e271e...
==========

[commit-bot: I haz the power, 2017-02-24 16:22:04]

Committed patchset #4 (id:50001) as
https://chromium.googlesource.com/chromium/src/+/747265bacb740e3f91170b6e271e...