cc: Clean up RecordingSource API

cc/test/fake_recording_source.h

 // Copyright 2015 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_RECORDING_SOURCE_H_
 #define CC_TEST_FAKE_RECORDING_SOURCE_H_
 
 #include <stddef.h>
 
 #include "cc/base/region.h"
@@ skipping 39 matching lines @@
   // RecordingSource overrides.
   scoped_refptr<RasterSource> CreateRasterSource(
       bool can_use_lcd) const override;
   bool IsSuitableForGpuRasterization() const override;
 
   void SetDisplayListUsesCachedPicture(bool use_cached_picture) {
     client_.set_display_list_use_cached_picture(use_cached_picture);
   }
 
   void SetRecordedViewport(const gfx::Rect& recorded_viewport) {
-    recorded_viewport_ = recorded_viewport;
+    clc_data_.recorded_viewport = recorded_viewport;

[Menglin, 2016/07/19 22:56:37]

Oh I think you don't have to look into the change here and in
fake_recording_source.cc because they're test code

   }
 
   void SetLayerBounds(const gfx::Size& layer_bounds) {
-    size_ = layer_bounds;
+    pl_data_.size = layer_bounds;
     client_.set_bounds(layer_bounds);
   }
 
   void SetClearCanvasWithDebugColor(bool clear) {
-    clear_canvas_with_debug_color_ = clear;
+    pl_data_.clear_canvas_with_debug_color = clear;
   }
 
   void Rerecord() {
-    SetNeedsDisplayRect(recorded_viewport_);
+    SetNeedsDisplayRect(clc_data_.recorded_viewport);
     Region invalidation;
-    UpdateAndExpandInvalidation(&client_, &invalidation, size_, 0,
+    UpdateAndExpandInvalidation(&client_, &invalidation, pl_data_.size,
                                 RECORD_NORMALLY);
   }
 
   void add_draw_rect(const gfx::Rect& rect) {
     client_.add_draw_rect(rect, default_paint_);
   }
 
   void add_draw_rect_with_paint(const gfx::Rect& rect, const SkPaint& paint) {
     client_.add_draw_rect(rect, paint);
   }
@@ skipping 23 matching lines @@
   }
 
   void set_default_paint(const SkPaint& paint) { default_paint_ = paint; }
 
   void set_reported_memory_usage(size_t reported_memory_usage) {
     client_.set_reported_memory_usage(reported_memory_usage);
   }
 
   void reset_draws() {
     client_ = FakeContentLayerClient();
-    client_.set_bounds(size_);
+    client_.set_bounds(pl_data_.size);
   }
 
   void SetUnsuitableForGpuRasterization() {
     force_unsuitable_for_gpu_rasterization_ = true;
   }
 
   void SetPlaybackAllowedEvent(base::WaitableEvent* event) {
     playback_allowed_event_ = event;
   }
 
-  DisplayItemList* display_list() const { return display_list_.get(); }
+  DisplayItemList* display_list() const { return clc_data_.display_list.get(); }
 
   // Checks that the basic properties of the |other| match |this|.  For the
   // DisplayItemList, it checks that the painted result matches the painted
   // result of |other|.
   bool EqualsTo(const FakeRecordingSource& other);
 
  private:
   FakeContentLayerClient client_;
   SkPaint default_paint_;
   bool force_unsuitable_for_gpu_rasterization_;
   base::WaitableEvent* playback_allowed_event_;
 };
 
 }  // namespace cc
 
 #endif  // CC_TEST_FAKE_RECORDING_SOURCE_H_