Remove SurfaceFactory And SurfaceFactoryClient

cc/surfaces/surface_resource_holder.cc

Index: cc/surfaces/surface_resource_holder.cc
diff --git a/cc/surfaces/surface_resource_holder.cc b/cc/surfaces/surface_resource_holder.cc
index dff955cbde28f294bf2c3cb0c5410366339e4b48..8499400251b0517ec4c32e5d2d0f01eb3c2c5ffd 100644
--- a/cc/surfaces/surface_resource_holder.cc
+++ b/cc/surfaces/surface_resource_holder.cc
@@ -6,17 +6,14 @@
 
 #include "cc/surfaces/surface_resource_holder_client.h"
 namespace cc {
-
 SurfaceResourceHolder::SurfaceResourceHolder(
     SurfaceResourceHolderClient* client)
     : client_(client) {}
 
-SurfaceResourceHolder::~SurfaceResourceHolder() {
-}
+SurfaceResourceHolder::~SurfaceResourceHolder() {}
 
 SurfaceResourceHolder::ResourceRefs::ResourceRefs()
-    : refs_received_from_child(0), refs_holding_resource_alive(0) {
-}
+    : refs_received_from_child(0), refs_holding_resource_alive(0) {}
 
 void SurfaceResourceHolder::Reset() {
   resource_id_info_map_.clear();
@@ -25,8 +22,7 @@ void SurfaceResourceHolder::Reset() {
 void SurfaceResourceHolder::ReceiveFromChild(
     const TransferableResourceArray& resources) {
   for (TransferableResourceArray::const_iterator it = resources.begin();

[Fady Samuel, 2017/04/26 00:44:14]

for (const auto& resource : resources) {
  ...
}

[Alex Z., 2017/04/27 22:10:01]

const auto& doesn't work since we have ++it. I changed it to 
auto.

-       it != resources.end();
-       ++it) {
+       it != resources.end(); ++it) {
     ResourceRefs& ref = resource_id_info_map_[it->id];
     ref.refs_holding_resource_alive++;
     ref.refs_received_from_child++;