cc: Clean up RecordingSource API

cc/test/fake_picture_layer.h

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CC_TEST_FAKE_PICTURE_LAYER_H_
 #define CC_TEST_FAKE_PICTURE_LAYER_H_
 
 #include <stddef.h>
 
 #include <memory>
@@ skipping 24 matching lines @@
   void reset_push_properties_count() { push_properties_count_ = 0; }
 
   void set_always_update_resources(bool always_update_resources) {
     always_update_resources_ = always_update_resources;
   }
 
   bool Update() override;
 
   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,
                    std::unique_ptr<RecordingSource> source);
   ~FakePictureLayer() override;
 
   int update_count_;
   size_t push_properties_count_;
   bool always_update_resources_;
+
+  bool force_unsuitable_for_gpu_rasterization_;
 };
 
 }  // namespace cc
 
 #endif  // CC_TEST_FAKE_PICTURE_LAYER_H_