Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(228)

Issue 57433013: Roll Skia DEPS to r12124 (Closed)

Created:
7 years, 1 month ago by robertphillips
Modified:
7 years, 1 month ago
Reviewers:
bsalomon, rmistry, reed1
CC:
chromium-reviews, skia-team_google.com
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : Suppress blur optimization #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -2 lines) Patch
M DEPS View 1 1 chunk +2 lines, -2 lines 0 comments Download
M skia/skia_library.gypi View 1 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
robertphillips
control: https://codereview.chromium.org/57433014/
7 years, 1 month ago (2013-11-05 02:20:32 UTC) #1
robertphillips
This roll attempt is failing on the following 79 layout tests: Regressions: Unexpected image-only failures ...
7 years, 1 month ago (2013-11-05 12:47:54 UTC) #2
robertphillips
The majority of these changes are what you would expect from r12113 (Change SkBlurImageFilter to ...
7 years, 1 month ago (2013-11-05 13:07:03 UTC) #3
robertphillips
Added the suppression to skia_library.gypi & reran the bots. This looks ready to go.
7 years, 1 month ago (2013-11-05 13:44:46 UTC) #4
rmistry
LGTM
7 years, 1 month ago (2013-11-05 13:46:25 UTC) #5
robertphillips
Committed patchset #2 manually as r232963 (presubmit successful).
7 years, 1 month ago (2013-11-05 13:47:10 UTC) #6
reed1
7 years, 1 month ago (2013-11-05 16:00:15 UTC) #7
Message was sent while issue was closed.
Does the new impl of blur maintain opaqueness? i.e. if all the sample values are
0xFF, is the result 0xFF? Couldn't tell if we need to add anything in the code
before shifting to ensure this.

Powered by Google App Engine
This is Rietveld 408576698