Returning scoped_ptr<> instead of raw pointer in ProxyConfig::ToValue

Returning scoped_ptr<> instead of raw pointer in ProxyConfig::ToValue

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

[Paritosh Kumar, 2015-08-10 13:08:01]

paritosh.in@samsung.com changed reviewers:
+ eroman@chromium.org

[Paritosh Kumar, 2015-08-10 13:08:02]

Please have a look.

[eroman, 2015-08-10 22:12:42]

looks good after fixing comment

https://codereview.chromium.org/1280973004/diff/1/net/proxy/proxy_config.h
File net/proxy/proxy_config.h (right):

https://codereview.chromium.org/1280973004/diff/1/net/proxy/proxy_config.h#ne...
net/proxy/proxy_config.h:177: // Creates a Value dump of this configuration. The
caller is responsible for
Please delete the comment on ownership (as it is now explicit in the API)

[Paritosh Kumar, 2015-08-11 04:39:40]

On 2015/08/10 22:12:42, eroman wrote:
> looks good after fixing comment
> 
> https://codereview.chromium.org/1280973004/diff/1/net/proxy/proxy_config.h
> File net/proxy/proxy_config.h (right):
> 
>
https://codereview.chromium.org/1280973004/diff/1/net/proxy/proxy_config.h#ne...
> net/proxy/proxy_config.h:177: // Creates a Value dump of this configuration.
The
> caller is responsible for
> Please delete the comment on ownership (as it is now explicit in the API)

Thanks eroman
updated. Please have a look.

[eroman, 2015-08-11 04:41:39]

lgtm

[Paritosh Kumar, 2015-08-11 04:41:58]

The CQ bit was checked by paritosh.in@samsung.com

[commit-bot: I haz the power, 2015-08-11 04:42:04]

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

[commit-bot: I haz the power, 2015-08-11 05:53:37]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-08-11 05:54:20]

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