[OSX] Turn on canvas_skia for OSX

[OSX] Turn on canvas_skia for OSX

Re-submit of r222192, with fixed tests.

R=asvitkine@chromium.org
BUG=151935
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=223202

[groby-ooo-7-16, 2013-09-12 23:20:32]

PTAL: Trying once more, this time with a test that runs in debug, too.

(Still not sure why the DCHECK didn't fire in the CQ. Don't we run tests with
DCHECK enabled?)

[Alexei Svitkine (slow), 2013-09-13 02:05:54]

I think it only uses the _rel bots which don't have dchecks enabled. It's quite
unfortunate.

[Alexei Svitkine (slow), 2013-09-13 02:06:36]

https://codereview.chromium.org/23604055/diff/1/ui/gfx/canvas_unittest_mac.mm
File ui/gfx/canvas_unittest_mac.mm (right):

https://codereview.chromium.org/23604055/diff/1/ui/gfx/canvas_unittest_mac.mm...
ui/gfx/canvas_unittest_mac.mm:54: int mac_width = kReallyLargeNumber;
Just pass in 0 instead.

[groby-ooo-7-16, 2013-09-13 18:09:07]

https://codereview.chromium.org/23604055/diff/1/ui/gfx/canvas_unittest_mac.mm
File ui/gfx/canvas_unittest_mac.mm (right):

https://codereview.chromium.org/23604055/diff/1/ui/gfx/canvas_unittest_mac.mm...
ui/gfx/canvas_unittest_mac.mm:54: int mac_width = kReallyLargeNumber;
I do this deliberately to catch cases where it accidentally returns the default
value. If you still want me to remove this, LMK
On 2013/09/13 02:06:37, Alexei Svitkine wrote:
> Just pass in 0 instead.

[Alexei Svitkine (slow), 2013-09-13 18:11:36]

Okay, LGTM

[commit-bot: I haz the power, 2013-09-13 19:10:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/groby@chromium.org/23604055/1

[commit-bot: I haz the power, 2013-09-14 01:32:18]

Change committed as 223202