Remove raw base::DictionaryValue::Set in //net

Remove raw DictionaryValue::Set in //net

This change removes the deprecated raw pointer version of
base::DictionaryValue::Set in //net and replaces it with the unique
pointer version or other convenience functions where appropriate.

BUG=646113, 581865
Review-Url: https://codereview.chromium.org/2899723003
Cr-Commit-Position: refs/heads/master@{#477614}
Committed: https://chromium.googlesource.com/chromium/src/+/4eea70fa2aa13869e8599bc1cd511a0e05146af4

[jdoerrie, 2017-05-22 16:51:34]

The CQ bit was checked by jdoerrie@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-22 16:52:13]

Dry run: 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-05-22 19:24:24]

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

[commit-bot: I haz the power, 2017-05-22 19:24:25]

Dry run: This issue passed the CQ dry run.

[jdoerrie, 2017-05-30 09:58:08]

The CQ bit was checked by jdoerrie@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-30 09:58:19]

Dry run: 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-05-30 11:31:29]

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

[commit-bot: I haz the power, 2017-05-30 11:31:30]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[jdoerrie, 2017-06-02 13:36:55]

The CQ bit was checked by jdoerrie@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-02 13:37:09]

Dry run: CQ is trying da patch.

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

[jdoerrie, 2017-06-02 13:42:40]

Description was changed from

==========
Remove raw base::DictionaryValue::Set in //net

BUG=646113
==========

to

==========
Remove raw DictionaryValue::Set in //net

This change removes the deprecated raw pointer version of
base::DictionaryValue::Set in //net and replaces it with the unique
pointer version or other convenience functions where appropriate.

BUG=646113, 581865
==========

[jdoerrie, 2017-06-02 13:42:41]

jdoerrie@chromium.org changed reviewers:
+ agl@chromium.org, vabr@chromium.org

[jdoerrie, 2017-06-02 13:45:08]

vabr@chromium.org: Please review the whole CL in detail. 

agl@chromium.org: Please spot-check and rubber-stamp.

[commit-bot: I haz the power, 2017-06-02 15:04:56]

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

[commit-bot: I haz the power, 2017-06-02 15:04:59]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[vabr (Chromium), 2017-06-03 10:57:59]

LGTM, great to see the raw pointers gone!
Cheers,
Vaclav

[jdoerrie, 2017-06-06 12:25:13]

Thanks, Vaclav!

Friendly ping at agl@.

[agl, 2017-06-06 16:27:37]

lgtm

[jdoerrie, 2017-06-07 10:34:30]

The CQ bit was checked by jdoerrie@chromium.org

[commit-bot: I haz the power, 2017-06-07 10:34:39]

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-06-07 11:58:36]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1496831670644300,
"parent_rev": "750ac5aac33cec642a81603e2ecd8f58eadf7bb1", "commit_rev":
"4eea70fa2aa13869e8599bc1cd511a0e05146af4"}

[commit-bot: I haz the power, 2017-06-07 11:58:46]

Description was changed from

==========
Remove raw DictionaryValue::Set in //net

This change removes the deprecated raw pointer version of
base::DictionaryValue::Set in //net and replaces it with the unique
pointer version or other convenience functions where appropriate.

BUG=646113, 581865
==========

to

==========
Remove raw DictionaryValue::Set in //net

This change removes the deprecated raw pointer version of
base::DictionaryValue::Set in //net and replaces it with the unique
pointer version or other convenience functions where appropriate.

BUG=646113, 581865

Review-Url: https://codereview.chromium.org/2899723003
Cr-Commit-Position: refs/heads/master@{#477614}
Committed:
https://chromium.googlesource.com/chromium/src/+/4eea70fa2aa13869e8599bc1cd51...
==========

[commit-bot: I haz the power, 2017-06-07 11:58:47]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/4eea70fa2aa13869e8599bc1cd51...