cc: Moved contents_swizzled out of TileManager

cc/resources/managed_tile_state.h

Index: cc/resources/managed_tile_state.h
diff --git a/cc/resources/managed_tile_state.h b/cc/resources/managed_tile_state.h
index 186ddcb58d66cb8391f2f5de5a835d22d3f90312..ed7612f55db92e3e692e7d8e70927a541e6c6b8d 100644
--- a/cc/resources/managed_tile_state.h
+++ b/cc/resources/managed_tile_state.h
@@ -8,6 +8,7 @@
 #include <list>
 
 #include "base/memory/scoped_ptr.h"
+#include "cc/resources/platform_color.h"
 #include "cc/resources/resource_pool.h"
 #include "cc/resources/resource_provider.h"
 #include "cc/resources/tile_manager.h"
@@ -51,7 +52,11 @@ class CC_EXPORT ManagedTileState {
       }
 
       bool contents_swizzled() const {
-        return contents_swizzled_;
+        return !PlatformColor::SameComponentOrder(resource_format_);

[vmpstr, 2013/04/04 22:30:00]

Should we change PlatformColor functions to be hacker_style? They seem to be
fairly small functions all defined in the header.

[enne (OOO), 2013/04/04 22:37:28]

I'm pretty indifferent on this.  hacker style is optional.

+      }
+
+      void set_resource_format(GLenum format) {
+        resource_format_ = format;
       }
 
       bool requires_resource() const {
@@ -84,17 +89,13 @@ class CC_EXPORT ManagedTileState {
         mode_ = PICTURE_PILE_MODE;
       }
 
-      void set_contents_swizzled(bool contents_swizzled) {
-        contents_swizzled_ = contents_swizzled;
-      }
-
       Mode mode_;
       SkColor solid_color_;
 
       scoped_ptr<ResourcePool::Resource> resource_;
       bool resource_is_being_initialized_;
       bool can_be_freed_;
-      bool contents_swizzled_;
+      GLenum resource_format_;

[enne (OOO), 2013/04/04 22:37:28]

Is this different than resource_->format()?

   };