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

Issue 629115 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Sep 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

100s of LayoutTests fail on Linux after b0df9e3fcb

Project Member Reported by atotic@chromium.org, Jul 18 2016

Issue description

Version: b0df9e3fcbca19d2a625fa274172bd3be4116261
OS: Linux

What steps will reproduce the problem?
(1) python third_party/WebKit/Tools/Scripts/run-webkit-tests

What is the expected output?

Test PASS.

What do you see instead?

Tests FAIL:
[1506/38895] virtual/gpu-ras...mate-rotate.html failed unexpectedly (image diff)
[1603/38895] virtual/rootlay...ombinations.html failed unexpectedly 
......

See thread on blink-dev: https://groups.google.com/a/chromium.org/forum/?pli=1#!topic/blink-dev/kOnyZUSb61w

Bisect points to hubbe's b0df9e3fcbca19d2a625fa274172bd3be4116261

If you look at the failures, the difference is color being slightly off.

Since this patch is retreiving the monitor color space, it all makes sense. And the CR comment says: 

"This may change a bazillion layout tests on Linux."

Please fix, this makes running local LayoutTests on Linux impossible.

 

Comment 1 by thakis@chromium.org, Jul 18 2016

Maybe put some code in content/test/layouttest_support.cc to disable using monitor color spaces in layout test mode.
Cc: chrishtr@chromium.org
This seems like it should be higher than P3 since I believe most Blink eng work on Linux and would frequently sanity check changes locally before running on remote bots which is much more time intensive?

Thanks to atotic@ for bisecting.
Shall we go forward with a revert while we figure out how to make layout tests have a pre-defined profile?

Comment 4 by hubbe@chromium.org, Jul 18 2016

I wonder how many people this affects though. Using ICC profiles on linux is relatively rare I thought.

Comment 5 by atotic@chromium.org, Jul 18 2016

I've never calibrated my display, or set the ICC profile. Running Unity, 2 monitors, one is 40" 4K.
I'm using everything out of box as well, and I believe I've seen pdr@ and chrishtr@ experience these errors recently.

Comment 7 by hubbe@chromium.org, Jul 18 2016

Reverting CL until we can work around the layout test issue.

Comment 8 by hubbe@chromium.org, Jul 19 2016

Status: Started (was: Untriaged)

Comment 9 by hubbe@chromium.org, Sep 22 2016

Status: Fixed (was: Started)

Sign in to add a comment