Don't show caps lock notification bubble on lock screen.

Don't show caps lock notification bubble on lock screen.

BUG=105280
TEST=Lock the screen and enable caps lock. No bubble indicators are shown.


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

[flackr, 2011-12-07 22:44:05]

The underlying issue would require a substantial change. To get a fix in for the
screen locker on milestone 17 I've disabled caps lock notification bubbles while
the screen is locked. I don't think there's any direct way to know which caps
lock menu button is the one being shown in the screen locker status area.

[Nikita (slow), 2011-12-08 10:42:13]

LGTM

I agree that having this notification on screen lock is not essential at this
point as this notification is shown only once per user.

[Nikita (slow), 2011-12-08 10:42:34]

lgtm

please run trybots

[commit-bot: I haz the power, 2011-12-08 15:44:54]

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

[commit-bot: I haz the power, 2011-12-08 15:44:57]

Presubmit check for 8869002-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for:
chrome/browser/chromeos/status/caps_lock_menu_button.cc

Presubmit checks took 1.0s to calculate.

[flackr, 2011-12-08 18:52:37]

+davemoore

Need owner approval, PTAL, thanks!

[DaveMoore, 2011-12-14 17:06:07]

lgtm

On 2011/12/08 18:52:37, flackr wrote:
> +davemoore
> 
> Need owner approval, PTAL, thanks!

[commit-bot: I haz the power, 2011-12-15 18:13:02]

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

[commit-bot: I haz the power, 2011-12-15 20:01:12]

Change committed as 114687