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

Issue 48113022: Roll Skia DEPS to r12012 (Closed)

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

Description

Patch Set 1 #

Patch Set 2 : Roll Skia DEPS to r12012 with suppressions #

Patch Set 3 : Reverted change to cc/trees/layer_tree_host_pixeltest_filters.cc #

Patch Set 4 : rebaseline BackgroundFilterBlurOffAxis test #

Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -2 lines) Patch
M DEPS View 1 2 1 chunk +2 lines, -2 lines 0 comments Download
M cc/test/data/background_filter_blur_off_axis.png View 1 2 3 Binary file 0 comments Download
M skia/skia_test_expectations.txt View 1 1 chunk +20 lines, -0 lines 0 comments Download

Messages

Total messages: 11 (0 generated)
robertphillips
control: https://codereview.chromium.org/51643002/
7 years, 1 month ago (2013-10-29 22:55:35 UTC) #1
robertphillips
This roll is failing widely on the BackgroundFilterBlurOffAxis test (linux, linux_asan, mac, mac_asan) and is ...
7 years, 1 month ago (2013-10-30 12:44:23 UTC) #2
robertphillips
The BackgroundFilterBlurOffAxis failure appears to be a 1 out of 255 pixel diff at two ...
7 years, 1 month ago (2013-10-30 12:57:15 UTC) #3
robertphillips
+enne for cc change
7 years, 1 month ago (2013-10-30 13:56:20 UTC) #4
robertphillips
We have green on the mac and linux layout tests (both debug and release) and ...
7 years, 1 month ago (2013-10-30 14:41:26 UTC) #5
reed1
lgtm
7 years, 1 month ago (2013-10-30 14:52:16 UTC) #6
enne (OOO)
lgtm
7 years, 1 month ago (2013-10-30 18:42:25 UTC) #7
robertphillips
Committed patchset #4 manually as r231886 (presubmit successful).
7 years, 1 month ago (2013-10-30 18:44:39 UTC) #8
danakj
On 2013/10/30 18:44:39, robertphillips wrote: > Committed patchset #4 manually as r231886 (presubmit successful). Looks ...
7 years, 1 month ago (2013-10-30 19:04:29 UTC) #9
danakj
On 2013/10/30 18:44:39, robertphillips wrote: > Committed patchset #4 manually as r231886 (presubmit successful). Looks ...
7 years, 1 month ago (2013-10-30 19:04:41 UTC) #10
danakj
7 years, 1 month ago (2013-10-30 19:09:13 UTC) #11
Message was sent while issue was closed.
On 2013/10/30 19:04:41, danakj wrote:
> On 2013/10/30 18:44:39, robertphillips wrote:
> > Committed patchset #4 manually as r231886 (presubmit successful).
> 
> Looks like thie rebaseline isn't correctly matching the new output?

Works fine after sync. PEBKAC.

Powered by Google App Engine
This is Rietveld 408576698