Add AudioDirectiveHandler timed tests.

Add AudioDirectiveHandler timed tests.

This CL adds tests for AudioDirectiveHandler which move time forward to verify
functionality. It also adds a ref counted tick clock proxy that can be used to
pass around a tick clock among multiple classes.

BUG=None.
R=xiyuan@chromium.org

Committed: https://crrev.com/48d872c84c0b1a6d74b3fd0fa6e04673d787d247
Cr-Commit-Position: refs/heads/master@{#301400}

[rkc, 2014-10-21 18:59:22]

rkc@chromium.org changed reviewers:
+ xiyuan@chromium.org

[rkc, 2014-10-21 19:00:36]

Added xiyuan@ as a reviewer. I'll land this CL at least while
https://codereview.chromium.org/670623002/ goes through discussions. It should
be really easy to swap out the tick clocks later if we do go with a ref counted
tick clock instead.

[xiyuan, 2014-10-21 21:08:13]

lgtm

https://codereview.chromium.org/665353002/diff/40001/components/copresence/ha...
File components/copresence/handlers/audio/tick_clock_ref_counted.h (right):

https://codereview.chromium.org/665353002/diff/40001/components/copresence/ha...
components/copresence/handlers/audio/tick_clock_ref_counted.h:9: #include
<vector>
nit: both headers seems not used.

https://codereview.chromium.org/665353002/diff/40001/components/copresence/ha...
components/copresence/handlers/audio/tick_clock_ref_counted.h:11: #include
"base/callback.h"
nit: not used?

https://codereview.chromium.org/665353002/diff/40001/components/copresence/ha...
components/copresence/handlers/audio/tick_clock_ref_counted.h:23: class
BASE_EXPORT TickClockRefCounted
Is BASE_EXPORT intentional here?

https://codereview.chromium.org/665353002/diff/40001/components/copresence/ha...
components/copresence/handlers/audio/tick_clock_ref_counted.h:26: explicit
TickClockRefCounted(base::TickClock* clock);
nit: Document ownership or use scoped_ptr<>

https://codereview.chromium.org/665353002/diff/40001/components/copresence/ha...
components/copresence/handlers/audio/tick_clock_ref_counted.h:34: };
nit: DISALLOW_COPY_AND_ASSIGN?

[rkc, 2014-10-22 00:16:58]

https://codereview.chromium.org/665353002/diff/40001/components/copresence/ha...
File components/copresence/handlers/audio/tick_clock_ref_counted.h (right):

https://codereview.chromium.org/665353002/diff/40001/components/copresence/ha...
components/copresence/handlers/audio/tick_clock_ref_counted.h:9: #include
<vector>
On 2014/10/21 21:08:12, xiyuan wrote:
> nit: both headers seems not used.

Done.

https://codereview.chromium.org/665353002/diff/40001/components/copresence/ha...
components/copresence/handlers/audio/tick_clock_ref_counted.h:11: #include
"base/callback.h"
On 2014/10/21 21:08:12, xiyuan wrote:
> nit: not used?

Done.

https://codereview.chromium.org/665353002/diff/40001/components/copresence/ha...
components/copresence/handlers/audio/tick_clock_ref_counted.h:23: class
BASE_EXPORT TickClockRefCounted
On 2014/10/21 21:08:12, xiyuan wrote:
> Is BASE_EXPORT intentional here?

Leftover from copying this code from base/timer :)
Removed.
Done.

https://codereview.chromium.org/665353002/diff/40001/components/copresence/ha...
components/copresence/handlers/audio/tick_clock_ref_counted.h:26: explicit
TickClockRefCounted(base::TickClock* clock);
On 2014/10/21 21:08:12, xiyuan wrote:
> nit: Document ownership or use scoped_ptr<>

Makes the test code a bit uglier :)
Done.

https://codereview.chromium.org/665353002/diff/40001/components/copresence/ha...
components/copresence/handlers/audio/tick_clock_ref_counted.h:34: };
On 2014/10/21 21:08:12, xiyuan wrote:
> nit: DISALLOW_COPY_AND_ASSIGN?

Done.

[rkc, 2014-10-23 04:38:42]

The CQ bit was checked by rkc@chromium.org

[commit-bot: I haz the power, 2014-10-23 04:39:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/665353002/80001

[commit-bot: I haz the power, 2014-10-23 04:43:04]

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

[commit-bot: I haz the power, 2014-10-23 04:43:05]

Try jobs failed on following builders:
  linux_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/linux_gpu/builds/...)
  mac_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/mac_gpu/builds/72393)
  win_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu/builds/77323)
  android_aosp on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_aosp/bu...)
  android_arm64_dbg_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_arm64_d...)
  android_chromium_gn_compile_dbg on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_chromiu...)
  android_chromium_gn_compile_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_chromiu...)
  linux_chromium_asan_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_gn_dbg on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_gn_rel on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  win_chromium_compile_dbg on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)
  win_chromium_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel/...)
  win_chromium_x64_rel on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[rkc, 2014-10-27 15:18:28]

The CQ bit was checked by rkc@chromium.org

[commit-bot: I haz the power, 2014-10-27 15:19:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/665353002/100001

[commit-bot: I haz the power, 2014-10-27 15:53:30]

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

[commit-bot: I haz the power, 2014-10-27 15:53:32]

Try jobs failed on following builders:
  win_gpu on tryserver.chromium.gpu
(http://build.chromium.org/p/tryserver.chromium.gpu/builders/win_gpu/builds/78895)

[rkc, 2014-10-27 17:26:01]

The CQ bit was checked by rkc@chromium.org

[commit-bot: I haz the power, 2014-10-27 17:26:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/665353002/120001

[commit-bot: I haz the power, 2014-10-27 18:12:43]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2014-10-27 18:13:30]

Patchset 7 (id:??) landed as
https://crrev.com/48d872c84c0b1a6d74b3fd0fa6e04673d787d247
Cr-Commit-Position: refs/heads/master@{#301400}