[counters] RuntimeStats: fix wrong bookkeeping when dynamically changing counters.

src/base/platform/elapsed-timer.h

 // Copyright 2013 the V8 project 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 V8_BASE_PLATFORM_ELAPSED_TIMER_H_
 #define V8_BASE_PLATFORM_ELAPSED_TIMER_H_
 
 #include "src/base/logging.h"
 #include "src/base/platform/time.h"
 
@@ skipping 34 matching lines @@
     DCHECK(started_ || start_ticks_.IsNull());
     DCHECK(!started_ || !start_ticks_.IsNull());
     return !start_ticks_.IsNull();
   }
 
   // Restarts the timer and returns the time elapsed since the previous start.
   // This method is equivalent to obtaining the elapsed time with |Elapsed()|
   // and then starting the timer again, but does so in one single operation,
   // avoiding the need to obtain the clock value twice. It may only be called
   // on a previously started timer.
-  TimeDelta Restart() {
+  TimeDelta Restart() { return Restart(Now()); }
+
+  TimeDelta Restart(TimeTicks ticks) {

[Igor Sheludko, 2016/11/18 18:56:27]

s/ticks/now/ to be clear.

[Camillo Bruni, 2016/11/21 10:50:03]

done.

     DCHECK(IsStarted());
-    TimeTicks ticks = Now();
     TimeDelta elapsed = ticks - start_ticks_;
     DCHECK(elapsed.InMicroseconds() >= 0);
     start_ticks_ = ticks;
     DCHECK(IsStarted());
     return elapsed;
   }
 
   // Returns the time elapsed since the previous start. This method may only
   // be called on a previously started timer.
   TimeDelta Elapsed() const {
     DCHECK(IsStarted());
     TimeDelta elapsed = Now() - start_ticks_;
     DCHECK(elapsed.InMicroseconds() >= 0);
     return elapsed;
   }
 
+  // Move the start_ticks_ to adjust the current timer.
+  void Subtract(TimeDelta delta) {
+    DCHECK(IsStarted());
+    // If the delta is negative we want to make the elapsed time smaller, hence
+    // we have to move the start_ticks_ in the opposite direction.
+    start_ticks_ += delta;
+  }
+
   // Returns |true| if the specified |time_delta| has elapsed since the
   // previous start, or |false| if not. This method may only be called on
   // a previously started timer.
   bool HasExpired(TimeDelta time_delta) const {
     DCHECK(IsStarted());
     return Elapsed() >= time_delta;
   }
 
  private:
   static V8_INLINE TimeTicks Now() {
     TimeTicks now = TimeTicks::HighResolutionNow();
     DCHECK(!now.IsNull());
     return now;
   }
 
   TimeTicks start_ticks_;
 #ifdef DEBUG
   bool started_;
 #endif
 };
 
 }  // namespace base
 }  // namespace v8
 
 #endif  // V8_BASE_PLATFORM_ELAPSED_TIMER_H_