New issue
Advanced search Search tips

Issue 834311 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: ----
Type: ----



Sign in to add a comment

[WPT] New failures introduced in external/wpt/css/css-tables by import https://crrev.com/c/1016840

Project Member Reported by monorail...@ecosystem-infra.iam.gserviceaccount.com, Apr 18 2018

Issue description

WPT import https://crrev.com/c/1016840 introduced new failures in external/wpt/css/css-tables:

List of new failures:
external/wpt/css/css-tables/anonymous-table-ws-001.html [ Failure ]

This import contains upstream changes from b4e3ffbc3251172b79a9a82fa8e4ff290cc61449 to e2c839d70a30ce0a3816bc63601a99ccf65a1654:
Fetch: MediaSource blob: URLs: https://github.com/w3c/web-platform-tests/commit/e2c839d70a30ce0a3816bc63601a99ccf65a1654
Add WPT test for white space between adjacent elements in the same anonymous table cell. bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1452870 gecko-commit: b06fe2f80ddfc8baa9af24ac5ad5a9e4b98940a8 gecko-integration-branch: central gecko-reviewers: dholbert: https://github.com/w3c/web-platform-tests/commit/f70fcac671d4160ed6c752d24d4f8c2dab6f9745 [affecting this directory]
WorkerRef should make APIs able to work until the worker is completely terminated - part 2 - BroadcastChannel bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1450644 gecko-commit: f2341e59a4b58d36766806dab967a24526a35368 gecko-integration-branch: mozilla-inbound gecko-reviewers: asuth: https://github.com/w3c/web-platform-tests/commit/53d666cd5a4d5e5dc448f63e668c87766f9bc03d

 
@robertma,

There's already an issue filed for our broken behavior that this test shows (issue 694374). What's the suggested procedure here? Do I submit a CL that updates the bug number for this new TestExpectations line and then manually close this bug?
Re #1: Yes, that's correct. Thank you!

Comment 3 by e...@chromium.org, Apr 19 2018

Owner: dgro...@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 4 by bugdroid1@chromium.org, Apr 23 2018

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

commit d4fd62da30bfecc64d7d9bd17ce5904ca3333764
Author: David Grogan <dgrogan@chromium.org>
Date: Mon Apr 23 10:47:46 2018

[css-tables] Move failing wpt test to its real bug

A test that blink fails was imported and given a new bug by the
importer. We already have a bug for this behavior.

Just bookkeeping.

Bug:  834311 ,694374
Change-Id: I101c3b5dc24da8df90330a33c633ae2864295dcf
Reviewed-on: https://chromium-review.googlesource.com/1023038
Commit-Queue: Morten Stenshorne <mstensho@chromium.org>
Reviewed-by: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552664}
[modify] https://crrev.com/d4fd62da30bfecc64d7d9bd17ce5904ca3333764/third_party/WebKit/LayoutTests/TestExpectations

Status: Fixed (was: Assigned)

Sign in to add a comment