New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 714862 link

Starred by 1 user
Status: Fixed
Owner:
█ robho...@gmail.com
Last visit > 30 days ago
Closed: Jul 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Disable Test - Webkit Layout Test bug41262-3.html

Project Member Reported by hcarmona@chromium.org, Apr 24 2017 
Test failure caused by https://codereview.chromium.org/2593263003

That's a very large CL and the test failure looks expected based on the CL description "Use a non-replaced inline container for image alt text", so opting to disable test rather than revert change.

Test failures here:
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Android%20%28Nexus4%29?numbuilds=100

First failure here:
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Android%20%28Nexus4%29/builds/64085

Please update test and re-enable.
expected.png
39.9 KB View Download
actual.png
39.6 KB View Download
diff.png
39.0 KB View Download

Comment 1 by hcarmona@chromium.org, Apr 24 2017

Summary: Disable Test - Webkit Layout Test bug41262-3.html (was: Disable Test)
Project Member

Comment 3 by bugdroid1@chromium.org, Apr 25 2017 

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

commit d77cac6405f78b26e5764bc23470ed088e209673
Author: hcarmona <hcarmona@chromium.org>
Date: Tue Apr 25 01:09:17 2017

Disable Test - Webkit Layout Test  bug41262 -3.html

TBR=robhogan@gmail.com
BUG= 714862 

Review-Url: https://codereview.chromium.org/2837913003
Cr-Commit-Position: refs/heads/master@{#466849}

[modify] https://crrev.com/d77cac6405f78b26e5764bc23470ed088e209673/third_party/WebKit/LayoutTests/TestExpectations
Project Member

Comment 4 by bugdroid1@chromium.org, Jul 30 2017 

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

commit b1758611761612bb3cbb04210cdb1cb091e70e0b
Author: Robert Hogan <robhogan@gmail.com>
Date: Sun Jul 30 21:25:50 2017

Re-enable test  bug41262 -3.html

Bug:  714862 
Change-Id: Ic584853df872456a788b3ef45a06d8c346f50516
Reviewed-on: https://chromium-review.googlesource.com/593307
Reviewed-by: Robert Hogan <robhogan@gmail.com>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Commit-Queue: Emil A Eklund <eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#490690}
[modify] https://crrev.com/b1758611761612bb3cbb04210cdb1cb091e70e0b/third_party/WebKit/LayoutTests/TestExpectations

Comment 5 by robho...@gmail.com, Jul 31 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment