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;
 
+  int NumNowCalls() const { return num_now_calls_; }
+
  private:
-  scoped_refptr<cc::TestNowSource> time_source_;
+  base::SimpleTestTickClock* time_source_;
+  mutable int num_now_calls_;

[mithro-old, 2015/05/29 08:26:48]

The mutable keyword doesn't do what you probably think it does. However, that
looks like it was a bug in the original code too.

[Ankur Verma, 2015/06/01 05:30:44]

Shall remove this if its not really mutable.

[mithro-old, 2015/06/01 07:32:15]

I don't think you should add the NumNowCalls() functionality to this clock here.
I think that means you will need to do the refactoring mentioned in
https://code.google.com/p/chromium/issues/detail?id=494887

It also looks like this TimeSource interface is a duplicate of TickClock? I've
logged a bug about that too
https://code.google.com/p/chromium/issues/detail?id=494892

[Ankur Verma, 2015/06/01 09:21:40]

Right. I've put in my thoughts in relation to this in an earlier comment. Pls
let me know if you think otherwise.
Yes it seems so. History here: https://codereview.chromium.org/1098033002

[Ankur Verma, 2015/06/03 14:56:10]

Done.

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