Adding DebugDumpReplayer.

Adding DebugDumpReplayer.

It would be good to have a dedicated DebugDumpReplayer. There is one but it hides itself in DebugDumpTest.

This CL is to separate it out.

BUG=

Committed: https://crrev.com/0de1c1374c4dba65ade1ec96f839dbd5bc620e3e
Cr-Commit-Position: refs/heads/master@{#12029}

[minyue-webrtc, 2016-03-16 09:41:52]

Description was changed from

==========
step 1 copy debug_dump_test.cc

BUG=
==========

to

==========
Adding DebugDumpReplayer.

It would be good to have a dedicated DebugDumpReplayer. There is one but it
hides itself in DebugDumpTest.

This CL is to separate it out.

BUG=
==========

[minyue-webrtc, 2016-03-16 09:41:52]

minyue@webrtc.org changed reviewers:
+ henrik.lundin@webrtc.org

[minyue-webrtc, 2016-03-16 09:44:02]

Hi Henrik,

Would you review this CL? Since debug_dump_replayer.h/cc is separated from
debug_dump_test.cc, Patch set 1 is just to copy debug_dump_test.cc to
debug_dump_replayer.cc, and Patch set 2 modifies debug_dump_replayer.h/cc. I
think this will make review simpler.

Thank you.

[hlundin-webrtc, 2016-03-16 13:30:07]

Nice! LGTM.

https://codereview.webrtc.org/1810463002/diff/20001/webrtc/modules/audio_proc...
File webrtc/modules/audio_processing/test/debug_dump_replayer.cc (right):

https://codereview.webrtc.org/1810463002/diff/20001/webrtc/modules/audio_proc...
webrtc/modules/audio_processing/test/debug_dump_replayer.cc:78: // We do not
expect receive UNKNOWN event.
expect to receive

[minyue-webrtc, 2016-03-16 14:55:24]

The CQ bit was checked by minyue@webrtc.org

[minyue-webrtc, 2016-03-16 14:55:24]

The patchset sent to the CQ was uploaded after l-g-t-m from
henrik.lundin@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1810463002/#ps40001 (title:
"a nit")

[commit-bot: I haz the power, 2016-03-16 14:55:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1810463002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1810463002/40001

[commit-bot: I haz the power, 2016-03-16 16:56:09]

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

[commit-bot: I haz the power, 2016-03-16 16:56:09]

Try jobs failed on following builders:
  android_arm64_rel on tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_baremetal on tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[minyue-webrtc, 2016-03-17 06:29:32]

The CQ bit was checked by minyue@webrtc.org

[commit-bot: I haz the power, 2016-03-17 06:29:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1810463002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1810463002/40001

[commit-bot: I haz the power, 2016-03-17 07:20:35]

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

[commit-bot: I haz the power, 2016-03-17 07:20:36]

Try jobs failed on following builders:
  win_baremetal on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/win_baremetal/builds/9818)

[minyue-webrtc, 2016-03-17 08:36:11]

The CQ bit was checked by minyue@webrtc.org

[commit-bot: I haz the power, 2016-03-17 08:36:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1810463002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1810463002/40001

[commit-bot: I haz the power, 2016-03-17 09:39:33]

Description was changed from

==========
Adding DebugDumpReplayer.

It would be good to have a dedicated DebugDumpReplayer. There is one but it
hides itself in DebugDumpTest.

This CL is to separate it out.

BUG=
==========

to

==========
Adding DebugDumpReplayer.

It would be good to have a dedicated DebugDumpReplayer. There is one but it
hides itself in DebugDumpTest.

This CL is to separate it out.

BUG=
==========

[commit-bot: I haz the power, 2016-03-17 09:39:34]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-17 09:39:45]

Description was changed from

==========
Adding DebugDumpReplayer.

It would be good to have a dedicated DebugDumpReplayer. There is one but it
hides itself in DebugDumpTest.

This CL is to separate it out.

BUG=
==========

to

==========
Adding DebugDumpReplayer.

It would be good to have a dedicated DebugDumpReplayer. There is one but it
hides itself in DebugDumpTest.

This CL is to separate it out.

BUG=

Committed: https://crrev.com/0de1c1374c4dba65ade1ec96f839dbd5bc620e3e
Cr-Commit-Position: refs/heads/master@{#12029}
==========

[commit-bot: I haz the power, 2016-03-17 09:39:46]

Patchset 3 (id:??) landed as
https://crrev.com/0de1c1374c4dba65ade1ec96f839dbd5bc620e3e
Cr-Commit-Position: refs/heads/master@{#12029}