New issue
Advanced search Search tips

Issue 882292 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 10
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Layout Test http/tests/security/link-crossorigin-stylesheet-no-cors.html is flaky

Project Member Reported by yhirano@chromium.org, Sep 10

Issue description

The following layout test is flaky on [insert platform]

http/tests/security/link-crossorigin-stylesheet-no-cors.html


 
Project Member

Comment 2 by bugdroid1@chromium.org, Sep 10

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

commit 356db9ab7b6ba80603a03683b2ae862b5dca4cf1
Author: Yutaka Hirano <yhirano@chromium.org>
Date: Mon Sep 10 02:27:33 2018

Mark http/tests/security/link-crossorigin-stylesheet-no-cors.html as flaky

Bug:  882292 
Change-Id: I1baaabbe07a8eeaa5dd42fb6d657c1f77861bd56
Tbr: toyoshim@chromium.org
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/1214970
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Commit-Queue: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#589821}
[modify] https://crrev.com/356db9ab7b6ba80603a03683b2ae862b5dca4cf1/third_party/WebKit/LayoutTests/TestExpectations

Project Member

Comment 3 by bugdroid1@chromium.org, Sep 10

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

commit 79dcde71dc6707ae1c8b69d7b013ef310aaa762a
Author: Yutaka Hirano <yhirano@chromium.org>
Date: Mon Sep 10 06:12:33 2018

Unflake http/tests/security/link-crossorigin-stylesheet-no-cors.html

The test is flaky because speculative preloads start, and in some cases
finish loading before disabling console message. Let's use testharness.js
in order to ignore console messages.

Bug:  882292 
Change-Id: Ic6627f0bb941e7299ac611c2fd10e6469e1ed5e9
Reviewed-on: https://chromium-review.googlesource.com/1214978
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Commit-Queue: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#589832}
[modify] https://crrev.com/79dcde71dc6707ae1c8b69d7b013ef310aaa762a/third_party/WebKit/LayoutTests/TestExpectations
[delete] https://crrev.com/0e8dc0e5f7fbd16ed18ae659ba8c137bbd3823fb/third_party/WebKit/LayoutTests/http/tests/security/link-crossorigin-stylesheet-no-cors-expected.txt
[modify] https://crrev.com/79dcde71dc6707ae1c8b69d7b013ef310aaa762a/third_party/WebKit/LayoutTests/http/tests/security/link-crossorigin-stylesheet-no-cors.html

Status: Fixed (was: Assigned)

Sign in to add a comment