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

Issue 8890034: Turn on use_skia by default for Mac (Closed)

Created:
9 years ago by epoger
Modified:
9 years ago
Reviewers:
reed1
CC:
chromium-reviews
Visibility:
Public.

Description

Turn on use_skia by default for Mac BUG=101731 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=113807

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M build/use_skia_on_mac.gypi View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
epoger
I plan to commit this Friday morning, as detailed in bug 101731
9 years ago (2011-12-08 20:59:55 UTC) #1
reed1
lgtm
9 years ago (2011-12-08 21:08:12 UTC) #2
epoger
9 years ago (2011-12-09 15:01:14 UTC) #3
On 2011/12/08 21:08:12, reed1 wrote:
> lgtm

Mac trybot is failing, but that's because the trybot is apparently building
against an old version of chrome/renderer/print_web_view_helper_mac.mm . 
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/print_web_vie...
verifies that I updated that file 20 hours ago, and it has not been reverted
since then...

The waterfall bots may need a clobber to pick up the new use_skia value anyway,
so I'm going to go ahead and commit this change.

Powered by Google App Engine
This is Rietveld 408576698