Make disable vsync run the renderer independently

cc/surfaces/surface_manager_unittest.cc

 // Copyright 2016 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 <stddef.h>
 
 #include "cc/scheduler/begin_frame_source.h"
 #include "cc/surfaces/surface_factory_client.h"
 #include "cc/surfaces/surface_manager.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 38 matching lines @@
     DCHECK(!source_ || !begin_frame_source);
     source_ = begin_frame_source;
   };
 
  private:
   BeginFrameSource* source_;
   SurfaceManager* manager_;
   uint32_t id_namespace_;
 };
 
-class EmptyBeginFrameSource : public BeginFrameSource {
- public:
-  void DidFinishFrame(BeginFrameObserver* obs,
-                      size_t remaining_frames) override {}
-  void AddObserver(BeginFrameObserver* obs) override {}
-  void RemoveObserver(BeginFrameObserver* obs) override {}
-};
-
 class SurfaceManagerTest : public testing::Test {
  public:
   // These tests don't care about namespace registration, so just preregister
   // a set of namespaces that tests can use freely without worrying if they're
   // valid or not.
   enum { MAX_NAMESPACE = 10 };
 
   SurfaceManagerTest() {
     for (size_t i = 0; i < MAX_NAMESPACE; ++i)
       manager_.RegisterSurfaceClientId(i);
   }
 
   ~SurfaceManagerTest() override {
     for (size_t i = 0; i < MAX_NAMESPACE; ++i)
       manager_.InvalidateSurfaceClientId(i);
   }
 
  protected:
   SurfaceManager manager_;
 };
 
 TEST_F(SurfaceManagerTest, SingleClients) {
   FakeSurfaceFactoryClient client(1);
   FakeSurfaceFactoryClient other_client(2);
-  EmptyBeginFrameSource source;
+  StubBeginFrameSource source;
 
   EXPECT_EQ(client.source(), nullptr);
   EXPECT_EQ(other_client.source(), nullptr);
   client.Register(&manager_);
   other_client.Register(&manager_);
   EXPECT_EQ(client.source(), nullptr);
   EXPECT_EQ(other_client.source(), nullptr);
 
   // Test setting unsetting BFS
   manager_.RegisterBeginFrameSource(&source, client.id_namespace());
@@ skipping 12 matching lines @@
   EXPECT_EQ(other_client.source(), nullptr);
 
   // Re-set BFS for original
   manager_.RegisterBeginFrameSource(&source, client.id_namespace());
   EXPECT_EQ(client.source(), &source);
   manager_.UnregisterBeginFrameSource(&source);
   EXPECT_EQ(client.source(), nullptr);
 }
 
 TEST_F(SurfaceManagerTest, MultipleDisplays) {
-  EmptyBeginFrameSource root1_source;
-  EmptyBeginFrameSource root2_source;
+  StubBeginFrameSource root1_source;
+  StubBeginFrameSource root2_source;
 
   // root1 -> A -> B
   // root2 -> C
   FakeSurfaceFactoryClient root1(1, &manager_);
   FakeSurfaceFactoryClient root2(2, &manager_);
   FakeSurfaceFactoryClient client_a(3, &manager_);
   FakeSurfaceFactoryClient client_b(4, &manager_);
   FakeSurfaceFactoryClient client_c(5, &manager_);
 
   manager_.RegisterBeginFrameSource(&root1_source, root1.id_namespace());
@@ skipping 148 matching lines @@
       // A valid but not attached to anything.
       EXPECT_EQ(client_a_.source(), &source_);
       EXPECT_EQ(client_b_.source(), nullptr);
       EXPECT_EQ(client_c_.source(), nullptr);
       return;
     }
 
     AssertAllValidBFS();
   }
 
-  EmptyBeginFrameSource source_;
+  StubBeginFrameSource source_;
   // A -> B -> C hierarchy, with A always having the BFS.
   FakeSurfaceFactoryClient client_a_;
   FakeSurfaceFactoryClient client_b_;
   FakeSurfaceFactoryClient client_c_;
 
   bool hierarchy_registered_;
   bool clients_registered_;
   bool bfs_registered_;
 };
 
@@ skipping 67 matching lines @@
 }
 
 INSTANTIATE_TEST_CASE_P(
     SurfaceManagerOrderingParamTestInstantiation,
     SurfaceManagerOrderingParamTest,
     ::testing::Combine(::testing::ValuesIn(kRegisterOrderList),
                        ::testing::ValuesIn(kUnregisterOrderList),
                        ::testing::ValuesIn(kBFSOrderList)));
 
 }  // namespace cc