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

Issue 660118 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Pixel_2DCanvasWebGL fails on Android GPU.FYI bots

Project Member Reported by ynovikov@chromium.org, Oct 27 2016

Issue description

Started in:
https://build.chromium.org/p/chromium.gpu.fyi/builders/Android%20Release%20%28Nexus%205%29/builds/2442
https://build.chromium.org/p/chromium.gpu.fyi/builders/Android%20Release%20%28Nexus%205X%29/builds/3498
https://build.chromium.org/p/chromium.gpu.fyi/builders/Android%20Release%20%28Nexus%206%29/builds/2584
https://build.chromium.org/p/chromium.gpu.fyi/builders/Android%20Release%20%28Nexus%206P%29/builds/2885
https://build.chromium.org/p/chromium.gpu.fyi/builders/Android%20Release%20%28Nexus%209%29/builds/2767

Error is:
Traceback (most recent call last):
  _RunGpuTest at content/test/gpu/gpu_tests/gpu_integration_test.py:78
    self.RunActualGpuTest(url, *args)
  RunActualGpuTest at content/test/gpu/gpu_tests/pixel_integration_test.py:202
    self.fail('Reference image did not match captured screen')
  fail at /usr/lib/python2.7/unittest/case.py:412
    raise self.failureException(msg)
AssertionError: Reference image did not match captured screen

Locals:
  msg : 'Re`ference image did not match captured screen'

I think maybe https://codereview.chromium.org/2451273003 is the reason?
 
Cc: ajuma@chromium.org
This should be fixed by https://codereview.chromium.org/2451383003/

Comment 2 by ajuma@chromium.org, Oct 27 2016

Actually, that patch was incorrect but I'm working on a correct one:https://codereview.chromium.org/2459613003/

Comment 3 by ajuma@chromium.org, Oct 27 2016

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

Comment 4 by bugdroid1@chromium.org, Oct 28 2016

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

commit 679b26ff61f303d85a56f5df1fd6199f209ad7ca
Author: ajuma <ajuma@chromium.org>
Date: Fri Oct 28 02:14:49 2016

Correctly rebaseline 2DCanvasWebGL pixel test

This is a follow-up to http://crrev.com/2451383003, which updated
an obsolete file.

BUG= 629522 , 660118 

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
NOTRY=true

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

[modify] https://crrev.com/679b26ff61f303d85a56f5df1fd6199f209ad7ca/content/test/gpu/gpu_tests/pixel_expectations.py
[modify] https://crrev.com/679b26ff61f303d85a56f5df1fd6199f209ad7ca/content/test/gpu/gpu_tests/pixel_test_pages.py

Project Member

Comment 5 by bugdroid1@chromium.org, Oct 28 2016

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

commit 7ca50e2014e8987c6a4d5e74d39be65893c91837
Author: ajuma <ajuma@chromium.org>
Date: Fri Oct 28 15:47:07 2016

Remove failure expectation for 2DCanvasWebGL pixel test

The reference images have been regenerated and the test is now
passing.

TBR=kbr@chromium.org

BUG= 629522 , 660118 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel

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

[modify] https://crrev.com/7ca50e2014e8987c6a4d5e74d39be65893c91837/content/test/gpu/gpu_tests/pixel_expectations.py

Comment 6 by ajuma@chromium.org, Oct 28 2016

Status: Fixed (was: Assigned)

Sign in to add a comment