New issue
Advanced search Search tips

Issue 688613 link

Starred by 1 user
Status: Archived
Owner:
rbyers@chromium.org
Closed: Mar 2018
Cc:
guidou@chromium.org
foolip@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug

Blocking:
issue 412381



Sign in to add a comment

external/wpt/mediacapture-streams/MediaStreamTrack-MediaElement-disabled-audio-is-silence.https.html times out

Project Member Reported by rbyers@chromium.org, Feb 4 2017 
As part of trying to improve our web-platform-test coverage I took a look at the mediacapture-stream tests.  This was the only one I couldn't get passing or failing reliably - it seems to just timeout without any obvious errors being raised (checked for lots of different events, but I don't know much about MediaStream so maybe I just wasn't looking in the right place for completion/error events).

The test passes fine when I run it in Chrome canary: https://w3c-test.org/mediacapture-streams/MediaStreamTrack-MediaElement-disabled-audio-is-silence.https.html

So the timeout has something to do with the special configuration in content_shell.
Project Member

Comment 1 by bugdroid1@chromium.org, Feb 24 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/747265bacb740e3f91170b6e271efcc72737c5b8

commit 747265bacb740e3f91170b6e271efcc72737c5b8
Author: rbyers <rbyers@chromium.org>
Date: Fri Feb 24 16:21:15 2017

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}

[modify] https://crrev.com/747265bacb740e3f91170b6e271efcc72737c5b8/third_party/WebKit/LayoutTests/TestExpectations
[modify] https://crrev.com/747265bacb740e3f91170b6e271efcc72737c5b8/third_party/WebKit/LayoutTests/external/wpt/MANIFEST.json
[add] https://crrev.com/747265bacb740e3f91170b6e271efcc72737c5b8/third_party/WebKit/LayoutTests/external/wpt/mediacapture-streams/GUM-deny.https-expected.txt
[add] https://crrev.com/747265bacb740e3f91170b6e271efcc72737c5b8/third_party/WebKit/LayoutTests/external/wpt/mediacapture-streams/GUM-empty-option-param.https-expected.txt
[add] https://crrev.com/747265bacb740e3f91170b6e271efcc72737c5b8/third_party/WebKit/LayoutTests/external/wpt/mediacapture-streams/GUM-impossible-constraint.https-expected.txt
[add] https://crrev.com/747265bacb740e3f91170b6e271efcc72737c5b8/third_party/WebKit/LayoutTests/external/wpt/mediacapture-streams/GUM-unknownkey-option-param.https-expected.txt
[add] https://crrev.com/747265bacb740e3f91170b6e271efcc72737c5b8/third_party/WebKit/LayoutTests/external/wpt/mediacapture-streams/MediaStream-MediaElement-preload-none.https-expected.txt
[modify] https://crrev.com/747265bacb740e3f91170b6e271efcc72737c5b8/third_party/WebKit/LayoutTests/external/wpt/mediacapture-streams/MediaStream-MediaElement-preload-none.https.html
[add] https://crrev.com/747265bacb740e3f91170b6e271efcc72737c5b8/third_party/WebKit/LayoutTests/external/wpt/mediacapture-streams/MediaStream-MediaElement-srcObject.https-expected.txt
[add] https://crrev.com/747265bacb740e3f91170b6e271efcc72737c5b8/third_party/WebKit/LayoutTests/external/wpt/mediacapture-streams/MediaStreamTrack-MediaElement-disabled-audio-is-silence.https-expected.txt
[add] https://crrev.com/747265bacb740e3f91170b6e271efcc72737c5b8/third_party/WebKit/LayoutTests/external/wpt/mediacapture-streams/MediaStreamTrack-MediaElement-disabled-video-is-black.https-expected.txt
[rename] https://crrev.com/747265bacb740e3f91170b6e271efcc72737c5b8/third_party/WebKit/LayoutTests/external/wpt/mediacapture-streams/MediaStreamTrack-end-manual.https.html
[add] https://crrev.com/747265bacb740e3f91170b6e271efcc72737c5b8/third_party/WebKit/LayoutTests/external/wpt/mediacapture-streams/MediaStreamTrack-init.https-expected.txt

Comment 2 by anatolid@chromium.org, Mar 10 2017

Owner: rbyers@chromium.org
Status: Started (was: Unconfirmed)
rbyers@ Is this fixed now?

Comment 3 by guidou@chromium.org, Mar 13 2018

Status: Archived (was: Started)
Archiving this issue. This was probably fixed with the removal of mocking for Layout Tests and WPT. 
We have other bugs to track spec compliance and WPT status.
►

Sign in to add a comment