Revert of Stop passing raw pointers to base::Value API in c/b/chromeos and c/b/extensions

Revert of Stop passing raw pointers to base::Value API in c/b/chromeos and c/b/extensions (patchset #5 id:80001 of https://codereview.chromium.org/2777063003/ )

Reason for revert:

Findit identified CL at revision 463189 as the culprit for
failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtLzRiMGM0OWVjNDEyYzAyMTAzZTA5MjU5MzA0MWM5OWNiNjlmMTcxOTEM

Original issue's description:
> Stop passing raw pointers to base::Value API in c/b/chromeos and c/b/extensions
> 
> Passing ownership of base::Value into API methods DictionaryValue::Set,
> DictionaryValue::SetWithoutPathExpansion, and ListValue::Set through raw
> pointers is deprecated, can hide bugs and should be done via unique_ptr
> instead.
> 
> Therefore, this CL migrates c/b/chromeos, c/b/extensions and a bunch of related
> files to use the unique_ptr-based API.
> 
> BUG=697817
> TBR=cpu@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2777063003
> Cr-Commit-Position: refs/heads/master@{#463189}
> Committed: https://chromium.googlesource.com/chromium/src/+/4b0c49ec412c02103e092593041c99cb69f17191

TBR=atwilson@chromium.org,derat@chromium.org,finnur@chromium.org,bauerb@chromium.org,benwells@chromium.org,dgozman@chromium.org,cpu@chromium.org,jdoerrie@chromium.org,vabr@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=697817
Review-Url: https://codereview.chromium.org/2806283002
Cr-Commit-Position: refs/heads/master@{#463206}
Committed: https://chromium.googlesource.com/chromium/src/+/8e19271dd4d215d9c967a6054263e8e0f5434500

[findit-for-me, 2017-04-10 09:01:28]

Created Revert of Stop passing raw pointers to base::Value API in c/b/chromeos
and c/b/extensions

[findit-for-me, 2017-04-10 09:01:29]

findit-for-me@appspot.gserviceaccount.com changed reviewers:
+ haraken@google.com, hongchan@google.com, mkwst@google.com, rouslan@google.com

[findit-for-me, 2017-04-10 09:01:30]

Sheriffs:

Please confirm and "Quick L-G-T-M & CQ" this revert if it is correct.
If it is a false positive, please close it.

The original CL was identified as the culprit for failures in the build
cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3Itb...

[Mike West, 2017-04-10 09:45:50]

mkwst@chromium.org changed reviewers:
+ mkwst@chromium.org

[Mike West, 2017-04-10 09:45:50]

The CQ bit was checked by mkwst@chromium.org

[Mike West, 2017-04-10 09:45:51]

lgtm

[commit-bot: I haz the power, 2017-04-10 09:46:02]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-10 09:48:01]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491817550574710, "parent_rev":
"a0158c4544b8dbf598dfb829426ab6d755843369", "commit_rev":
"8e19271dd4d215d9c967a6054263e8e0f5434500"}

[commit-bot: I haz the power, 2017-04-10 09:48:51]

Description was changed from

==========
Revert of Stop passing raw pointers to base::Value API in c/b/chromeos and
c/b/extensions (patchset #5 id:80001 of
https://codereview.chromium.org/2777063003/ )

Reason for revert:

Findit identified CL at revision 463189 as the culprit for
failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3Itb...

Original issue's description:
> Stop passing raw pointers to base::Value API in c/b/chromeos and
c/b/extensions
> 
> Passing ownership of base::Value into API methods DictionaryValue::Set,
> DictionaryValue::SetWithoutPathExpansion, and ListValue::Set through raw
> pointers is deprecated, can hide bugs and should be done via unique_ptr
> instead.
> 
> Therefore, this CL migrates c/b/chromeos, c/b/extensions and a bunch of
related
> files to use the unique_ptr-based API.
> 
> BUG=697817
> TBR=cpu@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2777063003
> Cr-Commit-Position: refs/heads/master@{#463189}
> Committed:
https://chromium.googlesource.com/chromium/src/+/4b0c49ec412c02103e092593041c...

TBR=atwilson@chromium.org,derat@chromium.org,finnur@chromium.org,bauerb@chrom...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=697817
==========

to

==========
Revert of Stop passing raw pointers to base::Value API in c/b/chromeos and
c/b/extensions (patchset #5 id:80001 of
https://codereview.chromium.org/2777063003/ )

Reason for revert:

Findit identified CL at revision 463189 as the culprit for
failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3Itb...

Original issue's description:
> Stop passing raw pointers to base::Value API in c/b/chromeos and
c/b/extensions
> 
> Passing ownership of base::Value into API methods DictionaryValue::Set,
> DictionaryValue::SetWithoutPathExpansion, and ListValue::Set through raw
> pointers is deprecated, can hide bugs and should be done via unique_ptr
> instead.
> 
> Therefore, this CL migrates c/b/chromeos, c/b/extensions and a bunch of
related
> files to use the unique_ptr-based API.
> 
> BUG=697817
> TBR=cpu@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2777063003
> Cr-Commit-Position: refs/heads/master@{#463189}
> Committed:
https://chromium.googlesource.com/chromium/src/+/4b0c49ec412c02103e092593041c...

TBR=atwilson@chromium.org,derat@chromium.org,finnur@chromium.org,bauerb@chrom...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=697817

Review-Url: https://codereview.chromium.org/2806283002
Cr-Commit-Position: refs/heads/master@{#463206}
Committed:
https://chromium.googlesource.com/chromium/src/+/8e19271dd4d215d9c967a6054263...
==========

[commit-bot: I haz the power, 2017-04-10 09:48:52]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/8e19271dd4d215d9c967a6054263...