Remove lock screen item from menu and disable screen lock on guest session.

Remove lock screen item from menu and disable screen lock on guest session.

BUG=chromium-os:23764
TEST=Log in as guest, attempt to lock screen, nothing should happen.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=113485

[flackr, 2011-12-07 16:08:50]

Achuith, I see you added the menu item to lock the screen. Does it make sense to
only show this for regular users? Currently the WebUI screen locker doesn't
support unlock_on_input which means we can't "lock" (black out) the screen for
guest on lock request.

[achuithb, 2011-12-07 18:46:02]

LGTM provided nit below is taken care of.

http://codereview.chromium.org/8842002/diff/1/chrome/browser/chromeos/login/s...
File chrome/browser/chromeos/login/screen_locker.cc (right):

http://codereview.chromium.org/8842002/diff/1/chrome/browser/chromeos/login/s...
chrome/browser/chromeos/login/screen_locker.cc:359: VLOG(1) << "Show: Refusing
to lock screen for guest account.";
Could you please change this to DVLOG? And change the other VLOGs in this
function too?

[achuithb, 2011-12-07 18:47:25]

Thanks for fixing this, Robert!

On 2011/12/07 18:46:02, achuith.bhandarkar wrote:
> LGTM provided nit below is taken care of.
> 
>
http://codereview.chromium.org/8842002/diff/1/chrome/browser/chromeos/login/s...
> File chrome/browser/chromeos/login/screen_locker.cc (right):
> 
>
http://codereview.chromium.org/8842002/diff/1/chrome/browser/chromeos/login/s...
> chrome/browser/chromeos/login/screen_locker.cc:359: VLOG(1) << "Show: Refusing
> to lock screen for guest account.";
> Could you please change this to DVLOG? And change the other VLOGs in this
> function too?

[flackr, 2011-12-07 19:05:50]

http://codereview.chromium.org/8842002/diff/1/chrome/browser/chromeos/login/s...
File chrome/browser/chromeos/login/screen_locker.cc (right):

http://codereview.chromium.org/8842002/diff/1/chrome/browser/chromeos/login/s...
chrome/browser/chromeos/login/screen_locker.cc:359: VLOG(1) << "Show: Refusing
to lock screen for guest account.";
On 2011/12/07 18:46:02, achuith.bhandarkar wrote:
> Could you please change this to DVLOG? And change the other VLOGs in this
> function too? 

Done.

[commit-bot: I haz the power, 2011-12-07 21:06:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/flackr@chromium.org/8842002/1002

[commit-bot: I haz the power, 2011-12-07 22:27:34]

Change committed as 113485