Split SurfaceFactoryClient Into Four Interfaces

cc/surfaces/framesink_manager.h

Index: cc/surfaces/framesink_manager.h
diff --git a/cc/surfaces/framesink_manager.h b/cc/surfaces/framesink_manager.h
index 087126c1fb28361da98709cafc236eda18a0bcbe..ab4e571de0fe68fe27e142440c511f384bd78dfb 100644
--- a/cc/surfaces/framesink_manager.h
+++ b/cc/surfaces/framesink_manager.h
@@ -18,7 +18,7 @@
 
 namespace cc {
 class BeginFrameSource;
-class SurfaceFactoryClient;
+class FrameSinkManagerClient;
 
 namespace test {
 class CompositorFrameSinkSupportTest;
@@ -43,11 +43,11 @@ class CC_SURFACES_EXPORT FrameSinkManager {
   // However, DelegatedFrameHost can register itself as a client before its
   // relationship with the ui::Compositor is known.
 
-  // Associates a SurfaceFactoryClient with the  frame_sink_id it  uses.
-  // SurfaceFactoryClient and framesink allocators have a 1:1 mapping.
+  // Associates a FrameSinkManagerClient with the  frame_sink_id it  uses.

[danakj, 2017/04/19 14:52:56]

bonus whitespace after "with the"

[Alex Z., 2017/04/19 15:15:40]

Done.

+  // FrameSinkManagerClient and framesink allocators have a 1:1 mapping.
   // Caller guarantees the client is alive between register/unregister.
   void RegisterSurfaceFactoryClient(const FrameSinkId& frame_sink_id,
-                                    SurfaceFactoryClient* client);
+                                    FrameSinkManagerClient* client);
   void UnregisterSurfaceFactoryClient(const FrameSinkId& frame_sink_id);
 
   // Associates a |source| with a particular framesink.  That framesink and
@@ -103,7 +103,7 @@ class CC_SURFACES_EXPORT FrameSinkManager {
     std::vector<FrameSinkId> children;
   };
 
-  std::unordered_map<FrameSinkId, SurfaceFactoryClient*, FrameSinkIdHash>
+  std::unordered_map<FrameSinkId, FrameSinkManagerClient*, FrameSinkIdHash>
       clients_;
 
   std::unordered_map<FrameSinkId, FrameSinkSourceMapping, FrameSinkIdHash>