Switch AwPixelInfo to wrap SkCanvasLayerState

Switch AwPixelInfo to wrap SkCanvasLayerState

Using the new skia utility makes the canvas stack export much simpler

BUG=

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

[djsollen, 2013-09-05 12:10:06]

https://chromiumcodereview.appspot.com/23789003/diff/7001/android_webview/pub...
File android_webview/public/browser/draw_sw.h (right):

https://chromiumcodereview.appspot.com/23789003/diff/7001/android_webview/pub...
android_webview/public/browser/draw_sw.h:24: // TODO(joth): Remove the rest in
next patch.
is there a benefit to deferring the removal to a later CL?  If not, let's clean
it up now when we are updating the struct's version.

[joth, 2013-09-05 16:10:34]

https://codereview.chromium.org/23789003/diff/7001/android_webview/public/bro...
File android_webview/public/browser/draw_sw.h (right):

https://codereview.chromium.org/23789003/diff/7001/android_webview/public/bro...
android_webview/public/browser/draw_sw.h:24: // TODO(joth): Remove the rest in
next patch.
On 2013/09/05 12:10:07, djsollen wrote:
> is there a benefit to deferring the removal to a later CL?  If not, let's
clean
> it up now when we are updating the struct's version.

yeah... it's mostly so we have a clean compilation while the thing is merged
across repos and branches and what-not, otherwise I need to simultaneously land
this in 5 places.

That said, I see I've broken stuff anyway as this patch removes the 'config'
member and constants so I either need to fix that, or just delete it all as you
say.

[djsollen, 2013-09-05 16:23:44]

lgtm from a technical point of view, but you will want to update this patch to
either stage the removal of the other fields in the struct or remove them
entirely.

https://codereview.chromium.org/23789003/diff/7001/android_webview/public/bro...
File android_webview/public/browser/draw_sw.h (right):

https://codereview.chromium.org/23789003/diff/7001/android_webview/public/bro...
android_webview/public/browser/draw_sw.h:24: // TODO(joth): Remove the rest in
next patch.
I'm fine if you want to do it in stages, but it may be more hassle than it's
worth.  I'll leave that decision to you though.

[joth, 2013-09-05 22:03:39]

https://codereview.chromium.org/23789003/diff/7001/android_webview/public/bro...
File android_webview/public/browser/draw_sw.h (right):

https://codereview.chromium.org/23789003/diff/7001/android_webview/public/bro...
android_webview/public/browser/draw_sw.h:24: // TODO(joth): Remove the rest in
next patch.
On 2013/09/05 16:23:44, djsollen wrote:
> I'm fine if you want to do it in stages, but it may be more hassle than it's
> worth.  I'll leave that decision to you though.

Done.

[Kristian Monsen, 2013-09-05 23:43:19]

lgtm

[commit-bot: I haz the power, 2013-09-06 02:06:44]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/joth@chromium.org/23789003/23001

[commit-bot: I haz the power, 2013-09-06 06:01:44]

Change committed as 221593