mash: Move user_switch_util to chrome.

mash: Move user_switch_util to chrome.

Move nearer use in session_state_delegate_chromeos.cc.

A temporary/punt measure to avoid some minor issues.
It'll need to communicate w/ash for share/capture stuff.
Avoids SystemTray access and dialog conversion tasks.

BUG=619636
TEST=compiles; unit tests; no behavior changes.
R=jamescook@chromium.org
TBR=sky@chromium.org

Committed: https://crrev.com/47b22eb9317cfd04c39dc20aaa94418aa4749b37
Cr-Commit-Position: refs/heads/master@{#405288}

[msw, 2016-07-13 19:13:46]

Description was changed from

==========
mash: Move user_switch_util to chrome.

Move nearer its only usage in session_state_delegate_chromeos.cc.
This move is a temporary measure to avoids some minor weird issues.
(it'll need to communicate w/ash over mojo for share/capture stuff)
(avoids minor SystemTray access and dialog creation conversion tasks)

BUG=619636
TEST=compiles; unit tests; no behavior changes.
R=jamescook@chromium.org
==========

to

==========
mash: Move user_switch_util to chrome.

Move nearer use in session_state_delegate_chromeos.cc.

A temporary/punt measure to avoid some minor issues.
It'll need to communicate w/ash for share/capture stuff.
Avoids SystemTray access and dialog conversion tasks.

BUG=619636
TEST=compiles; unit tests; no behavior changes.
R=jamescook@chromium.org
==========

[msw, 2016-07-13 19:15:22]

Hey James, please take a look; thanks!

[msw, 2016-07-13 19:15:27]

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

[commit-bot: I haz the power, 2016-07-13 19:16:16]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[James Cook, 2016-07-13 19:26:06]

LGTM

https://codereview.chromium.org/2144083002/diff/1/chrome/browser/ui/ash/multi...
File chrome/browser/ui/ash/multi_user/user_switch_util.cc (right):

https://codereview.chromium.org/2144083002/diff/1/chrome/browser/ui/ash/multi...
chrome/browser/ui/ash/multi_user/user_switch_util.cc:32: explicit
DesktopCastingWarningView(base::Callback<void()> on_accept);
Thanks for fixing. (Out of curiosity, do you use "git cl lint"? I didn't used
to, but I turned it on recently in Sublime. It's sometimes useful, sometimes
annoying.)

[msw, 2016-07-13 19:31:52]

https://codereview.chromium.org/2144083002/diff/1/chrome/browser/ui/ash/multi...
File chrome/browser/ui/ash/multi_user/user_switch_util.cc (right):

https://codereview.chromium.org/2144083002/diff/1/chrome/browser/ui/ash/multi...
chrome/browser/ui/ash/multi_user/user_switch_util.cc:32: explicit
DesktopCastingWarningView(base::Callback<void()> on_accept);
On 2016/07/13 19:26:06, James Cook wrote:
> Out of curiosity, do you use "git cl lint"?

Nope; just saw this lint error in presubmit.

[msw, 2016-07-13 19:32:26]

Description was changed from

==========
mash: Move user_switch_util to chrome.

Move nearer use in session_state_delegate_chromeos.cc.

A temporary/punt measure to avoid some minor issues.
It'll need to communicate w/ash for share/capture stuff.
Avoids SystemTray access and dialog conversion tasks.

BUG=619636
TEST=compiles; unit tests; no behavior changes.
R=jamescook@chromium.org
==========

to

==========
mash: Move user_switch_util to chrome.

Move nearer use in session_state_delegate_chromeos.cc.

A temporary/punt measure to avoid some minor issues.
It'll need to communicate w/ash for share/capture stuff.
Avoids SystemTray access and dialog conversion tasks.

BUG=619636
TEST=compiles; unit tests; no behavior changes.
R=jamescook@chromium.org
TBR=sky@chromium.org
==========

[msw, 2016-07-13 19:32:26]

msw@chromium.org changed reviewers:
+ sky@chromium.org

[msw, 2016-07-13 19:32:41]

+TBR Scott for simple chrome/* changes.

[commit-bot: I haz the power, 2016-07-13 20:00:03]

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

[commit-bot: I haz the power, 2016-07-13 20:00:03]

Dry run: This issue passed the CQ dry run.

[msw, 2016-07-13 21:09:59]

The CQ bit was checked by msw@chromium.org

[commit-bot: I haz the power, 2016-07-13 21:11:24]

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-13 21:27:08]

Description was changed from

==========
mash: Move user_switch_util to chrome.

Move nearer use in session_state_delegate_chromeos.cc.

A temporary/punt measure to avoid some minor issues.
It'll need to communicate w/ash for share/capture stuff.
Avoids SystemTray access and dialog conversion tasks.

BUG=619636
TEST=compiles; unit tests; no behavior changes.
R=jamescook@chromium.org
TBR=sky@chromium.org
==========

to

==========
mash: Move user_switch_util to chrome.

Move nearer use in session_state_delegate_chromeos.cc.

A temporary/punt measure to avoid some minor issues.
It'll need to communicate w/ash for share/capture stuff.
Avoids SystemTray access and dialog conversion tasks.

BUG=619636
TEST=compiles; unit tests; no behavior changes.
R=jamescook@chromium.org
TBR=sky@chromium.org
==========

[commit-bot: I haz the power, 2016-07-13 21:27:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-13 21:30:20]

Description was changed from

==========
mash: Move user_switch_util to chrome.

Move nearer use in session_state_delegate_chromeos.cc.

A temporary/punt measure to avoid some minor issues.
It'll need to communicate w/ash for share/capture stuff.
Avoids SystemTray access and dialog conversion tasks.

BUG=619636
TEST=compiles; unit tests; no behavior changes.
R=jamescook@chromium.org
TBR=sky@chromium.org
==========

to

==========
mash: Move user_switch_util to chrome.

Move nearer use in session_state_delegate_chromeos.cc.

A temporary/punt measure to avoid some minor issues.
It'll need to communicate w/ash for share/capture stuff.
Avoids SystemTray access and dialog conversion tasks.

BUG=619636
TEST=compiles; unit tests; no behavior changes.
R=jamescook@chromium.org
TBR=sky@chromium.org

Committed: https://crrev.com/47b22eb9317cfd04c39dc20aaa94418aa4749b37
Cr-Commit-Position: refs/heads/master@{#405288}
==========

[commit-bot: I haz the power, 2016-07-13 21:30:21]

Patchset 1 (id:??) landed as
https://crrev.com/47b22eb9317cfd04c39dc20aaa94418aa4749b37
Cr-Commit-Position: refs/heads/master@{#405288}