Improved handling of importer profile creation.

Improved handling of importer profile creation.
This patch improves the handling of importer profile creation to remove
reliance on bare pointers. It also wraps created importer objects in a
scoped_refptr which is what the pointer is assigned to in the caller.

BUG=539746
Committed: https://crrev.com/d07bcf1b1fb4172a9a00fcc7bbddb685c0a33f8e
Cr-Commit-Position: refs/heads/master@{#363033}

[forshaw, 2015-12-03 18:31:11]

forshaw@chromium.org changed reviewers:
+ isherman@chromium.org

[forshaw, 2015-12-03 18:31:12]

isherman@ this is the CL for fixing up the memory handling of importers you
requested. I've made it a copy version as I don't expect it's a particularly hot
path for performance. The only possible issue is I've had to add the import for
the structure type, but I could probably replace that with a scoped_ptr in a
vector if need be. PTAL.

Thanks.

[Ilya Sherman, 2015-12-03 18:34:47]

LGTM.  Thanks very much for the cleanup +)

[Ilya Sherman, 2015-12-03 18:35:01]

On 2015/12/03 18:34:47, Ilya Sherman wrote:
> LGTM.  Thanks very much for the cleanup +)

Erm, that was meant to be a smiley: =)

[Ilya Sherman, 2015-12-03 18:35:05]

The CQ bit was checked by isherman@chromium.org

[commit-bot: I haz the power, 2015-12-03 18:35:42]

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

[commit-bot: I haz the power, 2015-12-03 19:30:48]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-03 19:31:57]

Description was changed from

==========
Improved handling of importer profile creation.
This patch improves the handling of importer profile creation to remove
reliance on bare pointers. It also wraps created importer objects in a
scoped_refptr which is what the pointer is assigned to in the caller.

BUG=539746
==========

to

==========
Improved handling of importer profile creation.
This patch improves the handling of importer profile creation to remove
reliance on bare pointers. It also wraps created importer objects in a
scoped_refptr which is what the pointer is assigned to in the caller.

BUG=539746

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

[commit-bot: I haz the power, 2015-12-03 19:31:58]

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