Remove SurfaceFactory And SurfaceFactoryClient

cc/surfaces/frame_sink_manager.cc

Index: cc/surfaces/frame_sink_manager.cc
diff --git a/cc/surfaces/frame_sink_manager.cc b/cc/surfaces/frame_sink_manager.cc
index 7e9095c1b72c1297a75f0451b2d4568543c12c4f..14649dc4879f5158cd679d6d843b320d779dea6e 100644
--- a/cc/surfaces/frame_sink_manager.cc
+++ b/cc/surfaces/frame_sink_manager.cc
@@ -9,7 +9,6 @@
 
 #include "base/logging.h"
 #include "cc/surfaces/frame_sink_manager_client.h"
-#include "cc/surfaces/surface_factory_client.h"
 
 #if DCHECK_IS_ON()
 #include <sstream>
@@ -28,8 +27,8 @@ FrameSinkManager::FrameSinkSourceMapping::~FrameSinkSourceMapping() {}
 FrameSinkManager::FrameSinkManager() {}
 
 FrameSinkManager::~FrameSinkManager() {
-  // All surface factory clients should be unregistered prior to SurfaceManager
-  // destruction.
+  // All CompositorFrameSinks should be unregistered prior to
+  // SurfaceManager destruction.
   DCHECK_EQ(clients_.size(), 0u);
   DCHECK_EQ(registered_sources_.size(), 0u);
 }
@@ -127,8 +126,9 @@ void FrameSinkManager::RecursivelyDetachBeginFrameSource(
   if (iter->second.source == source) {
     iter->second.source = nullptr;
     auto client_iter = clients_.find(frame_sink_id);
-    if (client_iter != clients_.end())
+    if (client_iter != clients_.end()) {

[danakj, 2017/05/03 16:08:39]

unrelated diff

[Alex Z., 2017/05/03 18:07:37]

Done.

       client_iter->second->SetBeginFrameSource(nullptr);
+    }
   }
 
   if (!iter->second.has_children() && !clients_.count(frame_sink_id)) {
@@ -207,7 +207,7 @@ void FrameSinkManager::UnregisterFrameSinkHierarchy(
   }
   DCHECK(found_child);
 
-  // The SurfaceFactoryClient and hierarchy can be registered/unregistered
+  // The CompositorFrameSinkSupport and hierarchy can be registered/unregistered
   // in either order, so empty frame_sink_source_map entries need to be
   // checked when removing either clients or relationships.
   if (!iter->second.has_children() && !clients_.count(parent_frame_sink_id) &&