Switch AwPixelInfo to wrap SkCanvasLayerState

android_webview/public/browser/draw_sw.h

Index: android_webview/public/browser/draw_sw.h
diff --git a/android_webview/public/browser/draw_sw.h b/android_webview/public/browser/draw_sw.h
index 463f59cb1f57db12817df662741cae32576cbebf..201679b91e93a6c68b6f8dbfc1deb934542e047c 100644
--- a/android_webview/public/browser/draw_sw.h
+++ b/android_webview/public/browser/draw_sw.h
@@ -12,21 +12,16 @@
 #error "Can't mix C and C++ when using jni.h"
 #endif
 
+struct SkCanvasState;
 class SkPicture;
 
-static const int kAwPixelInfoVersion = 2;
-
-// Values of the AwPixelInfo::config field.
-enum AwPixelConfig {
-  AwConfig_RGB_565 = 4,
-  AwConfig_ARGB_4444 = 5,
-  AwConfig_ARGB_8888 = 6,
-};
+static const int kAwPixelInfoVersion = 3;
 
 // Holds the information required to implement the SW draw to system canvas.
 struct AwPixelInfo {
   int version;          // The kAwPixelInfoVersion this struct was built with.
-  int config;           // |pixel| format: a value from AwPixelConfig.
+  SkCanvasState* state; // The externalize state in skia format.
+  // TODO(joth): Remove the rest in next patch.

[djsollen, 2013/09/05 12:10:07]

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]

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]

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]

Done.

   int width;            // In pixels.
   int height;           // In pixels.
   int row_bytes;        // Number of bytes from start of one line to next.