DescriptionRevert 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
Patch Set 1 #
Messages
Total messages: 7 (3 generated)
|