Provision to start base::SimpleTestTickClock at initial ticks

base/test/simple_test_tick_clock.h

 // Copyright (c) 2012 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 BASE_TEST_SIMPLE_TEST_TICK_CLOCK_H_
 #define BASE_TEST_SIMPLE_TEST_TICK_CLOCK_H_
 
 #include "base/compiler_specific.h"
 #include "base/synchronization/lock.h"
 #include "base/time/tick_clock.h"
 #include "base/time/time.h"
 
 namespace base {
 
 // SimpleTestTickClock is a TickClock implementation that gives
 // control over the returned TimeTicks objects.  All methods may be
 // called from any thread.
 class SimpleTestTickClock : public TickClock {
  public:
-  // Starts off with a clock set to TimeTicks().
-  SimpleTestTickClock();
+  // Start off with a clock set to some initial tick(s)
+  // Avoids having to Advance(initial) for tests
+  SimpleTestTickClock(TimeTicks initial = TimeTicks());

[Paweł Hajdan Jr., 2015/06/30 11:40:19]

Default arguments are prohibited, please see
https://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Default_Argu...
.

[Ankur Verma, 2015/06/30 11:58:39]

Quoting from the above documentation:
"In addition, default function parameters are allowed in constructors. Most of
the cons listed above don't apply to constructors because it's impossible to
take their address."

[Paweł Hajdan Jr., 2015/07/15 17:36:59]

Ah, you're right, I didn't notice this. Sorry.

   ~SimpleTestTickClock() override;
 
   TimeTicks NowTicks() override;
 
   // Advances the clock by |delta|, which must not be negative.
   void Advance(TimeDelta delta);
 
  private:
   // Protects |now_ticks_|.
   Lock lock_;
 
   TimeTicks now_ticks_;
 };
 
 }  // namespace base
 
 #endif  // BASE_TEST_SIMPLE_TEST_TICK_CLOCK_H_