Add remove non-device-owner user removal confirmation

Add remove non-device-owner user removal confirmation

Just like removing supervised user, there will be a removing warning UI for removing non-device-owner (since device owner doesn't show "Remove this person" entry).

The removing warning message right now is just a placeholder. Will update it later once I hear it back from the discussion.

BUG=611904
TEST=device test

Committed: https://crrev.com/636ea5a0427aa08d516eca5fa1171db36fb7e391
Cr-Commit-Position: refs/heads/master@{#408646}

[Qiang(Joe) Xu, 2016-07-28 21:18:28]

Description was changed from

==========
Add remove non-device-owner user removal confirmation

BUG=611904
TEST=device test
==========

to

==========
Add remove non-device-owner user removal confirmation

Just like removing supervised user, there will be a removing warning UI for
removing non-device-owner (since device owner doesn't show "Remove this person"
entry).

The removing warning message right now is just a placeholder. Will update it
later once I hear it back from the discussion.

BUG=611904
TEST=device test
==========

[Qiang(Joe) Xu, 2016-07-28 21:18:28]

warx@chromium.org changed reviewers:
+ alemate@chromium.org

[Qiang(Joe) Xu, 2016-07-28 21:18:52]

Hi Alex, could you help take a look at it? Thanks!

[Alexander Alekseev, 2016-07-28 21:32:00]

lgtm

[Qiang(Joe) Xu, 2016-07-28 21:40:27]

The CQ bit was checked by warx@chromium.org

[commit-bot: I haz the power, 2016-07-28 21:40:47]

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, 2016-07-28 22:24:56]

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

[commit-bot: I haz the power, 2016-07-28 22:24:57]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Qiang(Joe) Xu, 2016-07-29 03:23:57]

The CQ bit was checked by warx@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-29 03:24:15]

Dry run: 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, 2016-07-29 04:00:23]

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

[commit-bot: I haz the power, 2016-07-29 04:00:24]

Dry run: This issue passed the CQ dry run.

[Qiang(Joe) Xu, 2016-07-29 04:02:22]

The CQ bit was checked by warx@chromium.org

[Qiang(Joe) Xu, 2016-07-29 04:02:23]

The patchset sent to the CQ was uploaded after l-g-t-m from alemate@chromium.org
Link to the patchset: https://codereview.chromium.org/2189163002/#ps20001
(title: "fix tests")

[commit-bot: I haz the power, 2016-07-29 04:02:32]

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, 2016-07-29 04:07:59]

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

[commit-bot: I haz the power, 2016-07-29 04:08:00]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Qiang(Joe) Xu, 2016-07-29 04:23:18]

warx@chromium.org changed reviewers:
+ xiyuan@chromium.org

[Qiang(Joe) Xu, 2016-07-29 04:25:24]

Hi xiyuan@, can you help review on the new diff? It is missing owner review.
Thanks!

[xiyuan, 2016-07-29 13:34:44]

lgtm

[Qiang(Joe) Xu, 2016-07-29 15:58:50]

The CQ bit was checked by warx@chromium.org

[commit-bot: I haz the power, 2016-07-29 15:59:09]

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, 2016-07-29 16:02:44]

Description was changed from

==========
Add remove non-device-owner user removal confirmation

Just like removing supervised user, there will be a removing warning UI for
removing non-device-owner (since device owner doesn't show "Remove this person"
entry).

The removing warning message right now is just a placeholder. Will update it
later once I hear it back from the discussion.

BUG=611904
TEST=device test
==========

to

==========
Add remove non-device-owner user removal confirmation

Just like removing supervised user, there will be a removing warning UI for
removing non-device-owner (since device owner doesn't show "Remove this person"
entry).

The removing warning message right now is just a placeholder. Will update it
later once I hear it back from the discussion.

BUG=611904
TEST=device test
==========

[commit-bot: I haz the power, 2016-07-29 16:02:45]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-29 16:04:27]

Description was changed from

==========
Add remove non-device-owner user removal confirmation

Just like removing supervised user, there will be a removing warning UI for
removing non-device-owner (since device owner doesn't show "Remove this person"
entry).

The removing warning message right now is just a placeholder. Will update it
later once I hear it back from the discussion.

BUG=611904
TEST=device test
==========

to

==========
Add remove non-device-owner user removal confirmation

Just like removing supervised user, there will be a removing warning UI for
removing non-device-owner (since device owner doesn't show "Remove this person"
entry).

The removing warning message right now is just a placeholder. Will update it
later once I hear it back from the discussion.

BUG=611904
TEST=device test

Committed: https://crrev.com/636ea5a0427aa08d516eca5fa1171db36fb7e391
Cr-Commit-Position: refs/heads/master@{#408646}
==========

[commit-bot: I haz the power, 2016-07-29 16:04:28]

Patchset 2 (id:??) landed as
https://crrev.com/636ea5a0427aa08d516eca5fa1171db36fb7e391
Cr-Commit-Position: refs/heads/master@{#408646}