cc: Let skia veto gpu rasterization.

cc/layers/picture_layer.h

Index: cc/layers/picture_layer.h
diff --git a/cc/layers/picture_layer.h b/cc/layers/picture_layer.h
index 78bb8f44644fa59aa0902207e8a9067792c3b14f..28f4f906c0fe9b1c80db633b3cdc0039436a9cec 100644
--- a/cc/layers/picture_layer.h
+++ b/cc/layers/picture_layer.h
@@ -41,6 +41,9 @@ class CC_EXPORT PictureLayer : public Layer {
   ContentLayerClient* client() { return client_; }
 
   void SetHasGpuRasterizationHint(bool has_hint);
+  bool ShouldUseGpuRasterization() const;
+
+  PicturePile* GetPicturePileForTesting() const { return pile_.get(); }
 
  protected:
   explicit PictureLayer(ContentLayerClient* client);
@@ -57,7 +60,9 @@ class CC_EXPORT PictureLayer : public Layer {
   Region pile_invalidation_;
   gfx::Rect last_updated_visible_content_rect_;
   bool is_mask_;
-  bool has_gpu_rasterization_hint_;
+
+  enum TriBool { TRIBOOL_UNKNOWN, TRIBOOL_FALSE, TRIBOOL_TRUE };

[enne (OOO), 2014/04/04 18:34:19]

I think this would be clearer if you just added a second bool for
gpu_rasterization_has_ever_been_disabled_.  

Actually, rather than adding a second boolean, could you also use the
PicturePile's suitable boolean as the "sticky" disabled flag directly?

[alokp, 2014/04/04 18:50:34]

I do not understand the comment about PicturePile boolean. Can you elaborate?

[enne (OOO), 2014/04/04 19:40:07]

I'm saying that pile_->is_suitable_for_gpu_rasterization() is already sticky
across frames, and that rather than using a tri-state enum (which I find
confusing in logic), that you can just check that function as a "has ever been
disabled" flag.

+  TriBool has_gpu_rasterization_hint_;
 
   int update_source_frame_number_;