Revert of Set HexSSID in network config before matching it against policies

Revert of Set HexSSID in network config before matching it against policies (patchset #18 id:330001 of https://codereview.chromium.org/2767253006/ )

Reason for revert:
Breaking content_browsertests on Marshmallow 64 bit tester: https://uberchromegw.corp.google.com/i/chromium.android/builders/Marshmallow%2064%20bit%20Tester/builds/10654

Original issue's description:
> Set HexSSID in network config before matching it against policies
> 
> When creating a new network configuration, the config is matched
> against existing, policy configured configs. If the network is already
> configured, network configuration should fail. The problem is that
> policy matching compares network SSIDs using HexSSID property, which
> was not required in the new network config (network configuration
> creation would succeed with only SSID set). This CL ensures that the
> HexSSID property gets populated before matching the configuration
> against existing policies. Also, it adds a step for verifying the
> user defined oncconfiguration (in particular that property types are
> properly set).
> 
> BUG=705024
> 
> Review-Url: https://codereview.chromium.org/2767253006
> Cr-Commit-Position: refs/heads/master@{#460018}
> Committed: https://chromium.googlesource.com/chromium/src/+/8000869bbcac8cfd3e24de549d8cf07547d152a5

TBR=stevenjb@chromium.org,rdevlin.cronin@chromium.org,tbarzic@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=705024
Review-Url: https://codereview.chromium.org/2776403002
Cr-Commit-Position: refs/heads/master@{#460126}
Committed: https://chromium.googlesource.com/chromium/src/+/78d2f5428e959b95e1342a26ace0f29b17493ec3

[mdjones, 2017-03-28 16:39:44]

The CQ bit was checked by mdjones@chromium.org

[mdjones, 2017-03-28 16:39:44]

Created Revert of Set HexSSID in network config before matching it against
policies

[commit-bot: I haz the power, 2017-03-28 16:40:15]

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-03-28 16:41:32]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490719184102260, "parent_rev":
"ba5cc8e55eacf6fc90a8504c34aff210ae82f7f6", "commit_rev":
"78d2f5428e959b95e1342a26ace0f29b17493ec3"}

[commit-bot: I haz the power, 2017-03-28 16:42:22]

Description was changed from

==========
Revert of Set HexSSID in network config before matching it against policies
(patchset #18 id:330001 of https://codereview.chromium.org/2767253006/ )

Reason for revert:
Breaking content_browsertests on Marshmallow 64 bit tester:
https://uberchromegw.corp.google.com/i/chromium.android/builders/Marshmallow%...

Original issue's description:
> Set HexSSID in network config before matching it against policies
> 
> When creating a new network configuration, the config is matched
> against existing, policy configured configs. If the network is already
> configured, network configuration should fail. The problem is that
> policy matching compares network SSIDs using HexSSID property, which
> was not required in the new network config (network configuration
> creation would succeed with only SSID set). This CL ensures that the
> HexSSID property gets populated before matching the configuration
> against existing policies. Also, it adds a step for verifying the
> user defined oncconfiguration (in particular that property types are
> properly set).
> 
> BUG=705024
> 
> Review-Url: https://codereview.chromium.org/2767253006
> Cr-Commit-Position: refs/heads/master@{#460018}
> Committed:
https://chromium.googlesource.com/chromium/src/+/8000869bbcac8cfd3e24de549d8c...

TBR=stevenjb@chromium.org,rdevlin.cronin@chromium.org,tbarzic@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=705024
==========

to

==========
Revert of Set HexSSID in network config before matching it against policies
(patchset #18 id:330001 of https://codereview.chromium.org/2767253006/ )

Reason for revert:
Breaking content_browsertests on Marshmallow 64 bit tester:
https://uberchromegw.corp.google.com/i/chromium.android/builders/Marshmallow%...

Original issue's description:
> Set HexSSID in network config before matching it against policies
> 
> When creating a new network configuration, the config is matched
> against existing, policy configured configs. If the network is already
> configured, network configuration should fail. The problem is that
> policy matching compares network SSIDs using HexSSID property, which
> was not required in the new network config (network configuration
> creation would succeed with only SSID set). This CL ensures that the
> HexSSID property gets populated before matching the configuration
> against existing policies. Also, it adds a step for verifying the
> user defined oncconfiguration (in particular that property types are
> properly set).
> 
> BUG=705024
> 
> Review-Url: https://codereview.chromium.org/2767253006
> Cr-Commit-Position: refs/heads/master@{#460018}
> Committed:
https://chromium.googlesource.com/chromium/src/+/8000869bbcac8cfd3e24de549d8c...

TBR=stevenjb@chromium.org,rdevlin.cronin@chromium.org,tbarzic@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=705024

Review-Url: https://codereview.chromium.org/2776403002
Cr-Commit-Position: refs/heads/master@{#460126}
Committed:
https://chromium.googlesource.com/chromium/src/+/78d2f5428e959b95e1342a26ace0...
==========

[commit-bot: I haz the power, 2017-03-28 16:42:23]

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

[mdjones, 2017-03-28 20:25:58]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2784643002/ by mdjones@chromium.org.

The reason for reverting is: Relanding since the original patch did not cause
the break..