New issue
Advanced search Search tips

Issue 868225 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 7
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----

Blocking:
issue 678905



Sign in to add a comment

external/wpt/service-workers/service-worker/clients-matchall-order.https.html is timing out on mac

Project Member Reported by monorail...@ecosystem-infra.iam.gserviceaccount.com, Jul 27

Issue description

WPT import https://crrev.com/c/1152529 introduced new failures in external/wpt/service-workers:

List of new failures:
[ Mac10.12 Mac10.13 ] external/wpt/service-workers/service-worker/clients-matchall-order.https.html [ Timeout ]

This import contains upstream changes from efef417d9f9ceb9016f682c3aa523d31405e1238 to 849352db27a750476f5cbd42a279025884948400:
Don't reset pending tasks when setting the target effect to null (#12194): https://github.com/web-platform-tests/wpt/commit/849352db27a750476f5cbd42a279025884948400
Add the webgl1 IDL file + test (#12196): https://github.com/web-platform-tests/wpt/commit/8263509514ae6aec4e63d2bd2c787a862d8c361a
Use navigation start time in the case where the refresh driver's time stamp is behind from the start time to avoid negative timelime time at the very beginning of document load.: https://github.com/web-platform-tests/wpt/commit/ee1ce48e3834f99db45cef18c41b6e5b189486fa
Fix manifest paths obtained from the file system: https://github.com/web-platform-tests/wpt/commit/bdae4c34e7490ddc5f16c5718d445cd0af159941
Added a way to specify aliases using command line.: https://github.com/web-platform-tests/wpt/commit/3f67724a2abefcbdf43e8797579a2e2c2c6a40f8
<input>: INPUT element with unknown type attribute value should be rendered same as <input type=text>.: https://github.com/web-platform-tests/wpt/commit/65dd8395f9303543a90abbb58ef31be6fba3a33e
Created new interface and class TrustedTypePolicy: https://github.com/web-platform-tests/wpt/commit/2a5fed3d1f1691b8b02b6ee645c1ea00143d6c57
url: DecodeURLEscapeSequences() should not apply UTF-8 and isomorphic encoding to a single input.: https://github.com/web-platform-tests/wpt/commit/c54b4511852cd7acdb0aed4108f38487ab95418f
[idlharness.js] Add inheritor as dep for includes (#12165): https://github.com/web-platform-tests/wpt/commit/2fe1bdb04188923e8933161a46e71718f33f8fdb

 
Labels: M-70 Pri-2
Status: Available (was: Untriaged)
Summary: external/wpt/service-workers/service-worker/clients-matchall-order.https.html is timing out on mac (was: [WPT] New failures introduced in external/wpt/service-workers by import https://crrev.com/c/1152529)
Not sure the root cause but the failure seems consistent on mac.
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_layout_tests&tests=xternal/wpt/service-workers/service-worker/clients-matchall-order.https.html


Blocking: 678905
I'm not seeing timeouts, but the test is slow. We should add a long timeout.
Owner: falken@chromium.org
Status: Started (was: Available)
https://github.com/web-platform-tests/wpt/pull/13684
Project Member

Comment 5 by bugdroid1@chromium.org, Nov 7

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

commit 7d77502b28d7d93a7913a8517e48af9f52c79e4a
Author: Matt Falkenhagen <falken@chromium.org>
Date: Wed Nov 07 07:11:47 2018

Gardening: service-worker clients-matchall-order does not timeout.

Since adding the long timeout in
https://github.com/web-platform-tests/wpt/pull/13684, it doesn't
timeout. However, it still has recently started timing out on
Mac for oob-cors.

Bug:  868225 , 870173
Change-Id: I3b460ae75d0e25916ed2d836e0e5d75d29f2124b
Reviewed-on: https://chromium-review.googlesource.com/c/1322335
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#605982}
[modify] https://crrev.com/7d77502b28d7d93a7913a8517e48af9f52c79e4a/third_party/WebKit/LayoutTests/TestExpectations

Labels: -M-70 M-72
Status: Fixed (was: Started)

Sign in to add a comment