Add WriteUVarint to ByteBufferWriter and ReadUVarint to ByteBufferReader

Add WriteUVarint to ByteBufferWriter and ReadUVarint to ByteBufferReader

Methods to write/read a varint as described by
https://developers.google.com/protocol-buffers/docs/encoding#varints
that will be used for a QUIC data channel.

Split from CL https://codereview.webrtc.org/1844803002/.

Committed: https://crrev.com/9a20fa6292e3eb6ecf30aa49124ce837ff052b55
Cr-Commit-Position: refs/heads/master@{#12322}

[mikescarlett, 2016-04-01 23:08:51]

Description was changed from

==========
Add WriteUVarint to ByteBufferWriter and ReadUVarint to ByteBufferReader

Methods to write/read a varint as described in
https://developers.google.com/protocol-buffers/docs/encoding#varints.

Split from CL https://codereview.webrtc.org/1844803002/.

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

to

==========
Add WriteUVarint to ByteBufferWriter and ReadUVarint to ByteBufferReader

Methods to write/read a varint as described by
https://developers.google.com/protocol-buffers/docs/encoding#varints
that will be used for a QUIC data channel.

Split from CL https://codereview.webrtc.org/1844803002/.

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

[mikescarlett, 2016-04-02 00:14:38]

mikescarlett@webrtc.org changed reviewers:
+ pthatcher@webrtc.org

[mikescarlett, 2016-04-02 02:01:04]

The CQ bit was checked by mikescarlett@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2016-04-02 02:01:08]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1844333006/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1844333006/1

[mikescarlett, 2016-04-02 02:01:11]

The CQ bit was unchecked by mikescarlett@webrtc.org

[mikescarlett, 2016-04-04 18:28:08]

mikescarlett@webrtc.org changed reviewers:
+ deadbeef@webrtc.org

[Taylor Brandstetter, 2016-04-05 20:25:42]

Description was changed from

==========
Add WriteUVarint to ByteBufferWriter and ReadUVarint to ByteBufferReader

Methods to write/read a varint as described by
https://developers.google.com/protocol-buffers/docs/encoding#varints
that will be used for a QUIC data channel.

Split from CL https://codereview.webrtc.org/1844803002/.

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

to

==========
Add WriteUVarint to ByteBufferWriter and ReadUVarint to ByteBufferReader

Methods to write/read a varint as described by
https://developers.google.com/protocol-buffers/docs/encoding#varints
that will be used for a QUIC data channel.

Split from CL https://codereview.webrtc.org/1844803002/.
==========

[Taylor Brandstetter, 2016-04-05 20:28:29]

lgtm; just noticed a minor typo.

https://codereview.webrtc.org/1844333006/diff/1/webrtc/base/bytebuffer.cc
File webrtc/base/bytebuffer.cc (right):

https://codereview.webrtc.org/1844333006/diff/1/webrtc/base/bytebuffer.cc#new...
webrtc/base/bytebuffer.cc:96: // Write 7 bits at a time, then set the msb to a
contination byte (msb=1).
"continuation"

[mikescarlett, 2016-04-06 17:21:42]

https://codereview.webrtc.org/1844333006/diff/1/webrtc/base/bytebuffer.cc
File webrtc/base/bytebuffer.cc (right):

https://codereview.webrtc.org/1844333006/diff/1/webrtc/base/bytebuffer.cc#new...
webrtc/base/bytebuffer.cc:96: // Write 7 bits at a time, then set the msb to a
contination byte (msb=1).
On 2016/04/05 20:28:29, Taylor Brandstetter wrote:
> "continuation"

Done.

[pthatcher1, 2016-04-11 21:41:25]

lgtm

[mikescarlett, 2016-04-11 22:04:16]

The CQ bit was checked by mikescarlett@webrtc.org

[mikescarlett, 2016-04-11 22:04:17]

The patchset sent to the CQ was uploaded after l-g-t-m from deadbeef@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1844333006/#ps20001 (title:
"Fix typo")

[commit-bot: I haz the power, 2016-04-11 22:04:23]

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

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

Description was changed from

==========
Add WriteUVarint to ByteBufferWriter and ReadUVarint to ByteBufferReader

Methods to write/read a varint as described by
https://developers.google.com/protocol-buffers/docs/encoding#varints
that will be used for a QUIC data channel.

Split from CL https://codereview.webrtc.org/1844803002/.
==========

to

==========
Add WriteUVarint to ByteBufferWriter and ReadUVarint to ByteBufferReader

Methods to write/read a varint as described by
https://developers.google.com/protocol-buffers/docs/encoding#varints
that will be used for a QUIC data channel.

Split from CL https://codereview.webrtc.org/1844803002/.
==========

[commit-bot: I haz the power, 2016-04-11 23:11:43]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-11 23:11:52]

Description was changed from

==========
Add WriteUVarint to ByteBufferWriter and ReadUVarint to ByteBufferReader

Methods to write/read a varint as described by
https://developers.google.com/protocol-buffers/docs/encoding#varints
that will be used for a QUIC data channel.

Split from CL https://codereview.webrtc.org/1844803002/.
==========

to

==========
Add WriteUVarint to ByteBufferWriter and ReadUVarint to ByteBufferReader

Methods to write/read a varint as described by
https://developers.google.com/protocol-buffers/docs/encoding#varints
that will be used for a QUIC data channel.

Split from CL https://codereview.webrtc.org/1844803002/.

Committed: https://crrev.com/9a20fa6292e3eb6ecf30aa49124ce837ff052b55
Cr-Commit-Position: refs/heads/master@{#12322}
==========

[commit-bot: I haz the power, 2016-04-11 23:11:53]

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