QUIC - use HostPortPair instead of scoped_ptr<HostPortPair>.

QUIC - use HostPortPair instead of scoped_ptr<HostPortPair>.

R=wtc@chromium.org

Committed: https://crrev.com/b77a6a1dffd033fa238751b034b5d136e560eb5d
Cr-Commit-Position: refs/heads/master@{#292274}

[wtc, 2014-08-27 22:15:03]

Patch set 1 LGTM. Please make the following change.

https://codereview.chromium.org/511543006/diff/1/net/quic/quic_client_session.cc
File net/quic/quic_client_session.cc (right):

https://codereview.chromium.org/511543006/diff/1/net/quic/quic_client_session...
net/quic/quic_client_session.cc:167: server_host_port_ =
HostPortPair(server_id.host_port_pair());

This should say:
  server_host_port_ = server_id.host_port_pair();

[ramant (doing other things), 2014-08-27 22:24:56]

thanks Wan-Teh (missed we are getting host_port_pair from server_id).

https://codereview.chromium.org/511543006/diff/1/net/quic/quic_client_session.cc
File net/quic/quic_client_session.cc (right):

https://codereview.chromium.org/511543006/diff/1/net/quic/quic_client_session...
net/quic/quic_client_session.cc:167: server_host_port_ =
HostPortPair(server_id.host_port_pair());
On 2014/08/27 22:15:03, wtc wrote:
> 
> This should say:
>   server_host_port_ = server_id.host_port_pair();
>   

Done.

[ramant (doing other things), 2014-08-27 22:25:48]

The CQ bit was checked by rtenneti@chromium.org

[commit-bot: I haz the power, 2014-08-27 22:27:29]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rtenneti@chromium.org/511543006/20001

[wtc, 2014-08-27 22:45:02]

Patch set 2 LGTM.

[commit-bot: I haz the power, 2014-08-27 23:40:37]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  ios_dbg_simulator on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)

[commit-bot: I haz the power, 2014-08-27 23:54:12]

Committed patchset #2 (id:20001) as db3dc5325fce5bee800497de2a1e633d063cea4d

[commit-bot: I haz the power, 2014-09-10 02:55:51]

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