Fix WebRTC transport to normalize SDP messages.

Fix WebRTC transport to normalize SDP messages.

WebRTC generates SDP messages that contain CRLF line endings. Depending
on the signaling transport these may be converted to LF, which was
causing signature verification to fail. Now SDP is normalized on both
ends to use LF line endings.

BUG=547158
Committed: https://crrev.com/e7aa37d9324beb59d3d7edd574e13346f1b2283a
Cr-Commit-Position: refs/heads/master@{#384322}

[Sergey Ulanov, 2016-03-30 23:23:49]

sergeyu@chromium.org changed reviewers:
+ kelvinp@chromium.org

[kelvinp, 2016-03-31 01:09:38]

lgtm

https://codereview.chromium.org/1843333002/diff/1/remoting/protocol/webrtc_tr...
File remoting/protocol/webrtc_transport.cc (right):

https://codereview.chromium.org/1843333002/diff/1/remoting/protocol/webrtc_tr...
remoting/protocol/webrtc_transport.cc:51: // verifications is the same that was
signed on the sending side.This is
Nit: missing space after .

[Sergey Ulanov, 2016-03-31 16:52:19]

https://codereview.chromium.org/1843333002/diff/1/remoting/protocol/webrtc_tr...
File remoting/protocol/webrtc_transport.cc (right):

https://codereview.chromium.org/1843333002/diff/1/remoting/protocol/webrtc_tr...
remoting/protocol/webrtc_transport.cc:51: // verifications is the same that was
signed on the sending side.This is
On 2016/03/31 01:09:38, kelvinp wrote:
> Nit: missing space after .

Done.

[Sergey Ulanov, 2016-03-31 16:53:17]

Description was changed from

==========
Fix WebRTC transport to normalize SDP messages.

WebRTC generates SDP messages that contain CRLF line endings. Depending
on the signaling transport these may be converted to LF, which was
causing signature verification to fail. Now SDP is normalized on both
ends to use LF line endings.
==========

to

==========
Fix WebRTC transport to normalize SDP messages.

WebRTC generates SDP messages that contain CRLF line endings. Depending
on the signaling transport these may be converted to LF, which was
causing signature verification to fail. Now SDP is normalized on both
ends to use LF line endings.

BUG=547158
==========

[Sergey Ulanov, 2016-03-31 16:54:33]

The CQ bit was checked by sergeyu@chromium.org

[Sergey Ulanov, 2016-03-31 16:54:33]

The patchset sent to the CQ was uploaded after l-g-t-m from kelvinp@chromium.org
Link to the patchset: https://codereview.chromium.org/1843333002/#ps20001
(title: " ")

[commit-bot: I haz the power, 2016-03-31 16:54:51]

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

[commit-bot: I haz the power, 2016-03-31 17:55:48]

Description was changed from

==========
Fix WebRTC transport to normalize SDP messages.

WebRTC generates SDP messages that contain CRLF line endings. Depending
on the signaling transport these may be converted to LF, which was
causing signature verification to fail. Now SDP is normalized on both
ends to use LF line endings.

BUG=547158
==========

to

==========
Fix WebRTC transport to normalize SDP messages.

WebRTC generates SDP messages that contain CRLF line endings. Depending
on the signaling transport these may be converted to LF, which was
causing signature verification to fail. Now SDP is normalized on both
ends to use LF line endings.

BUG=547158
==========

[commit-bot: I haz the power, 2016-03-31 17:55:49]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-31 17:57:28]

Description was changed from

==========
Fix WebRTC transport to normalize SDP messages.

WebRTC generates SDP messages that contain CRLF line endings. Depending
on the signaling transport these may be converted to LF, which was
causing signature verification to fail. Now SDP is normalized on both
ends to use LF line endings.

BUG=547158
==========

to

==========
Fix WebRTC transport to normalize SDP messages.

WebRTC generates SDP messages that contain CRLF line endings. Depending
on the signaling transport these may be converted to LF, which was
causing signature verification to fail. Now SDP is normalized on both
ends to use LF line endings.

BUG=547158

Committed: https://crrev.com/e7aa37d9324beb59d3d7edd574e13346f1b2283a
Cr-Commit-Position: refs/heads/master@{#384322}
==========

[commit-bot: I haz the power, 2016-03-31 17:57:29]

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