Introducing SurfaceReferenceFactory

cc/layers/surface_layer_impl.h

Index: cc/layers/surface_layer_impl.h
diff --git a/cc/layers/surface_layer_impl.h b/cc/layers/surface_layer_impl.h
index e5d4128bc161f7a7a2bc9f6e4c9ad391c21da183..ca6ae5946fcabf8fecf598452f16117e002ff2c5 100644
--- a/cc/layers/surface_layer_impl.h
+++ b/cc/layers/surface_layer_impl.h
@@ -11,6 +11,7 @@
 #include "base/memory/ptr_util.h"
 #include "cc/base/cc_export.h"
 #include "cc/layers/layer_impl.h"
+#include "cc/surfaces/surface_embedding_base.h"
 #include "cc/surfaces/surface_id.h"
 
 namespace cc {
@@ -23,10 +24,8 @@ class CC_EXPORT SurfaceLayerImpl : public LayerImpl {
   }
   ~SurfaceLayerImpl() override;
 
-  void SetSurfaceId(const SurfaceId& surface_id);
-  void SetSurfaceScale(float scale);
-  void SetSurfaceSize(const gfx::Size& size);
-  const SurfaceId& surface_id() const { return surface_id_; }
+  void SetSurfaceInfo(const SurfaceInfo&);

[Fady Samuel, 2016/12/12 16:12:00]

const SurfaceInfo& surface_info

+  const SurfaceId& surface_id() const { return surface_info_.id; }

[Fady Samuel, 2016/12/12 16:12:00]

nit: is this necessary?

 
   // LayerImpl overrides.
   std::unique_ptr<LayerImpl> CreateLayerImpl(LayerTreeImpl* tree_impl) override;
@@ -43,9 +42,7 @@ class CC_EXPORT SurfaceLayerImpl : public LayerImpl {
   void AsValueInto(base::trace_event::TracedValue* dict) const override;
   const char* LayerTypeAsString() const override;
 
-  SurfaceId surface_id_;
-  gfx::Size surface_size_;
-  float surface_scale_;
+  SurfaceInfo surface_info_;
 
   DISALLOW_COPY_AND_ASSIGN(SurfaceLayerImpl);
 };