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

Issue 8486011: roll skia to 2695 (Closed)

Created:
9 years, 1 month ago by reed1
Modified:
9 years, 1 month ago
Reviewers:
CC:
chromium-reviews, vandebo (ex-Chrome), bungeman-skia
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

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

Messages

Total messages: 2 (0 generated)
reed1
9 years, 1 month ago (2011-11-16 13:59:51 UTC) #1
epoger
9 years, 1 month ago (2011-11-16 16:36:55 UTC) #2
On 2011/11/16 13:59:51, reed1 wrote:

LGTM, with the following caveats based on trybot results.  The win_layout_rel
one is the only one I am really concerned about.

- win_layout_rel is failing one gpu test (canvas-radial-gradient-spreadMethod)
which does not appear to be flaky
- win crashed in chrome_framte_net_tests, but it looks like a non-code problem
- linux_chromeos failed 2 browser_tests, but I think those are flaky
- linux failed 3 ui_tests, but I think those are flaky

Powered by Google App Engine
This is Rietveld 408576698