Cleanup RateCounter

Cleanup RateCounter

Previously RateCounter was using base::Time instead of base::TimeTicks,
which may cause it to generate incorrect values in some cases. Fxed it
to use TimeTicks. Also updated it to use TickClock interface for tests.

Committed: https://crrev.com/fcfab00938f82a3830a9cf64eac4a5436f456250
Cr-Commit-Position: refs/heads/master@{#437712}

[Sergey Ulanov, 2016-12-09 22:00:15]

sergeyu@chromium.org changed reviewers:
+ lambroslambrou@chromium.org

[Jamie, 2016-12-09 23:03:36]

jamiewalch@chromium.org changed reviewers:
+ jamiewalch@chromium.org

[Jamie, 2016-12-09 23:03:36]

lgtm

[Sergey Ulanov, 2016-12-09 23:16:01]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2016-12-09 23:16:43]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-10 01:19:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-10 01:19:34]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)

[Sergey Ulanov, 2016-12-10 01:25:51]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2016-12-10 01:26:41]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-10 01:46:40]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481333151109530, "parent_rev":
"5df90cbbc817d86fc89cdaa68f3859d85a0e0360", "commit_rev":
"3cd6654d79d3a0e4760a9e3a5a8502e11c308622"}

[commit-bot: I haz the power, 2016-12-10 01:47:08]

Description was changed from

==========
Cleanup RateCounter

Previously RateCounter was using base::Time instead of base::TimeTicks,
which may cause it to generate incorrect values in some cases. Fxed it
to use TimeTicks. Also updated it to use TickClock interface for tests.
==========

to

==========
Cleanup RateCounter

Previously RateCounter was using base::Time instead of base::TimeTicks,
which may cause it to generate incorrect values in some cases. Fxed it
to use TimeTicks. Also updated it to use TickClock interface for tests.

Review-Url: https://codereview.chromium.org/2561173004
==========

[commit-bot: I haz the power, 2016-12-10 01:47:09]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-12 14:59:13]

Description was changed from

==========
Cleanup RateCounter

Previously RateCounter was using base::Time instead of base::TimeTicks,
which may cause it to generate incorrect values in some cases. Fxed it
to use TimeTicks. Also updated it to use TickClock interface for tests.

Review-Url: https://codereview.chromium.org/2561173004
==========

to

==========
Cleanup RateCounter

Previously RateCounter was using base::Time instead of base::TimeTicks,
which may cause it to generate incorrect values in some cases. Fxed it
to use TimeTicks. Also updated it to use TickClock interface for tests.

Committed: https://crrev.com/fcfab00938f82a3830a9cf64eac4a5436f456250
Cr-Commit-Position: refs/heads/master@{#437712}
==========

[commit-bot: I haz the power, 2016-12-12 14:59:14]

Patchset 1 (id:??) landed as
https://crrev.com/fcfab00938f82a3830a9cf64eac4a5436f456250
Cr-Commit-Position: refs/heads/master@{#437712}