[Extensions] Remove linked_ptr entirely from extensions generated code

[Extensions] Remove linked_ptr entirely from extensions generated code

After the work to implement movable types in extensions generated code
(completed in r384773), we now only use linked_ptrs to store base::Values in
containers. Luckily, we don't need to - convert these to scoped_ptrs and
eliminate all traces of linked_ptrs from extensions generated code.

BUG=595949
TBR=isherman@chromium.org (autofill IWYU fix)

Committed: https://crrev.com/3d8ce9c0d399ff8897632ce7469c56f2e2487234
Cr-Commit-Position: refs/heads/master@{#385005}

[Devlin, 2016-04-04 16:22:17]

Patchset #1 (id:1) has been deleted

[Devlin, 2016-04-04 16:42:17]

Description was changed from

==========
No linked
BUG=
==========

to

==========
[Extensions] Remove linked_ptr entirely from extensions generated code

After the work to implement movable types in extensions generated code
(completed in r384773), we now only use linked_ptrs to store base::Values in
containers. Luckily, we don't need to - convert these to scoped_ptrs and
eliminate all traces of linked_ptrs from extensions generated code.

BUG=595949
==========

[Devlin, 2016-04-04 16:43:49]

rdevlin.cronin@chromium.org changed reviewers:
+ lazyboy@chromium.org

[Devlin, 2016-04-04 16:43:50]

Istiaque, mind taking a look?

(Note: the added #includes are for IWYU errors)

[lazyboy, 2016-04-04 18:36:51]

lgtm

[Devlin, 2016-04-04 20:42:47]

The CQ bit was checked by rdevlin.cronin@chromium.org

[Devlin, 2016-04-04 20:42:48]

The patchset sent to the CQ was uploaded after l-g-t-m from lazyboy@chromium.org
Link to the patchset: https://codereview.chromium.org/1854993002/#ps40001
(title: "Rebase")

[commit-bot: I haz the power, 2016-04-04 20:43:34]

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

[commit-bot: I haz the power, 2016-04-04 20:55:07]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-04 20:55:09]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Devlin, 2016-04-04 21:02:40]

Description was changed from

==========
[Extensions] Remove linked_ptr entirely from extensions generated code

After the work to implement movable types in extensions generated code
(completed in r384773), we now only use linked_ptrs to store base::Values in
containers. Luckily, we don't need to - convert these to scoped_ptrs and
eliminate all traces of linked_ptrs from extensions generated code.

BUG=595949
==========

to

==========
[Extensions] Remove linked_ptr entirely from extensions generated code

After the work to implement movable types in extensions generated code
(completed in r384773), we now only use linked_ptrs to store base::Values in
containers. Luckily, we don't need to - convert these to scoped_ptrs and
eliminate all traces of linked_ptrs from extensions generated code.

BUG=595949
TBR=isherman@chromium.org (autofill IWYU fix)
==========

[Devlin, 2016-04-04 21:02:40]

rdevlin.cronin@chromium.org changed reviewers:
+ isherman@chromium.org

[Devlin, 2016-04-04 21:02:45]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2016-04-04 21:03:11]

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

[commit-bot: I haz the power, 2016-04-04 21:14:26]

Description was changed from

==========
[Extensions] Remove linked_ptr entirely from extensions generated code

After the work to implement movable types in extensions generated code
(completed in r384773), we now only use linked_ptrs to store base::Values in
containers. Luckily, we don't need to - convert these to scoped_ptrs and
eliminate all traces of linked_ptrs from extensions generated code.

BUG=595949
TBR=isherman@chromium.org (autofill IWYU fix)
==========

to

==========
[Extensions] Remove linked_ptr entirely from extensions generated code

After the work to implement movable types in extensions generated code
(completed in r384773), we now only use linked_ptrs to store base::Values in
containers. Luckily, we don't need to - convert these to scoped_ptrs and
eliminate all traces of linked_ptrs from extensions generated code.

BUG=595949
TBR=isherman@chromium.org (autofill IWYU fix)
==========

[commit-bot: I haz the power, 2016-04-04 21:14:28]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-04-04 21:15:32]

Description was changed from

==========
[Extensions] Remove linked_ptr entirely from extensions generated code

After the work to implement movable types in extensions generated code
(completed in r384773), we now only use linked_ptrs to store base::Values in
containers. Luckily, we don't need to - convert these to scoped_ptrs and
eliminate all traces of linked_ptrs from extensions generated code.

BUG=595949
TBR=isherman@chromium.org (autofill IWYU fix)
==========

to

==========
[Extensions] Remove linked_ptr entirely from extensions generated code

After the work to implement movable types in extensions generated code
(completed in r384773), we now only use linked_ptrs to store base::Values in
containers. Luckily, we don't need to - convert these to scoped_ptrs and
eliminate all traces of linked_ptrs from extensions generated code.

BUG=595949
TBR=isherman@chromium.org (autofill IWYU fix)

Committed: https://crrev.com/3d8ce9c0d399ff8897632ce7469c56f2e2487234
Cr-Commit-Position: refs/heads/master@{#385005}
==========

[commit-bot: I haz the power, 2016-04-04 21:15:34]

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