Moved struct definition to the header file for the ring buffer.

Moved struct definition to the header file for the ring buffer.

This is done in order to allow the ringbuffer to be recorded using protobufs.
The actual recording will be added in other CLs.

BUG=webrtc:5724
Committed: https://crrev.com/4d23447b1548fa31e54f92754661af0740008b3d
Cr-Commit-Position: refs/heads/master@{#12308}

[peah-webrtc, 2016-04-06 12:13:33]

Patchset #1 (id:1) has been deleted

[peah-webrtc, 2016-04-06 12:13:57]

peah@webrtc.org changed reviewers:
+ ivoc@webrtc.org

[ivoc, 2016-04-06 12:15:32]

LGTM :)

[peah-webrtc, 2016-04-08 21:31:23]

The CQ bit was checked by peah@webrtc.org

[commit-bot: I haz the power, 2016-04-08 21:31:34]

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

[commit-bot: I haz the power, 2016-04-08 21:41:27]

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

[commit-bot: I haz the power, 2016-04-08 21:41:28]

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

[peah-webrtc, 2016-04-09 21:04:51]

peah@webrtc.org changed reviewers:
+ kwiberg@webrtc.org

[peah-webrtc, 2016-04-09 21:04:51]

+kwiberg as being an OWNER of the ring_buffer

[kwiberg-webrtc, 2016-04-09 22:36:34]

lgtm

I don't really like exposing the internals of a data structure like this, but
RingBuffer is a disaster already, so I guess it's OK. But when the time comes to
replace it, the replacement should probably have a separately defined
serialization format instead of reusing the implementation struct.

[peah-webrtc, 2016-04-11 04:42:14]

The CQ bit was checked by peah@webrtc.org

[peah-webrtc, 2016-04-11 04:42:15]

The patchset sent to the CQ was uploaded after l-g-t-m from ivoc@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1862513007/#ps40001 (title:
"Rebase")

[commit-bot: I haz the power, 2016-04-11 04:42:21]

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

[commit-bot: I haz the power, 2016-04-11 05:51:12]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-04-11 05:51:18]

Description was changed from

==========
Moved struct definition to the header file for the ring buffer.

This is done in order to allow the ringbuffer to be recorded using protobufs.
The actual recording will be added in other CLs.

BUG=webrtc:5724
==========

to

==========
Moved struct definition to the header file for the ring buffer.

This is done in order to allow the ringbuffer to be recorded using protobufs.
The actual recording will be added in other CLs.

BUG=webrtc:5724

Committed: https://crrev.com/4d23447b1548fa31e54f92754661af0740008b3d
Cr-Commit-Position: refs/heads/master@{#12308}
==========

[commit-bot: I haz the power, 2016-04-11 05:51:19]

Patchset 2 (id:??) landed as
https://crrev.com/4d23447b1548fa31e54f92754661af0740008b3d
Cr-Commit-Position: refs/heads/master@{#12308}