New issue
Advanced search Search tips

Issue 840749 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 840659
Owner: ----
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: ----
Type: ----



Sign in to add a comment

[WPT] New failures introduced in external/wpt/webrtc by import https://crrev.com/c/1049368

Project Member Reported by monorail...@ecosystem-infra.iam.gserviceaccount.com, May 8 2018

Issue description

WPT import https://crrev.com/c/1049368 introduced new failures in external/wpt/webrtc:

List of new failures:
[ Mac-mac10.11 ] virtual/webrtc-wpt-unified-plan/external/wpt/webrtc/protocol/video-codecs.html new failing tests: https://crrev.com/c/1049368/4/third_party/WebKit/LayoutTests/platform/mac-mac10.11/virtual/webrtc-wpt-unified-plan/external/wpt/webrtc/protocol/video-codecs-expected.txt
[ Win ] virtual/webrtc-wpt-unified-plan/external/wpt/webrtc/protocol/video-codecs.html new failing tests: https://crrev.com/c/1049368/4/third_party/WebKit/LayoutTests/platform/win/virtual/webrtc-wpt-unified-plan/external/wpt/webrtc/protocol/video-codecs-expected.txt

This import contains upstream changes from b9f5402f16fa98c88428dace0a7088ef9dae8df3 to 42ae4a432d5264eba6c7171c8ecfcb26dc9fa1a8:
Worker: Throw an exception when importScripts() is called on "module" worker: https://github.com/w3c/web-platform-tests/commit/42ae4a432d5264eba6c7171c8ecfcb26dc9fa1a8
DOM: non-Text nodes are not touched by normalization: https://github.com/w3c/web-platform-tests/commit/d50825bfd2a2e887c24e63683a03efe5896e2b80
webrtc wpt: pass test in promise_test and async_test: https://github.com/w3c/web-platform-tests/commit/9949cb53d1e32bc9c34c687d8b89dc3ecb6d9553 [affecting this directory]
HTML: no longer depend on SVGElement in IDL: https://github.com/w3c/web-platform-tests/commit/fcf42b610c3eec811dc26a09b1404300aeb01ac1

 

Comment 1 by hta@chromium.org, May 8 2018

Mergedinto: 840659
Status: Duplicate (was: Untriaged)
This test was upstreamed, then reverted, then upstreamed again after disabling relevant tests. I guess it got downstreamed while reverted.

A lot of platforms pass, so it's better done in TestExpectations.

Sign in to add a comment