New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 697087 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Email to this user bounced
Closed: Apr 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

"external/wpt/service-workers/service-worker/registration-iframe.https.html" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Feb 28 2017

Issue description

"external/wpt/service-workers/service-worker/registration-iframe.https.html" is flaky.

This issue was created automatically by the chromium-try-flakes app. Please find the right owner to fix the respective test/step and assign this issue to them. If the step/test is infrastructure-related, please add Infra-Troopers label and change issue status to Untriaged. When done, please remove the issue from Sheriff Bug Queue by removing the Sheriff-Chromium label.

We have detected 5 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyVQsSBUZsYWtlIkpleHRlcm5hbC93cHQvc2VydmljZS13b3JrZXJzL3NlcnZpY2Utd29ya2VyL3JlZ2lzdHJhdGlvbi1pZnJhbWUuaHR0cHMuaHRtbAw.

Flaky tests should be disabled within 30 minutes unless culprit CL is found and reverted. Please see more details here: https://sites.google.com/a/chromium.org/dev/developers/tree-sheriffs/sheriffing-bug-queues#triaging-auto-filed-flakiness-bugs
 
Owner: jungkee....@samsung.com
Status: Assigned (was: Untriaged)
Jungkee, the failures started after your commit  https://codereview.chromium.org/2691903005.

Would you be able to look at this? I suspect somehow the scopes are colliding now so there is cross-talk between the tests.
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 1 2017

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

commit e26bb9456f6affe7c75cf3e19f0b36de4957cbf3
Author: yutak <yutak@chromium.org>
Date: Wed Mar 01 06:20:43 2017

registration-iframe.https.html is flaky.

BUG= 697087 
TBR=fsamuel@chromium.org,sergeyu@chromium.org,dvadym@chromium.org
NOTRY=true

Review-Url: https://codereview.chromium.org/2724743002
Cr-Commit-Position: refs/heads/master@{#453863}

[modify] https://crrev.com/e26bb9456f6affe7c75cf3e19f0b36de4957cbf3/third_party/WebKit/LayoutTests/TestExpectations

Comment 4 by yutak@chromium.org, Mar 1 2017

Labels: -Pri-1 -Sheriff-Chromium Pri-2
Removing sheriff tag and decrease priority since this gets an owner. Feel free to
change priority to whatever appropriate.
Thanks for having updated the TestExpectations. I'll find time to fix it.
Project Member

Comment 6 by bugdroid1@chromium.org, Mar 7 2017

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

commit 0d01f9fe710dfbe680f317621a617e383422e14d
Author: jungkee.song <jungkee.song@samsung.com>
Date: Tue Mar 07 09:28:44 2017

ServiceWorker: Fix registration-iframe.https.html

registration-iframe.https.html became flaky after https://codereview.chromium.org/2691903005.

This CL fixes the test by correcting the scope URL given to unregister
function in the first async_test and using different scope urls for
different tests to avoid conflicts in case there is cross-talk between
tests.

BUG= 697087 

Review-Url: https://codereview.chromium.org/2732513003
Cr-Commit-Position: refs/heads/master@{#455046}

[modify] https://crrev.com/0d01f9fe710dfbe680f317621a617e383422e14d/third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/registration-iframe.https.html

Project Member

Comment 7 by bugdroid1@chromium.org, Apr 7 2017

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

commit 0e617fa946919277991e615907e2617872e423c6
Author: qyearsley <qyearsley@chromium.org>
Date: Fri Apr 07 18:39:43 2017

Update test expectations for tests are recently not flaky.

This change was made automatically with update-test-expectations.

Flakiness dashboard:
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=webkit_layout_tests&tests=media/autoplay-muted.html,http/tests/inspector/service-workers/service-worker-manager.html,fast/text/international/inline-plaintext-relayout-with-leading-neutrals.html,http/tests/inspector/resource-tree/resource-tree-non-unique-url.html,http/tests/inspector-enabled/console-stack-overflow-source-url.html,virtual/mojo-loading/http/tests/inspector-enabled/console-stack-overflow-source-url.html,virtual/mojo-loading/http/tests/security/link-crossorigin-preload-no-cors.html,http/tests/inspector/tracing/timeline-script-parse.html,http/tests/media/media-source/mediasource-seek-beyond-duration.html,virtual/mojo-loading/http/tests/media/media-source/mediasource-seek-beyond-duration.html,external/wpt/fetch/api/redirect/redirect-location-worker.html,external/wpt/fetch/api/redirect/redirect-location.html,external/wpt/fetch/api/redirect/redirect-origin-worker.html,external/wpt/fetch/api/request/request-cache-default-conditional.html,external/wpt/service-workers/service-worker/registration-iframe.https.html

BUG= 691045 , 564403 ,651343, 581468 , 666628 ,664839, 678489 , 678490 ,705490, 697087 

Review-Url: https://codereview.chromium.org/2800123002
Cr-Commit-Position: refs/heads/master@{#462938}

[modify] https://crrev.com/0e617fa946919277991e615907e2617872e423c6/third_party/WebKit/LayoutTests/TestExpectations

Comment 8 by falken@chromium.org, Apr 13 2017

Status: Fixed (was: Assigned)
Fixed in comment 6

Sign in to add a comment