Bug 612230 - The bookmarks toolbar is empty in RTL Firefox if even one bookmark overflows the width of the toolbar [FOLLOWUP] (r=mak77, a=blocking)
authormano@mozilla.com
Tue, 16 Nov 2010 20:50:51 +0700
changeset 57577 622d3a8f8ab11e4c7447097aace1f03948fdc6e5
parent 57576 a1e24da8518f24c18c48222d4e7429cabfd45d8d
child 57578 bc41cecec7bd30771f6f7b909f8f909f068be59e
child 57824 98e10bbc80d3373d82b42871d5d8076a94164dc2
push id1
push usershaver@mozilla.com
push dateTue, 04 Jan 2011 17:58:04 +0000
reviewersmak77, blocking
bugs612230
milestone2.0b8pre
Bug 612230 - The bookmarks toolbar is empty in RTL Firefox if even one bookmark overflows the width of the toolbar [FOLLOWUP] (r=mak77, a=blocking)
browser/components/places/content/browserPlacesViews.js
--- a/browser/components/places/content/browserPlacesViews.js
+++ b/browser/components/places/content/browserPlacesViews.js
@@ -1050,17 +1050,17 @@ PlacesToolbar.prototype = {
 
   _updateChevronTimerCallback: function PT__updateChevronTimerCallback() {
     let scrollRect = this._rootElt.getBoundingClientRect();
     let childOverflowed = false;
     for (let i = 0; i < this._rootElt.childNodes.length; i++) {
       let child = this._rootElt.childNodes[i];
       // Once a child overflows, all the next ones will.
       if (!childOverflowed) {
-              let childRect = child.getBoundingClientRect();
+        let childRect = child.getBoundingClientRect();
         childOverflowed = this.isRTL ? (childRect.left < scrollRect.left)
                                      : (childRect.right > scrollRect.right);
                                       
       }
       child.style.visibility = childOverflowed ? "hidden" : "visible";
     }
 
     // We rebuild the chevron on popupShowing, so if it is open