Remove SurfaceFactory And SurfaceFactoryClient

Now that all usages of SurfaceFacotry are replaced with
CompositorFrameSinkSuppot, we can move all functionality in SurfaceFactory to
CompositorFrameSinkSupport and delete the class.

BUG=707105
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2802023002
Cr-Commit-Position: refs/heads/master@{#469954}

diff --git a/cc/BUILD.gn b/cc/BUILD.gn
index a957fa7..0684f0f 100644
--- a/cc/BUILD.gn
+++ b/cc/BUILD.gn

@@ -592,6 +592,8 @@
     "test/layer_tree_pixel_test.h",
     "test/layer_tree_test.cc",
     "test/layer_tree_test.h",
+    "test/mock_compositor_frame_sink_support_client.cc",
+    "test/mock_compositor_frame_sink_support_client.h",
     "test/mock_helper.h",
     "test/mock_occlusion_tracker.h",
     "test/ordered_simple_task_runner.cc",
@@ -857,11 +859,11 @@
     "surfaces/display_unittest.cc",
     "surfaces/referenced_surface_tracker_unittest.cc",
     "surfaces/surface_aggregator_unittest.cc",
-    "surfaces/surface_factory_unittest.cc",
     "surfaces/surface_hittest_unittest.cc",
     "surfaces/surface_manager_ref_unittest.cc",
     "surfaces/surface_manager_unittest.cc",
     "surfaces/surface_sequence_generator_unittest.cc",
+    "surfaces/surface_synchronization_unittest.cc",
     "surfaces/surface_unittest.cc",
     "surfaces/surfaces_pixeltest.cc",