aura: Enable use_canvas_skia=1 on aura.

aura: Enable use_canvas_skia=1 on aura.

BUG=111587, 105550
TEST=manual

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=128170

[reveman, 2012-03-21 18:12:07]

Here's the gfx::Canvas side of using platform independent canvases in aura.

[Alexei Svitkine (slow), 2012-03-21 18:25:47]

LGTM

Also, please add bug 105550 to the BUG= list.

https://chromiumcodereview.appspot.com/9813017/diff/1/build/common.gypi
File build/common.gypi (right):

https://chromiumcodereview.appspot.com/9813017/diff/1/build/common.gypi#newco...
build/common.gypi:429: # Use platform independent implementation of gfx::Canvas
in Aura.
Nit: Change this comment to:

# Enable Skia UI text drawing incrementally on different platforms.
# http://crbug.com/105550
#
# On Aura, this enables per-tile painting in the browser compositor.

[piman, 2012-03-21 18:39:11]

lgtm

[Alexei Svitkine (slow), 2012-03-21 21:23:06]

Can you change the CL name/description to say its enabling use_canvas_skia=1 on
aura?

Otherwise, the name is very confusing, especially with
https://chromiumcodereview.appspot.com/9722032/ having a similar name.

[google-reveman, 2012-03-21 21:29:18]

done

On Wed, Mar 21, 2012 at 5:23 PM,  <asvitkine@chromium.org> wrote:
> Can you change the CL name/description to say its enabling use_canvas_skia=1
> on
> aura?
>
> Otherwise, the name is very confusing, especially with
> https://chromiumcodereview.appspot.com/9722032/ having a similar name.
>
> https://chromiumcodereview.appspot.com/9813017/

[commit-bot: I haz the power, 2012-03-21 22:36:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/reveman@chromium.org/9813017/5003

[commit-bot: I haz the power, 2012-03-22 02:53:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/reveman@chromium.org/9813017/5003