New issue
Advanced search Search tips

Issue 888470 link

Starred by 1 user

Issue metadata

Status: Started
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

New failing tests from Mozilla in external/wpt/referrer-policy/css-integration/child-css/

Project Member Reported by foolip@chromium.org, Sep 24

Issue description

Two tests in https://github.com/web-platform-tests/wpt/pull/13029 are failing, and because a generated unique ID is part of the failure message, it's blocking wpt imports.

The two tests are:
external/wpt/referrer-policy/css-integration/child-css/internal-import-stylesheet.html
external/wpt/referrer-policy/css-integration/child-css/processing-instruction.html

I will manually mark these as expected to fail to allow the wpt import to proceed. When the tests have been imported it will be possible to investigate why they are failing.
 
https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/25171 is the currently ongoing import job where I found this problem.
Cc: kyleju@chromium.org robertma@chromium.org
robertma@ kyleju@ FYI that this is another failure mode of the importer. We probably already have an issue for a previous instance of this.
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 24

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

commit 71beac3ed5a3a8a9cf8869d9027cf65f638a0290
Author: Philip Jägenstedt <foolip@chromium.org>
Date: Mon Sep 24 11:55:09 2018

Mark two incoming WPT referrer-policy tests as expected fail

A UUID is part of the failure message, so baselines cannot be added.

TBR=jochen@chromium.org,estark@chromium.org

Bug: 888470
Change-Id: I763f290fc4e13c485ed773d39985d2f0c454c896
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/1240121
Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
Commit-Queue: Philip Jägenstedt <foolip@chromium.org>
Cr-Commit-Position: refs/heads/master@{#593508}
[modify] https://crrev.com/71beac3ed5a3a8a9cf8869d9027cf65f638a0290/third_party/WebKit/LayoutTests/TestExpectations

Owner: foolip@chromium.org
Status: Started (was: Untriaged)
#3 was not enough, the same tests under two virtual/ tests suites need to be skipped too.
Project Member

Comment 5 by bugdroid1@chromium.org, Sep 24

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

commit ca68ede1d057b7d5d925345d8b9efdc361bd7ab3
Author: Philip Jägenstedt <foolip@chromium.org>
Date: Mon Sep 24 19:51:09 2018

Mark two referrer-policy tests under virtual/ as failing too

An oversight in https://chromium-review.googlesource.com/1240121

TBR=jochen@chromium.org,estark@chromium.org

Bug: 888470
Change-Id: I006c76c4084bc78d3f3b3c248da28c6e13b9cef3
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/1241173
Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
Commit-Queue: Philip Jägenstedt <foolip@chromium.org>
Cr-Commit-Position: refs/heads/master@{#593647}
[modify] https://crrev.com/ca68ede1d057b7d5d925345d8b9efdc361bd7ab3/third_party/WebKit/LayoutTests/TestExpectations

Sign in to add a comment