cc: ReadyToDraw notifications.

cc/layers/picture_layer_impl.h

Index: cc/layers/picture_layer_impl.h
diff --git a/cc/layers/picture_layer_impl.h b/cc/layers/picture_layer_impl.h
index 967b9302de217f353f8dda0e335020997f648d77..5d7fd9c0c2c4a687bbe707940dd179347cab6544 100644
--- a/cc/layers/picture_layer_impl.h
+++ b/cc/layers/picture_layer_impl.h
@@ -151,7 +151,16 @@ class CC_EXPORT PictureLayerImpl
   bool IsOnActiveOrPendingTree() const;
   // Virtual for testing.
   virtual bool HasValidTilePriorities() const;
+  bool AllTilesRequiredAreReadyToDraw(

[vmpstr, 2014/10/24 18:02:23]

Can you put a comment here explaining what this function does (and what the
parameters are).

Also, I think this can be private.

+      const char* event_name,

[vmpstr, 2014/10/24 18:02:23]

const std::string&?

[ernstm, 2014/10/27 21:13:56]

Removed.

+      const base::Callback<bool(PictureLayerTiling* tiling, const Tile*)>&
+          IsTileRequiredCallback) const;

[vmpstr, 2014/10/24 18:02:23]

this should be named as a variable, not a function.

[ernstm, 2014/10/27 21:13:56]

Done.

+  static bool IsTileRequiredForActivation(PictureLayerTiling* tiling,

[vmpstr, 2014/10/24 18:02:23]

These should be anonymous in the implementation.

[ernstm, 2014/10/27 21:13:56]

Done.

+                                          const Tile* tile);
+  static bool IsTileRequiredForDraw(PictureLayerTiling* tiling,
+                                    const Tile* tile);
   bool AllTilesRequiredForActivationAreReadyToDraw() const;
+  bool AllTilesRequiredForDrawAreReadyToDraw() const;
 
  protected:
   friend class LayerRasterTileIterator;