login-webui: Close any bubble when window loses focus.

login-webui: Close any bubble when window loses focus.

If there is an error bubble on the login screen, and the user clicks
on the status-tray, then the bubble should close. Instead of sending
complicated messages from the UI to the login webui code, it is much
simpler to just close the bubbles when the login window loses focus.

BUG=127092
TEST=manually

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=148866

[sadrul, 2012-07-26 16:44:32]

ping!

[sadrul, 2012-07-27 23:19:46]

On 2012/07/26 16:44:32, sadrul wrote:
> ping!

ping!

[Nikita (slow), 2012-07-28 01:09:41]

lgtm

sorry for delay, was at the 2 day offsite.

[commit-bot: I haz the power, 2012-07-28 02:01:31]

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

[commit-bot: I haz the power, 2012-07-28 02:01:34]

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

Running presubmit commit checks ...

** Presubmit Messages **
See the JavaScript style guide at
http://www.chromium.org/developers/web-development-style-guide#TOC-JavaScript
and if you have any feedback about the JavaScript PRESUBMIT check, contact
tbreisacher@chromium.org

** Presubmit Warnings **
Found JavaScript style violations in
chrome/browser/resources/chromeos/login/bubble.js:
  line 95: Use var instead of const.
      const ARROW_OFFSET = 25;
      ^^^^^
  line 96: Use var instead of const.
      const DEFAULT_PADDING = 18;
      ^^^^^
  line 50: E0001: Extra space after "function"
    setAttachment_: function (attachment) {
                            ^
  line 91: E0200: Invalid JsDoc tag: public
     * @public
       ^^^^^^^
  line 103: E0001: Extra space after "["
          [ Math.min(ARROW_OFFSET, el.offsetWidth/2),
           ^
  line 103: E0002: Missing space before "/"
          [ Math.min(ARROW_OFFSET, el.offsetWidth/2),
                                                 ^
  line 103: E0002: Missing space after "/"
          [ Math.min(ARROW_OFFSET, el.offsetWidth/2),
                                                 ^
  line 104: E0002: Missing space before "/"
            Math.min(ARROW_OFFSET, el.offsetHeight/2) ] :
                                                  ^
  line 104: E0002: Missing space after "/"
            Math.min(ARROW_OFFSET, el.offsetHeight/2) ] :
                                                  ^
  line 104: E0001: Extra space before "]"
            Math.min(ARROW_OFFSET, el.offsetHeight/2) ] :
                                                     ^
  line 105: E0001: Extra space after "["
          [ opt_offset, opt_offset ];
           ^
  line 105: E0001: Extra space before "]"
          [ opt_offset, opt_offset ];
                                  ^
  line 164: E0200: Invalid JsDoc tag: public
     * @public
       ^^^^^^^

Presubmit checks took 1.3s to calculate.

Was the presubmit check useful? Please send feedback & hate mail to
maruel@chromium.org!

[Nikita (slow), 2012-07-28 02:02:29]

Apparently there were no changes to bubble.js since JS-style checker was
introduced. Sorry about that.

[sadrul, 2012-07-28 02:14:37]

On 2012/07/28 02:02:29, Nikita Kostylev wrote:
> Apparently there were no changes to bubble.js since JS-style checker was
> introduced. Sorry about that.

Oh, no worries. I landed manually. Thanks for review! :)