cc: Throttle swaps in Scheduler instead of OutputSurface

cc/trees/layer_tree_host_unittest_delegated.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/trees/layer_tree_host.h"
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/location.h"
@@ skipping 222 matching lines @@
         resources_to_return.push_back(resource_in_parent);
     }
 
     if (resources_to_return.empty())
       return;
 
     CompositorFrameAck ack;
     for (size_t i = 0; i < resources_to_return.size(); ++i)
       output_surface()->ReturnResource(resources_to_return[i], &ack);
     host_impl->ReclaimResources(&ack);
-    host_impl->OnSwapBuffersComplete();

[danakj, 2014/04/11 15:12:28]

Can you explain why this is no longer needed in tests?

[brianderson, 2014/04/11 20:49:40]

I think it was a bug that they were needed in the first place.

It was causing either timeouts or DCHECKs in the unit tests with the new code. I
think it had to do with the DidSwapBuffers notifications not being throttled by
is_lost_ before this patch.

   }
 };
 
 class LayerTreeHostDelegatedTestCaseSingleDelegatedLayer
     : public LayerTreeHostDelegatedTest,
       public DelegatedFrameResourceCollectionClient {
  public:
   LayerTreeHostDelegatedTestCaseSingleDelegatedLayer()
       : resource_collection_(new DelegatedFrameResourceCollection),
         available_(false) {
@@ skipping 1387 matching lines @@
         static_cast<FakeDelegatedRendererLayerImpl*>(root_impl->children()[0]);
 
     const ResourceProvider::ResourceIdMap& map =
         host_impl->resource_provider()->GetChildToParentMap(
             delegated_impl->ChildId());
 
     // Receive 999 back from the grandparent.
     CompositorFrameAck ack;
     output_surface()->ReturnResource(map.find(999)->second, &ack);
     host_impl->ReclaimResources(&ack);
-    host_impl->OnSwapBuffersComplete();
   }
 
   virtual void UnusedResourcesAreAvailable() OVERRIDE {
     EXPECT_EQ(3, layer_tree_host()->source_frame_number());
 
     ReturnedResourceArray resources;
 
     // 999 was returned from the grandparent and could be released.
     resource_collection_->TakeUnusedResourcesForChildCompositor(&resources);
     {
@@ skipping 656 matching lines @@
   }
 
   scoped_refptr<DelegatedRendererLayer> delegated_thief_;
 };
 
 SINGLE_AND_MULTI_THREAD_TEST_F(
     LayerTreeHostDelegatedTestRemoveAndChangeResources);
 
 }  // namespace
 }  // namespace cc