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

Reland of Set HexSSID in network config before matching it against policies (patchset #1 id:1 of https://codereview.chromium.org/2776403002/ )

Reason for revert:
Relanding since the original patch did not cause the break.

Original issue's description:
> 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

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/2784643002
Cr-Commit-Position: refs/heads/master@{#460207}
Committed: https://chromium.googlesource.com/chromium/src/+/92cb8c04c342bb35a96ee77c055f7cec8ff3ee41

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

The CQ bit was checked by mdjones@chromium.org

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

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

[commit-bot: I haz the power, 2017-03-28 20:26:35]

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 20:33:55]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490732758392390, "parent_rev":
"3a4ea7d396507cf516f8afb899aff72775f5d0d9", "commit_rev":
"92cb8c04c342bb35a96ee77c055f7cec8ff3ee41"}

[commit-bot: I haz the power, 2017-03-28 20:35:27]

Description was changed from

==========
Reland of Set HexSSID in network config before matching it against policies
(patchset #1 id:1 of https://codereview.chromium.org/2776403002/ )

Reason for revert:
Relanding since the original patch did not cause the break.

Original issue's description:
> 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...

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

==========
Reland of Set HexSSID in network config before matching it against policies
(patchset #1 id:1 of https://codereview.chromium.org/2776403002/ )

Reason for revert:
Relanding since the original patch did not cause the break.

Original issue's description:
> 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...

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/2784643002
Cr-Commit-Position: refs/heads/master@{#460207}
Committed:
https://chromium.googlesource.com/chromium/src/+/92cb8c04c342bb35a96ee77c055f...
==========

[commit-bot: I haz the power, 2017-03-28 20:35:28]

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