Layout Test images/paletted-png-with-color-profile.html is flaky |
|||||
Issue descriptionThe following layout test has been flaky on Android for long time images/paletted-png-with-color-profile.html Failing build log: https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Android%20%28Nexus4%29/builds/58513 Layout test result: https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Android__Nexus4_/58513/layout-test-results/results.html
,
Dec 22 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6f3b8fe15bdeef7ba4f3860042cbf99b25ea7812 commit 6f3b8fe15bdeef7ba4f3860042cbf99b25ea7812 Author: shimazu <shimazu@chromium.org> Date: Thu Dec 22 07:36:48 2016 Mark images/paletted-png-with-color-profile.html as skip on android BUG= 676561 TBR=schenney@chromium.org Review-Url: https://codereview.chromium.org/2596163002 Cr-Commit-Position: refs/heads/master@{#440353} [modify] https://crrev.com/6f3b8fe15bdeef7ba4f3860042cbf99b25ea7812/third_party/WebKit/LayoutTests/TestExpectations
,
Dec 22 2016
,
Dec 26 2016
Seems images/paletted-png-with-color-profile.html has no expected results on Android Nexus 4, looking at Layout test result: https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Android__Nexus4_/58513/layout-test-results/results.html
,
Feb 23 2017
,
Feb 23 2017
First issue: there is no expected image. schenney@ how about we try a rebaseline? Second issue, the actual image looks to have its R and B channels swapped. The test started failing around the time color correction was enabled for Android. msarett@ any clues as to why?
,
Feb 23 2017
I'm actually the one who added this test - and I'm fine with removing it. I'm not extremely familiar with the LayoutTest framework on Android, but it seems a real challenge to add a test there that isn't flaky... The BGRA/RGBA swap is pretty strange. I don't know how to explain that - I've not seen that outside of LayoutTests.
,
Apr 5 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/858bfb659e50d2d2debebd92847c59b884848ae4 commit 858bfb659e50d2d2debebd92847c59b884848ae4 Author: schenney <schenney@chromium.org> Date: Wed Apr 05 21:52:46 2017 Move Paint Team TestExpectations lines into appropriate sections This completes a review of all the Paint Team owned layout test failures. With this patch tests are grouped so we know which ones might be worth investigating in the future and so we can more easily incorporate new failures. There is one outstanding patch to be comnitted rebasing one set of tests. TBR=chrishtr@chromium.org BUG=309675,504613, 517449 , 549742 , 602110 , 606302 ,614910, 676561 Review-Url: https://codereview.chromium.org/2803633005 Cr-Commit-Position: refs/heads/master@{#462231} [modify] https://crrev.com/858bfb659e50d2d2debebd92847c59b884848ae4/third_party/WebKit/LayoutTests/NeverFixTests [modify] https://crrev.com/858bfb659e50d2d2debebd92847c59b884848ae4/third_party/WebKit/LayoutTests/TestExpectations
,
Apr 27 2017
We decided not to fix this. |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by shimazu@chromium.org
, Dec 22 2016Status: Assigned (was: Available)