Commit 836b2d8a authored by Andres Calderon Jaramillo's avatar Andres Calderon Jaramillo Committed by Commit Bot

Reduce size of test rectangles for Pixel_ReflectedDiv.

This CL reduces the sizes of the expected-colors rectangles for the
Pixel_ReflectedDiv test so that it passes on Nexus 5. The result in
that device looks visually fine [1], but pixels along some edges are
slightly different than the expected color:

actual   = rgba(255, 130, 4, 255)
expected = rgba(255, 128, 0, 255)

Therefore, reducing the size of the test rectangles to avoid the edges is
reasonable.

[1] https://chrome-gpu-gold.skia.org/img/images/f357a4dd6c87c632ae787203fcc307fa.png

Bug: 1033279
Test: Same reliability test as in CL:1979059.
Change-Id: I1d7ed61f5d1c7c2cacacb45d2d72b363580cba9d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2015560Reviewed-by: default avatarKenneth Russell <kbr@chromium.org>
Reviewed-by: default avatarBrian Sheedy <bsheedy@chromium.org>
Commit-Queue: Andres Calderon Jaramillo <andrescj@chromium.org>
Cr-Commit-Position: refs/heads/master@{#734248}
parent 3e86e830
...@@ -157,20 +157,20 @@ class PixelTestPages(object): ...@@ -157,20 +157,20 @@ class PixelTestPages(object):
expected_colors=[ expected_colors=[
{ {
'comment': 'inside original div, orange', 'comment': 'inside original div, orange',
'location': [0, 0], 'location': [5, 5],
'size': [100, 99], 'size': [90, 90],
'color': [255, 128, 0], 'color': [255, 128, 0],
}, },
{ {
'comment': 'outside both div and reflection, in between, white', 'comment': 'outside both div and reflection, in between, white',
'location': [0, 101], 'location': [5, 105],
'size': [100, 98], 'size': [90, 90],
'color': [255, 255, 255], 'color': [255, 255, 255],
}, },
{ {
'comment': 'inside reflection, orange', 'comment': 'inside reflection, orange',
'location': [0, 201], 'location': [5, 205],
'size': [100, 99], 'size': [90, 90],
'color': [255, 128, 0], 'color': [255, 128, 0],
} }
]), ]),
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment