Fix policy_util iterator and re-enable networking tests

Fix policy_util iterator and re-enable networking tests

This fixes a DCHECK that exposed a poor assumption in
RemoveFakeCredentials about DictionaryValue iteration. It was exposed
with this CL:
https://codereview.chromium.org/2807953002

It also re-enables tests the disabled tests.

BUG=710241
Review-Url: https://codereview.chromium.org/2810703005
Cr-Commit-Position: refs/heads/master@{#463783}
Committed: https://chromium.googlesource.com/chromium/src/+/6e4bc12318fac3839ddf6f0b69dcbd5bfd962b56

[stevenjb, 2017-04-11 16:48:35]

Description was changed from

==========
Fix policy_util iterator and re-enable networking tests

This fixes a DCHECK that exposed a poor assumption in
RemoveFakeCredentials about DictionaryValue iteration. It was exposed
with this CL:
https://codereview.chromium.org/2807953002

BUG=710241
==========

to

==========
Fix policy_util iterator and re-enable networking tests

This fixes a DCHECK that exposed a poor assumption in
RemoveFakeCredentials about DictionaryValue iteration. It was exposed
with this CL:
https://codereview.chromium.org/2807953002

It also re-enables tests the disabled tests.

BUG=710241
==========

[stevenjb, 2017-04-11 16:49:28]

stevenjb@chromium.org changed reviewers:
+ brettw@chromium.org, mkwst@chromium.org

[brettw, 2017-04-11 19:57:10]

lgtm

[stevenjb, 2017-04-11 20:45:14]

The CQ bit was checked by stevenjb@chromium.org

[commit-bot: I haz the power, 2017-04-11 20:45:52]

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-11 21:46:01]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491943514227590, "parent_rev":
"c92ef59a640cf2dacabef34f8cf798e39c1e61f5", "commit_rev":
"6e4bc12318fac3839ddf6f0b69dcbd5bfd962b56"}

[commit-bot: I haz the power, 2017-04-11 21:46:58]

Description was changed from

==========
Fix policy_util iterator and re-enable networking tests

This fixes a DCHECK that exposed a poor assumption in
RemoveFakeCredentials about DictionaryValue iteration. It was exposed
with this CL:
https://codereview.chromium.org/2807953002

It also re-enables tests the disabled tests.

BUG=710241
==========

to

==========
Fix policy_util iterator and re-enable networking tests

This fixes a DCHECK that exposed a poor assumption in
RemoveFakeCredentials about DictionaryValue iteration. It was exposed
with this CL:
https://codereview.chromium.org/2807953002

It also re-enables tests the disabled tests.

BUG=710241

Review-Url: https://codereview.chromium.org/2810703005
Cr-Commit-Position: refs/heads/master@{#463783}
Committed:
https://chromium.googlesource.com/chromium/src/+/6e4bc12318fac3839ddf6f0b69dc...
==========

[commit-bot: I haz the power, 2017-04-11 21:46:59]

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