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

Issue 10541005: roll skia to 4156 (Closed)

Created:
8 years, 6 months ago by reed1
Modified:
8 years, 6 months ago
Reviewers:
epoger
CC:
chromium-reviews, skia-dev_google.com, epoger
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -2 lines) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download
M skia/skia.gyp View 2 chunks +4 lines, -0 lines 0 comments Download
M skia/skia_test_expectations.txt View 1 2 1 chunk +7 lines, -1 line 0 comments Download
M webkit/tools/layout_tests/test_expectations.txt View 1 1 chunk +4 lines, -0 lines 1 comment Download

Messages

Total messages: 8 (0 generated)
reed1
control #2 https://chromiumcodereview.appspot.com/10477011
8 years, 6 months ago (2012-06-05 14:08:42 UTC) #1
reed1
Not sure about these... (brian to investigate) platform/chromium/virtual/gpu/canvas/philip/tests/2d.gradient.radial.cone.cylinder.html = TEXT platform/chromium/virtual/gpu/fast/canvas/canvas-currentColor.html = TEXT Rebaseline these ...
8 years, 6 months ago (2012-06-05 14:49:55 UTC) #2
reed1
https://code.google.com/p/chromium/issues/detail?id=131187
8 years, 6 months ago (2012-06-05 15:02:29 UTC) #3
reed1
patch #1 is a full try suite. It showed the need for some suppressions/rebaselines. patch ...
8 years, 6 months ago (2012-06-05 15:15:15 UTC) #4
reed1
patch #3 updated to include css3/filters/effect-saturate-hw.html to be rebaselined. PTAL
8 years, 6 months ago (2012-06-05 15:59:56 UTC) #5
epoger
Looks pretty good. But maybe run against the full set of trybots in http://skia.googlecode.com/svn/trunk/trybots_to_run/layout before ...
8 years, 6 months ago (2012-06-05 16:14:12 UTC) #6
reed1
webkit patch is in the queue (I couldn't land manually). PTAL
8 years, 6 months ago (2012-06-05 19:15:41 UTC) #7
bsalomon
8 years, 6 months ago (2012-06-05 19:19:24 UTC) #8
On 2012/06/05 19:15:41, reed1 wrote:
> webkit patch is in the queue (I couldn't land manually).
> 
> PTAL

LGTM

Powered by Google App Engine
This is Rietveld 408576698