New issue
Advanced search Search tips

Issue 840208 link

Starred by 1 user
Status: Fixed
Owner: ----
Closed: May 2018
Cc:
ksakamoto@chromium.org
e...@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

virtual/layout_ng/fast/inline/inline-empty-block-continuation-remove.html and 4 other(s) in webkit_layout_tests failing on chromium.webkit/WebKit Mac10.11

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, May 7 2018 
Filed by sheriff-o-matic@appspot.gserviceaccount.com on behalf of ksakamoto@chromium.org

* virtual/layout_ng/fast/inline/inline-empty-block-continuation-remove.html
* virtual/layout_ng_experimental/fast/multicol/dynamic/remove-block-from-content-after-spanner.html
* virtual/layout_ng_experimental/fast/multicol/dynamic/remove-block-from-content-before-spanner.html
* virtual/layout_ng_experimental/fast/multicol/dynamic/remove-block-from-content-between-spanners.html
* virtual/layout_ng_experimental/fast/multicol/dynamic/remove-inline-and-spanner-after-spanner-foreignObject.html

https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.10%20Tests/31870
https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.11%20Tests/25922
https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.12%20Tests/12582
https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.13%20Tests/2275

Suspected change: [LayoutNG] Optimize re-layout for inlines https://chromium-review.googlesource.com/c/chromium/src/+/986982

These tests already have Failure / Crash expectations, so I'll add Timeout expectation.
Project Member

Comment 1 by bugdroid1@chromium.org, May 7 2018 

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

commit 03f0979c7843376980ffa4674f128d2c947e1d17
Author: Kunihiko Sakamoto <ksakamoto@chromium.org>
Date: Mon May 07 01:49:18 2018

Add timeout expectations for LayoutNG

These tests are timing out on Mac:
* virtual/layout_ng/fast/inline/inline-empty-block-continuation-remove.html
* virtual/layout_ng_experimental/fast/multicol/dynamic/remove-block-from-content-after-spanner.html
* virtual/layout_ng_experimental/fast/multicol/dynamic/remove-block-from-content-before-spanner.html
* virtual/layout_ng_experimental/fast/multicol/dynamic/remove-block-from-content-between-spanners.html
* virtual/layout_ng_experimental/fast/multicol/dynamic/remove-inline-and-spanner-after-spanner-foreignObject.html

TBR=eae@chromium.org

Bug:  840208 
Change-Id: Ibf439881543b665ac8065c2b0bc2db4148ac2d1e
Reviewed-on: https://chromium-review.googlesource.com/1045956
Reviewed-by: Kunihiko Sakamoto <ksakamoto@chromium.org>
Commit-Queue: Kunihiko Sakamoto <ksakamoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#556363}
[modify] https://crrev.com/03f0979c7843376980ffa4674f128d2c947e1d17/third_party/WebKit/LayoutTests/TestExpectations

Comment 2 by ksakamoto@chromium.org, May 7 2018

Status: Fixed (was: Available)
►

Sign in to add a comment