[cros] Error bubble on login is displayed to the left of Gaia frame.

chrome/browser/resources/chromeos/login/bubble.js

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @fileoverview Bubble implementation.
  */
 
 // TODO(xiyuan): Move this into shared.
 cr.define('cr.ui', function() {
   /**
    * Creates a bubble div.
    * @constructor
    * @extends {HTMLDivElement}
    */
   var Bubble = cr.ui.define('div');
 
+  /**
+   * Bubble attachment side.
+   * @enum {string}
+   */
+  Bubble.Attachment = {
+    RIGHT: 'bubble-right',
+    LEFT: 'bubble-left',
+    TOP: 'bubble-top',
+    BOTTOM: 'bubble-bottom'
+  };
+
   Bubble.prototype = {
     __proto__: HTMLDivElement.prototype,
 
     // Anchor element
     anchor_: undefined,
 
     /** @inheritDoc */
     decorate: function() {
       this.ownerDocument.addEventListener('click',
                                           this.handleDocClick_.bind(this));
       this.ownerDocument.addEventListener('keydown',
                                           this.handleDocKeyDown_.bind(this));
       this.addEventListener('webkitTransitionEnd',
                             this.handleTransitionEnd_.bind(this));
     },
 
     /**
+     * Sets the attachment of the bubble.
+     * @param {!Attachment} attachment Bubble attachment.
+     */
+    setAttachment_: function (attachment) {
+      for (var k in Bubble.Attachment) {
+        var v = Bubble.Attachment[k];
+        this.classList[v == attachment ? 'add' : 'remove'](v);
+      }
+    },
+
+    /**
      * Shows the bubble for given anchor element.
-     * @param {number} x X position of bubble's reference point.
-     * @param {number} y Y position of bubble's reference point.
+     * @param {!Object} pos Bubble position (left, top, right, bottom).
      * @param {HTMLElement} content Content to show in bubble.
+     * @param {!Attachment} attachment Bubble attachment (on which side of the
+     *     specified position it should be displayed).
      * @public
      */
-    showContentAt: function(x, y, content) {
+    showContentAt: function(pos, content, attachment) {
       const ARROW_OFFSET = 14;
 
-      var anchorX = x - ARROW_OFFSET;
-      var anchorY = y;
+      // Adjust position so that arrow points exactly at |pos|.
+      if (attachment == Bubble.Attachment.TOP ||
+          attachment == Bubble.Attachment.BOTTOM) {
+        if (pos.left)
+          pos.left -= ARROW_OFFSET;
+      } else {
+        if (pos.top)
+          pos.top -= ARROW_OFFSET;
+      }
 
-      this.style.left = anchorX + 'px';
-      this.style.top = anchorY + 'px';
+      for (var k in pos) {
+        if (typeof pos[k] == 'number')
+          this.style[k] = pos[k] + 'px';
+      }
 
       this.innerHTML = '';
       this.appendChild(content);
+      this.setAttachment_(attachment);
       this.hidden = false;
       this.classList.remove('faded');
     },
 
     /**
      * Shows the bubble for given anchor element.
      * @param {!HTMLElement} el Anchor element of the bubble.
      * @param {HTMLElement} content Content to show in bubble.
+     * @param {!Attachment} attachment Bubble attachment (on which side of the
+     *     element it should be displayed).
      * @public
      */
-    showContentForElement: function(el, content) {
-      const HORIZONTAL_PADDING = 10;
-      const VERTICAL_PADDING = 5;
+    showContentForElement: function(el, content, attachment) {
+      const OFFSET = 10;
+      const PADDING = 5;
 
-      var elementOrigin = cr.ui.login.DisplayManager.getOffset(el);
-      var anchorX = elementOrigin.left + HORIZONTAL_PADDING;
-      var anchorY = elementOrigin.top + el.offsetHeight + VERTICAL_PADDING;
+      var origin = cr.ui.login.DisplayManager.getPosition(el);
+
+      // Adjust the side w.r.to document direction.
+      var side = attachment;
+      if (isRTL()) {
+        if (side == Bubble.Attachment.RIGHT)
+          side = Bubble.Attachment.LEFT;
+        else if (side == Bubble.Attachment.LEFT)
+          side = Bubble.Attachment.RIGHT;
+      }
+
+      var pos = {};
+      switch (side) {
+        case Bubble.Attachment.TOP:
+          pos.left = origin.left + OFFSET;
+          pos.bottom = origin.bottom + el.offsetHeight + PADDING;

[xiyuan, 2012/02/02 22:43:52]

Should this be
pos.bottom = origin.top - PADDING
to make bubble sit on top of the anchor element?

[Ivan Korotkov, 2012/02/03 10:33:04]

This would not do the right thing. Imagine if origin.top is PADDING (anchor at
nearly top of the page), this would set pos.bottom = 0 (i.e., bubble at the
bottom of the page).

+          break;
+        case Bubble.Attachment.RIGHT:
+          pos.top = origin.top + OFFSET;
+          pos.left = origin.left + el.offsetWidth + PADDING;
+          break;
+        case Bubble.Attachment.BOTTOM:
+          pos.left = origin.left + OFFSET;
+          pos.top = origin.top + el.offsetHeight + PADDING;
+          break;
+        case Bubble.Attachment.LEFT:
+          pos.top = origin.top + OFFSET;
+          pos.right = origin.right + el.offsetWidth + PADDING;

[xiyuan, 2012/02/02 22:43:52]

pos.right = origin.left - PADDING?

[Ivan Korotkov, 2012/02/03 10:33:04]

Same here.

+          break;
+      }
 
       this.anchor_ = el;
-      this.showContentAt(anchorX, anchorY, content);
+      this.showContentAt(pos, content, attachment);
     },
 
     /**
      * Shows the bubble for given anchor element.
      * @param {!HTMLElement} el Anchor element of the bubble.
      * @param {string} text Text content to show in bubble.
+     * @param {!Attachment} attachment Bubble attachment (on which side of the
+     *     element it should be displayed).
      * @public
      */
-    showTextForElement: function(el, text) {
+    showTextForElement: function(el, text, attachment) {
       var span = this.ownerDocument.createElement('span');
       span.textContent = text;
-      this.showContentForElement(el, span);
+      this.showContentForElement(el, span, attachment);
     },
 
     /**
      * Hides the bubble.
      */
     hide: function() {
       if (!this.classList.contains('faded'))
         this.classList.add('faded');
     },
 
@@ skipping 35 matching lines @@
     handleDocKeyDown_: function(e) {
       if (!this.hidden)
         this.hide();
     }
   };
 
   return {
     Bubble: Bubble
   };
 });