cc: Clean up RecordingSource API

cc/test/fake_picture_layer.h

Index: cc/test/fake_picture_layer.h
diff --git a/cc/test/fake_picture_layer.h b/cc/test/fake_picture_layer.h
index 038e97543d9ad206f0750f812363b93bf481d5f8..744048f30eae79dd71b811f3e5ea3b527a050f63 100644
--- a/cc/test/fake_picture_layer.h
+++ b/cc/test/fake_picture_layer.h
@@ -42,6 +42,18 @@ class FakePictureLayer : public PictureLayer {
 
   void PushPropertiesTo(LayerImpl* layer) override;
 
+  void SetRecordedViewport(const gfx::Rect& recorded_viewport) {

[vmpstr, 2016/07/27 18:18:37]

can you make these hacker_case instead of CamelCase? (most functions specific to
fakes do this, I think). 

[Menglin, 2016/07/27 19:17:48]

you mean SetRecordedViewport and SetForceUnsuitableForGpuRasterization  ?

[vmpstr, 2016/07/28 18:19:58]

I mean if the function is inlined and it's a fake only function, then it should
be

set_recorded_viewport and
set_force_unsuitable_for_gpu_rasterization

It's just a convention nit...

[Menglin, 2016/07/29 23:50:14]

Done.

+    inputs_.recorded_viewport = recorded_viewport;
+  }
+
+  DisplayItemList* display_list() const { return inputs_.display_list.get(); }
+
+  void SetForceUnsuitableForGpuRasterization(bool flag) {
+    force_unsuitable_for_gpu_rasterization_ = flag;
+  }
+
+  bool IsSuitableForGpuRasterization() const override;
+
  private:
   explicit FakePictureLayer(ContentLayerClient* client);
   FakePictureLayer(ContentLayerClient* client,
@@ -51,6 +63,8 @@ class FakePictureLayer : public PictureLayer {
   int update_count_;
   size_t push_properties_count_;
   bool always_update_resources_;
+
+  bool force_unsuitable_for_gpu_rasterization_;
 };
 
 }  // namespace cc