arc: Update device owner check implementation

arc: Update device owner check implementation

Update the device owner check to compare the account id of the active
user with the account id of the owner as provided by the user manager
API. The old implementation uses the logged in user state API which for
unbeknownst reasons returns the owner as a regular user. It can also
change asynchronously at boot resulting in racy behavior as its used
right now.

BUG=702311
BUG=b:34661187
BUG=b:33488433
TEST=run cts -c android.net.wifi.cts.WifiEnterpriseConfigTest -m
testAddEapNetwork
TEST=run cts -c android.net.wifi.cts.WifiManagerTest -m
testWifiManagerNetWork
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2756443003
Cr-Commit-Position: refs/heads/master@{#457563}
(cherry picked from commit eb260ae77f464ec4b53627278b9e842cafb3490e)

Review-Url: https://codereview.chromium.org/2757783002
Cr-Commit-Position: refs/branch-heads/3029@{#251}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}
Committed: https://chromium.googlesource.com/chromium/src/+/63b114848a8a4b72925cf14bbcd15d9f540271c3

[abhishekbh, 2017-03-16 23:08:23]

Description was changed from

==========
arc: Update device owner check implementation

Update the device owner check to compare the account id of the active
user with the account id of the owner as provided by the user manager
API. The old implementation uses the logged in user state API which for
unbeknownst reasons returns the owner as a regular user. It can also
change asynchronously at boot resulting in racy behavior as its used
right now.

BUG=b:34661187
BUG=b:33488433
TEST=run cts -c android.net.wifi.cts.WifiEnterpriseConfigTest -m
testAddEapNetwork
TEST=run cts -c android.net.wifi.cts.WifiManagerTest -m
testWifiManagerNetWork

Review-Url: https://codereview.chromium.org/2756443003
Cr-Commit-Position: refs/heads/master@{#457563}
(cherry picked from commit eb260ae77f464ec4b53627278b9e842cafb3490e)
==========

to

==========
arc: Update device owner check implementation

Update the device owner check to compare the account id of the active
user with the account id of the owner as provided by the user manager
API. The old implementation uses the logged in user state API which for
unbeknownst reasons returns the owner as a regular user. It can also
change asynchronously at boot resulting in racy behavior as its used
right now.

BUG=702311
BUG=b:34661187
BUG=b:33488433
TEST=run cts -c android.net.wifi.cts.WifiEnterpriseConfigTest -m
testAddEapNetwork
TEST=run cts -c android.net.wifi.cts.WifiManagerTest -m
testWifiManagerNetWork

Review-Url: https://codereview.chromium.org/2756443003
Cr-Commit-Position: refs/heads/master@{#457563}
(cherry picked from commit eb260ae77f464ec4b53627278b9e842cafb3490e)
==========

[Luis Héctor Chávez, 2017-03-16 23:09:49]

lhchavez@chromium.org changed reviewers:
+ lhchavez@chromium.org

[Luis Héctor Chávez, 2017-03-16 23:09:50]

lgtm

[abhishekbh, 2017-03-16 23:16:04]

The CQ bit was checked by abhishekbh@google.com

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

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-16 23:16:47]

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

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

CLs for remote refs other than refs/heads/master must contain NOTRY=true and
NOPRESUBMIT=true in order for the CQ to process them

[bhthompson1, 2017-03-16 23:25:50]

The CQ bit was checked by bhthompson@google.com

[commit-bot: I haz the power, 2017-03-16 23:26:46]

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-16 23:26:47]

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

[commit-bot: I haz the power, 2017-03-16 23:26:48]

CLs for remote refs other than refs/heads/master must contain NOTRY=true and
NOPRESUBMIT=true in order for the CQ to process them

[abhishekbh, 2017-03-16 23:31:07]

Description was changed from

==========
arc: Update device owner check implementation

Update the device owner check to compare the account id of the active
user with the account id of the owner as provided by the user manager
API. The old implementation uses the logged in user state API which for
unbeknownst reasons returns the owner as a regular user. It can also
change asynchronously at boot resulting in racy behavior as its used
right now.

BUG=702311
BUG=b:34661187
BUG=b:33488433
TEST=run cts -c android.net.wifi.cts.WifiEnterpriseConfigTest -m
testAddEapNetwork
TEST=run cts -c android.net.wifi.cts.WifiManagerTest -m
testWifiManagerNetWork

Review-Url: https://codereview.chromium.org/2756443003
Cr-Commit-Position: refs/heads/master@{#457563}
(cherry picked from commit eb260ae77f464ec4b53627278b9e842cafb3490e)
==========

to

==========
arc: Update device owner check implementation

Update the device owner check to compare the account id of the active
user with the account id of the owner as provided by the user manager
API. The old implementation uses the logged in user state API which for
unbeknownst reasons returns the owner as a regular user. It can also
change asynchronously at boot resulting in racy behavior as its used
right now.

BUG=702311
BUG=b:34661187
BUG=b:33488433
TEST=run cts -c android.net.wifi.cts.WifiEnterpriseConfigTest -m
testAddEapNetwork
TEST=run cts -c android.net.wifi.cts.WifiManagerTest -m
testWifiManagerNetWork
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2756443003
Cr-Commit-Position: refs/heads/master@{#457563}
(cherry picked from commit eb260ae77f464ec4b53627278b9e842cafb3490e)
==========

[abhishekbh, 2017-03-16 23:31:39]

The CQ bit was checked by abhishekbh@google.com

[commit-bot: I haz the power, 2017-03-16 23:32:56]

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-16 23:41:35]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489707099984030,
"parent_rev": "75ce6202acc3ef49f9c7c504c26b8f5805da97a2", "commit_rev":
"63b114848a8a4b72925cf14bbcd15d9f540271c3"}

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

Description was changed from

==========
arc: Update device owner check implementation

Update the device owner check to compare the account id of the active
user with the account id of the owner as provided by the user manager
API. The old implementation uses the logged in user state API which for
unbeknownst reasons returns the owner as a regular user. It can also
change asynchronously at boot resulting in racy behavior as its used
right now.

BUG=702311
BUG=b:34661187
BUG=b:33488433
TEST=run cts -c android.net.wifi.cts.WifiEnterpriseConfigTest -m
testAddEapNetwork
TEST=run cts -c android.net.wifi.cts.WifiManagerTest -m
testWifiManagerNetWork
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2756443003
Cr-Commit-Position: refs/heads/master@{#457563}
(cherry picked from commit eb260ae77f464ec4b53627278b9e842cafb3490e)
==========

to

==========
arc: Update device owner check implementation

Update the device owner check to compare the account id of the active
user with the account id of the owner as provided by the user manager
API. The old implementation uses the logged in user state API which for
unbeknownst reasons returns the owner as a regular user. It can also
change asynchronously at boot resulting in racy behavior as its used
right now.

BUG=702311
BUG=b:34661187
BUG=b:33488433
TEST=run cts -c android.net.wifi.cts.WifiEnterpriseConfigTest -m
testAddEapNetwork
TEST=run cts -c android.net.wifi.cts.WifiManagerTest -m
testWifiManagerNetWork
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2756443003
Cr-Commit-Position: refs/heads/master@{#457563}
(cherry picked from commit eb260ae77f464ec4b53627278b9e842cafb3490e)

Review-Url: https://codereview.chromium.org/2757783002
Cr-Commit-Position: refs/branch-heads/3029@{#251}
Cr-Branched-From:
939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}
Committed:
https://chromium.googlesource.com/chromium/src/+/63b114848a8a4b72925cf14bbcd1...
==========

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

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/63b114848a8a4b72925cf14bbcd1...