Replaced TestNowSource with SimpleTestTickClock.

components/scheduler/child/test_time_source.h

 // Copyright 2015 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.
 
 #ifndef COMPONENTS_SCHEDULER_CHILD_TEST_TIME_SOURCE_H_
 #define COMPONENTS_SCHEDULER_CHILD_TEST_TIME_SOURCE_H_
 
 #include "base/memory/ref_counted.h"
+#include "base/test/simple_test_tick_clock.h"
 #include "components/scheduler/child/time_source.h"
 
-namespace cc {
-class TestNowSource;
-}
-
 namespace scheduler {
 
 class TestTimeSource : public TimeSource {
  public:
-  explicit TestTimeSource(scoped_refptr<cc::TestNowSource> time_source);
+  explicit TestTimeSource(base::SimpleTestTickClock* time_source);
   ~TestTimeSource() override;
 
   base::TimeTicks Now() const override;
 
  private:
-  scoped_refptr<cc::TestNowSource> time_source_;
+  base::SimpleTestTickClock* time_source_;

[Sami, 2015/06/04 14:47:07]

Can you split this change out since you had another patch for it?

[Ankur Verma, 2015/06/04 15:25:08]

I could not understand completely (this being my first chromium bug).

This change bring us back to the original patch.
1. First, i had deleted test_now_source.* files and converted this to use
SimpleTestTickClock.
2. Then, i had added the num_now_calls_ functionality to it.
3. Then, i removed the num_now_calls_functionality from it. Hence back to 1.

Would you like me to restore test_now_source.* files and remove this change. And
then remove test_now_source.* in a separate patch?

[Ankur Verma, 2015/06/05 14:39:16]

On rebasing for patch set 8, i realised that
https://codereview.chromium.org/1163143002
performed a separate cleanup on this.

I've now replaced cc::TestNowSource with base::SimpleTestTickClock on top of
that.

 
   DISALLOW_COPY_AND_ASSIGN(TestTimeSource);
 };
 
 }  // namespace scheduler
 
 #endif  // COMPONENTS_SCHEDULER_CHILD_TEST_TIME_SOURCE_H_