Autofill's CountryCode should use scoped_ptr for owned pointers

Autofill's CountryCode should use scoped_ptr for owned pointers

This CL turns a map of raw pointers to a map of scoped_ptrs. It is a clean-up
which is unlikely to fix the associated crash, but the explicit semantics
might help discover the real cause. And besides, using scopers for owned
pointers is a good practice.

BUG=571371
Committed: https://crrev.com/e35c03b8f0a01e5882596b4527feeee2d4f5bd7d
Cr-Commit-Position: refs/heads/master@{#366414}

[vabr (Chromium), 2015-12-21 16:43:06]

vabr@chromium.org changed reviewers:
+ mathp@chromium.org

[vabr (Chromium), 2015-12-21 16:43:06]

Hello mathp@,

Please take a look.

Cheers,
Vaclav

[Mathieu, 2015-12-21 16:45:41]

lgtm, thanks for the cleanup!

[vabr (Chromium), 2015-12-21 16:49:46]

Thanks!

The compile errors on trybots seem unrelated, so sending this to the CQ.

Cheers,
Vaclav

[vabr (Chromium), 2015-12-21 16:49:53]

The CQ bit was checked by vabr@chromium.org

[commit-bot: I haz the power, 2015-12-21 16:50:23]

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

[commit-bot: I haz the power, 2015-12-21 17:48:38]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-21 17:49:23]

Description was changed from

==========
Autofill's CountryCode should use scoped_ptr for owned pointers

This CL turns a map of raw pointers to a map of scoped_ptrs. It is a clean-up
which is unlikely to fix the associated crash, but the explicit semantics
might help discover the real cause. And besides, using scopers for owned
pointers is a good practice.

BUG=571371
==========

to

==========
Autofill's CountryCode should use scoped_ptr for owned pointers

This CL turns a map of raw pointers to a map of scoped_ptrs. It is a clean-up
which is unlikely to fix the associated crash, but the explicit semantics
might help discover the real cause. And besides, using scopers for owned
pointers is a good practice.

BUG=571371

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

[commit-bot: I haz the power, 2015-12-21 17:49:24]

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