Convert //chrome from scoped_ptr to std::unique_ptr

Convert //chrome from scoped_ptr to std::unique_ptr

BUG=554298
NOPRESUBMIT=true

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

[dcheng, 2016-04-19 01:19:42]

dcheng@chromium.org changed reviewers:
+ thestig@chromium.org

[dcheng, 2016-04-19 01:19:44]

This should wrap up //chrome. I'll have to dcommit once the trybots are happy,
since one of the files (crdmg.cc) uses fprintf, even though it appears to be a
utility helper where you'd want to use printf/fprintf.

[Lei Zhang, 2016-04-19 01:35:43]

https://codereview.chromium.org/1899083002/diff/1/chrome/installer/mini_insta...
File chrome/installer/mini_installer/decompress.cc (right):

https://codereview.chromium.org/1899083002/diff/1/chrome/installer/mini_insta...
chrome/installer/mini_installer/decompress.cc:88: std::unique_ptr<wchar_t>
path(Utf8ToWide(pszFile));
Weren't we going to leave this alone?

https://codereview.chromium.org/1899083002/diff/1/chrome/service/cloud_print/...
File chrome/service/cloud_print/cloud_print_auth.h (right):

https://codereview.chromium.org/1899083002/diff/1/chrome/service/cloud_print/...
chrome/service/cloud_print/cloud_print_auth.h:83:
std::unique_ptr<gaia::GaiaOAuthClient> oauth_client_;
IWYU

https://codereview.chromium.org/1899083002/diff/1/chrome/service/service_proc...
File chrome/service/service_process_prefs.h (right):

https://codereview.chromium.org/1899083002/diff/1/chrome/service/service_proc...
chrome/service/service_process_prefs.h:60: void SetValue(const std::string& key,
std::unique_ptr<base::Value> value);
IWYU

https://codereview.chromium.org/1899083002/diff/1/chrome/service/service_util...
File chrome/service/service_utility_process_host.h (right):

https://codereview.chromium.org/1899083002/diff/1/chrome/service/service_util...
chrome/service/service_utility_process_host.h:13: #include
"build/build_config.h"
Probably not needed?

https://codereview.chromium.org/1899083002/diff/1/chrome/third_party/mozilla_...
File chrome/third_party/mozilla_security_manager/nsNSSCertHelper.h (right):

https://codereview.chromium.org/1899083002/diff/1/chrome/third_party/mozilla_...
chrome/third_party/mozilla_security_manager/nsNSSCertHelper.h:46: #include
<memory>
Not needed?

https://codereview.chromium.org/1899083002/diff/1/chrome/tools/service_discov...
File chrome/tools/service_discovery_sniffer/service_discovery_sniffer.h (right):

https://codereview.chromium.org/1899083002/diff/1/chrome/tools/service_discov...
chrome/tools/service_discovery_sniffer/service_discovery_sniffer.h:34:
std::unique_ptr<ServiceResolver> service_resolver_;
IWYU

https://codereview.chromium.org/1899083002/diff/1/chrome/utility/importer/edg...
File chrome/utility/importer/edge_database_reader_win.h (right):

https://codereview.chromium.org/1899083002/diff/1/chrome/utility/importer/edg...
chrome/utility/importer/edge_database_reader_win.h:11: #include <memory>
Move this down to after line 14.

[dcheng, 2016-04-19 03:51:34]

https://codereview.chromium.org/1899083002/diff/1/chrome/installer/mini_insta...
File chrome/installer/mini_installer/decompress.cc (right):

https://codereview.chromium.org/1899083002/diff/1/chrome/installer/mini_insta...
chrome/installer/mini_installer/decompress.cc:88: std::unique_ptr<wchar_t>
path(Utf8ToWide(pszFile));
On 2016/04/19 at 01:35:42, Lei Zhang wrote:
> Weren't we going to leave this alone?

Done.

https://codereview.chromium.org/1899083002/diff/1/chrome/service/cloud_print/...
File chrome/service/cloud_print/cloud_print_auth.h (right):

https://codereview.chromium.org/1899083002/diff/1/chrome/service/cloud_print/...
chrome/service/cloud_print/cloud_print_auth.h:83:
std::unique_ptr<gaia::GaiaOAuthClient> oauth_client_;
On 2016/04/19 at 01:35:42, Lei Zhang wrote:
> IWYU

Done.

https://codereview.chromium.org/1899083002/diff/1/chrome/service/service_proc...
File chrome/service/service_process_prefs.h (right):

https://codereview.chromium.org/1899083002/diff/1/chrome/service/service_proc...
chrome/service/service_process_prefs.h:60: void SetValue(const std::string& key,
std::unique_ptr<base::Value> value);
On 2016/04/19 at 01:35:42, Lei Zhang wrote:
> IWYU

Done.

https://codereview.chromium.org/1899083002/diff/1/chrome/service/service_util...
File chrome/service/service_utility_process_host.h (right):

https://codereview.chromium.org/1899083002/diff/1/chrome/service/service_util...
chrome/service/service_utility_process_host.h:13: #include
"build/build_config.h"
On 2016/04/19 at 01:35:42, Lei Zhang wrote:
> Probably not needed?

Done.

https://codereview.chromium.org/1899083002/diff/1/chrome/third_party/mozilla_...
File chrome/third_party/mozilla_security_manager/nsNSSCertHelper.h (right):

https://codereview.chromium.org/1899083002/diff/1/chrome/third_party/mozilla_...
chrome/third_party/mozilla_security_manager/nsNSSCertHelper.h:46: #include
<memory>
On 2016/04/19 at 01:35:42, Lei Zhang wrote:
> Not needed?

Done.

https://codereview.chromium.org/1899083002/diff/1/chrome/tools/service_discov...
File chrome/tools/service_discovery_sniffer/service_discovery_sniffer.h (right):

https://codereview.chromium.org/1899083002/diff/1/chrome/tools/service_discov...
chrome/tools/service_discovery_sniffer/service_discovery_sniffer.h:34:
std::unique_ptr<ServiceResolver> service_resolver_;
On 2016/04/19 at 01:35:42, Lei Zhang wrote:
> IWYU

Done.

https://codereview.chromium.org/1899083002/diff/1/chrome/utility/importer/edg...
File chrome/utility/importer/edge_database_reader_win.h (right):

https://codereview.chromium.org/1899083002/diff/1/chrome/utility/importer/edg...
chrome/utility/importer/edge_database_reader_win.h:11: #include <memory>
On 2016/04/19 at 01:35:43, Lei Zhang wrote:
> Move this down to after line 14.

Done.

[Lei Zhang, 2016-04-19 04:12:35]

lgtm

[dcheng, 2016-04-19 06:13:33]

Description was changed from

==========
Convert //chrome from scoped_ptr to std::unique_ptr

BUG=554298
==========

to

==========
Convert //chrome from scoped_ptr to std::unique_ptr

BUG=554298
NOPRESUBMIT=true
==========

[dcheng, 2016-04-19 06:13:39]

The CQ bit was checked by dcheng@chromium.org

[commit-bot: I haz the power, 2016-04-19 06:13:47]

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

[commit-bot: I haz the power, 2016-04-19 06:15:27]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-19 06:16:36]

Description was changed from

==========
Convert //chrome from scoped_ptr to std::unique_ptr

BUG=554298
NOPRESUBMIT=true
==========

to

==========
Convert //chrome from scoped_ptr to std::unique_ptr

BUG=554298
NOPRESUBMIT=true

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

[commit-bot: I haz the power, 2016-04-19 06:16:38]

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