Returning scoped_ptr instead of raw pointer in QuicInfoToValue in net/

Returning scoped_ptr instead of raw pointer in QuicInfoToValue in net/

BUG=472381
Committed: https://crrev.com/a18956abbc96a3714625dacaef1ff1285acb6136
Cr-Commit-Position: refs/heads/master@{#331538}

[payal.pandey, 2015-05-25 11:39:48]

payal.pandey@samsung.com changed reviewers:
+ eroman@chromium.org

[payal.pandey, 2015-05-25 11:39:48]

Please have a look, Thanks.

[eroman, 2015-05-25 17:48:56]

https://codereview.chromium.org/1149243003/diff/1/net/http/http_network_sessi...
File net/http/http_network_session.cc (right):

https://codereview.chromium.org/1149243003/diff/1/net/http/http_network_sessi...
net/http/http_network_session.cc:265:
quic_stream_factory_.QuicStreamFactoryInfoToValue().Pass());
Remove .Pass()

https://codereview.chromium.org/1149243003/diff/1/net/http/http_network_sessi...
File net/http/http_network_session.h (right):

https://codereview.chromium.org/1149243003/diff/1/net/http/http_network_sessi...
net/http/http_network_session.h:195: // configuration. The caller is responsible
for deleting the returned value.
Remove the comment "The caller is responsible..."

https://codereview.chromium.org/1149243003/diff/1/net/log/net_log_util.cc
File net/log/net_log_util.cc (right):

https://codereview.chromium.org/1149243003/diff/1/net/log/net_log_util.cc#new...
net/log/net_log_util.cc:453: http_network_session->QuicInfoToValue().Pass());
Remove .Pass()

https://codereview.chromium.org/1149243003/diff/1/net/quic/quic_stream_factor...
File net/quic/quic_stream_factory.cc (right):

https://codereview.chromium.org/1149243003/diff/1/net/quic/quic_stream_factor...
net/quic/quic_stream_factory.cc:944:
list->Append(session->GetInfoAsValue(hosts).Pass());
remove .Pass()

[payal.pandey, 2015-05-26 06:24:15]

Updated the CL as per review comments, 
Please have a look again, Thanks.

https://codereview.chromium.org/1149243003/diff/1/net/http/http_network_sessi...
File net/http/http_network_session.cc (right):

https://codereview.chromium.org/1149243003/diff/1/net/http/http_network_sessi...
net/http/http_network_session.cc:265:
quic_stream_factory_.QuicStreamFactoryInfoToValue().Pass());
On 2015/05/25 17:48:56, eroman wrote:
> Remove .Pass()

Done.

https://codereview.chromium.org/1149243003/diff/1/net/http/http_network_sessi...
File net/http/http_network_session.h (right):

https://codereview.chromium.org/1149243003/diff/1/net/http/http_network_sessi...
net/http/http_network_session.h:195: // configuration. The caller is responsible
for deleting the returned value.
On 2015/05/25 17:48:56, eroman wrote:
> Remove the comment "The caller is responsible..."

Done.

https://codereview.chromium.org/1149243003/diff/1/net/log/net_log_util.cc
File net/log/net_log_util.cc (right):

https://codereview.chromium.org/1149243003/diff/1/net/log/net_log_util.cc#new...
net/log/net_log_util.cc:453: http_network_session->QuicInfoToValue().Pass());
On 2015/05/25 17:48:56, eroman wrote:
> Remove .Pass()

Done.

https://codereview.chromium.org/1149243003/diff/1/net/quic/quic_stream_factor...
File net/quic/quic_stream_factory.cc (right):

https://codereview.chromium.org/1149243003/diff/1/net/quic/quic_stream_factor...
net/quic/quic_stream_factory.cc:944:
list->Append(session->GetInfoAsValue(hosts).Pass());
On 2015/05/25 17:48:56, eroman wrote:
> remove .Pass()

Done.

https://codereview.chromium.org/1149243003/diff/1/net/quic/quic_stream_factor...
net/quic/quic_stream_factory.cc:944:
list->Append(session->GetInfoAsValue(hosts).Pass());
On 2015/05/25 17:48:56, eroman wrote:
> remove .Pass()

Done.

[eroman, 2015-05-26 17:16:35]

lgtm

[payal.pandey, 2015-05-27 04:51:23]

The CQ bit was checked by payal.pandey@samsung.com

[commit-bot: I haz the power, 2015-05-27 04:51:34]

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

[commit-bot: I haz the power, 2015-05-27 05:58:03]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-27 05:58:51]

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