SignalPacketReceived should pass packet as a pointer instead of a non-const reference.

SignalPacketReceived should pass packet as a pointer instead of a non-const reference.

See https://google.github.io/styleguide/cppguide.html#Reference_Arguments.
The Bind to ProcessPacket in OnPacketReceived is safe because Bind captures arguments by value.

BUG=webrtc:7013
Review-Url: https://codereview.webrtc.org/2980923002
Cr-Commit-Position: refs/heads/master@{#19028}
Committed: https://chromium.googlesource.com/external/webrtc/+/634977b61107244a24652120b762fdce23eb5cb9

[Zach Stein, 2017-07-13 22:02:28]

Description was changed from

==========
SignalPacketReceived should pass a pointer instead of a non-const reference.

See https://google.github.io/styleguide/cppguide.html#Reference_Arguments.
The Bind to ProcessPacket in OnPacketReceived is safe because Bind captures
arguments by value.

BUG=webrtc:7013
==========

to

==========
SignalPacketReceived should pass packet as a pointer instead of a non-const
reference.

See https://google.github.io/styleguide/cppguide.html#Reference_Arguments.
The Bind to ProcessPacket in OnPacketReceived is safe because Bind captures
arguments by value.

BUG=webrtc:7013
==========

[Zach Stein, 2017-07-13 22:02:28]

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

[Zach Stein, 2017-07-13 22:03:17]

PTAL

[Taylor Brandstetter, 2017-07-13 23:47:31]

lgtm

[Zach Stein, 2017-07-14 18:46:04]

The CQ bit was checked by zstein@webrtc.org

[commit-bot: I haz the power, 2017-07-14 18:46:13]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-07-14 19:30:05]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1500057964382950, "parent_rev":
"e76bd3aa43821ff10f5cef5854836ea782b2464f", "commit_rev":
"634977b61107244a24652120b762fdce23eb5cb9"}

[commit-bot: I haz the power, 2017-07-14 19:30:10]

Description was changed from

==========
SignalPacketReceived should pass packet as a pointer instead of a non-const
reference.

See https://google.github.io/styleguide/cppguide.html#Reference_Arguments.
The Bind to ProcessPacket in OnPacketReceived is safe because Bind captures
arguments by value.

BUG=webrtc:7013
==========

to

==========
SignalPacketReceived should pass packet as a pointer instead of a non-const
reference.

See https://google.github.io/styleguide/cppguide.html#Reference_Arguments.
The Bind to ProcessPacket in OnPacketReceived is safe because Bind captures
arguments by value.

BUG=webrtc:7013

Review-Url: https://codereview.webrtc.org/2980923002
Cr-Commit-Position: refs/heads/master@{#19028}
Committed:
https://chromium.googlesource.com/external/webrtc/+/634977b61107244a24652120b...
==========

[commit-bot: I haz the power, 2017-07-14 19:30:11]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/634977b61107244a24652120b...