cc: Making BeginFrameSources support multiple observers.

cc/scheduler/begin_frame_source_unittest.cc

 // Copyright 2011 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 <deque>
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/gtest_prod_util.h"
 #include "base/test/test_simple_task_runner.h"
@@ skipping 213 matching lines @@
 // BeginFrameSource testing ----------------------------------------------
 TEST(BeginFrameSourceBaseTest, ObserverManipulation) {
   MockBeginFrameObserver obs;
   MockBeginFrameObserver otherObs;
   FakeBeginFrameSource source;
 
   source.AddObserver(&obs);
   EXPECT_EQ(&obs, source.GetObserver());
 
 #ifndef NDEBUG
-  // Adding an observer when an observer already exists should DCHECK fail.
-  EXPECT_DEATH({ source.AddObserver(&otherObs); }, "");
-
   // Removing wrong observer should DCHECK fail.
   EXPECT_DEATH({ source.RemoveObserver(&otherObs); }, "");
 
   // Removing an observer when there is no observer should DCHECK fail.
   EXPECT_DEATH({
                  source.RemoveObserver(&obs);
                  source.RemoveObserver(&obs);
                },
                "");
 #endif
   source.RemoveObserver(&obs);
 
   source.AddObserver(&otherObs);
   EXPECT_EQ(&otherObs, source.GetObserver());
   source.RemoveObserver(&otherObs);
+
+  // TODO(mithro): Add a test here for adding / removing multiple observers.
+
 }
 
 TEST(BeginFrameSourceBaseTest, Observer) {
   FakeBeginFrameSource source;
   MockBeginFrameObserver obs;
   source.AddObserver(&obs);
   EXPECT_BEGIN_FRAME_USED(obs, 100, 200, 300);
   EXPECT_BEGIN_FRAME_DROP(obs, 400, 600, 300);
   EXPECT_BEGIN_FRAME_DROP(obs, 450, 650, 300);
   EXPECT_BEGIN_FRAME_USED(obs, 700, 900, 300);
@@ skipping 508 matching lines @@
   mux_->SetActiveSource(source2_);
   SEND_BEGIN_FRAME_DROP(*source2_, 750, 1050, 300);
   SEND_BEGIN_FRAME_USED(*source2_, 1050, 1250, 300);
 
   mux_->SetActiveSource(source1_);
   SEND_BEGIN_FRAME_DROP(*source2_, 1100, 1400, 300);
 }
 
 }  // namespace
 }  // namespace cc