Returning scoped_ptr instead of raw pointer in SocketPoolInfoToValue() in net/

Returning scoped_ptr instead of raw pointer in SocketPoolInfoToValue() in net/

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

[payal.pandey, 2015-05-26 12:29:02]

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

[payal.pandey, 2015-05-26 12:29:02]

Please have a look, Thanks.

[eroman, 2015-05-26 17:46:51]

https://codereview.chromium.org/1153003002/diff/20001/net/socket/mock_client_...
File net/socket/mock_client_socket_pool_manager.cc (right):

https://codereview.chromium.org/1153003002/diff/20001/net/socket/mock_client_...
net/socket/mock_client_socket_pool_manager.cc:92: scoped_ptr<base::ListValue>
list(new base::ListValue());
This is not equivalent. Before it was returning NULL. A more direct translation
would be:

   return scoped_ptr<base::Value>(nullptr);

[payal.pandey, 2015-05-27 06:42:32]

Thanks for guidance, Update the CL as directed.

>    return scoped_ptr<base::Value>(nullptr);

Please review.

[eroman, 2015-05-27 18:25:04]

lgtm

[payal.pandey, 2015-05-28 07:26:09]

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

[commit-bot: I haz the power, 2015-05-28 07:26:23]

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

[commit-bot: I haz the power, 2015-05-28 09:30:05]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-05-28 09:31:08]

Patchset 3 (id:??) landed as
https://crrev.com/62a400297d375a9cd558104c5153182e0b08d80e
Cr-Commit-Position: refs/heads/master@{#331760}