New issue
Advanced search Search tips

Issue 824955 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 22
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

external/wpt/css/css-font-loading/idlharness.https.html is flaky

Project Member Reported by wolenetz@chromium.org, Mar 22 2018

Issue description

Note: the example build log doesn't show any unexpected flakiness in the summary, yet the dashboard definitely shows flakiness including that example build. I couldn't find any existing expectation in the code for flakiness.
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 22 2018

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

commit e8a86188ea668a5f88460c0a83099fd21820b5c9
Author: Matt Wolenetz <wolenetz@chromium.org>
Date: Thu Mar 22 20:51:33 2018

external/wpt/css/css-font-loading/idlharness.https.html is flaky

Mark flaky.

BUG= 824955 
TBR=kojii@chromium.org
NO-TRY=True

Change-Id: I96de18b7df56ec38d077c85e2a0ac4cb4e38ae1f
Reviewed-on: https://chromium-review.googlesource.com/976556
Reviewed-by: Matthew Wolenetz <wolenetz@chromium.org>
Commit-Queue: Matthew Wolenetz <wolenetz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545240}
[modify] https://crrev.com/e8a86188ea668a5f88460c0a83099fd21820b5c9/third_party/WebKit/LayoutTests/TestExpectations

Comment 3 by e...@chromium.org, Mar 23 2018

Components: -Blink>Fonts Blink>WebFonts

Comment 4 by kojii@chromium.org, Mar 26 2018

Cc: ksakamoto@chromium.org
Project Member

Comment 5 by bugdroid1@chromium.org, Nov 22

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

commit 24dd4ecd526a8604c5ee07abbcf820802ab58423
Author: Kunihiko Sakamoto <ksakamoto@chromium.org>
Date: Thu Nov 22 06:17:09 2018

Update external/wpt/css/css-font-loading/idlharness.https-expected.txt

This updates the expectation for css/css-font-loading/idlharness.https.html
and removes the Failure expectation from TestExpectation file.

The expected.txt file contains some FAIL lines because
- variationSettings attribute does not exist in FontFace (crbug.com/907723)
- FontFaceSet constructor is not exposed (crbug.com/651795)

Bug:  824955 ,907723,651795
Change-Id: I527badfeea8d21ac16a9bfd20d1f0f255ff553ce
Reviewed-on: https://chromium-review.googlesource.com/c/1347951
Commit-Queue: Kunihiko Sakamoto <ksakamoto@chromium.org>
Reviewed-by: Takashi Toyoshima <toyoshim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#610318}
[modify] https://crrev.com/24dd4ecd526a8604c5ee07abbcf820802ab58423/third_party/WebKit/LayoutTests/TestExpectations
[modify] https://crrev.com/24dd4ecd526a8604c5ee07abbcf820802ab58423/third_party/WebKit/LayoutTests/external/wpt/css/css-font-loading/idlharness.https-expected.txt

Cc: -ksakamoto@chromium.org
Owner: ksakamoto@chromium.org
Status: Fixed (was: Untriaged)

Sign in to add a comment