Move Work From CompositorFrameSinkSupport() To Init()

cc/surfaces/compositor_frame_sink_support.h

Index: cc/surfaces/compositor_frame_sink_support.h
diff --git a/cc/surfaces/compositor_frame_sink_support.h b/cc/surfaces/compositor_frame_sink_support.h
index bdafb4933e6c0f752edc1fde38a9d63fd003d6b5..16161d96fbf19b900be35555cc4195c1261cf284 100644
--- a/cc/surfaces/compositor_frame_sink_support.h
+++ b/cc/surfaces/compositor_frame_sink_support.h
@@ -37,6 +37,8 @@ class CC_SURFACES_EXPORT CompositorFrameSinkSupport
 
   ~CompositorFrameSinkSupport() override;
 
+  void Init();

[boliu, 2017/04/10 17:02:23]

it's usually better to have a public factory method that calls Init, and make
the constructor private. does that work here?

[Alex Z., 2017/04/10 17:34:29]

We're still in discussion about the necessity of the Init since
SurfaceFactory and SurfaceFactoryClient are soon to be removed.

I will get back to you on the factory method when that's settled.

+
   const FrameSinkId& frame_sink_id() const { return frame_sink_id_; }
 
   Surface* current_surface_for_testing() {