Backout df1cc5afcff9 (bug 608268) for Android keyboard regression. r=me
authorMatt Brubeck <mbrubeck@mozilla.com>
Thu, 09 Dec 2010 22:54:44 -0800
changeset 67105 2ba6c90b7fb335083b609f03f5ffb839277f98bb
parent 67104 4fb558cc87d18f3d75d543870d2d3bc341ea82e4
child 67106 f173c4baee6303331d44f2ea662883f34820beeb
push id1
push userroot
push dateTue, 26 Apr 2011 22:38:44 +0000
treeherdermozilla-beta@bfdb6e623a36 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersme
bugs608268
Backout df1cc5afcff9 (bug 608268) for Android keyboard regression. r=me
mobile/chrome/content/browser-ui.js
mobile/chrome/content/browser.js
--- a/mobile/chrome/content/browser-ui.js
+++ b/mobile/chrome/content/browser-ui.js
@@ -2797,32 +2797,32 @@ var SharingUI = {
                              .getService(Ci.nsIExternalProtocolService);
         extProtocolSvc.loadUrl(uri);
       }
     },
     {
       name: "Twitter",
       callback: function callback(aURL, aTitle) {
         let url = "http://twitter.com/home?status=" + encodeURIComponent((aTitle ? aTitle+": " : "")+aURL);
-        BrowserUI.newTab(url, Browser.selectedTab);
+        Browser.addTab(url, true, Browser.selectedTab);
       }
     },
     {
       name: "Google Reader",
       callback: function callback(aURL, aTitle) {
         let url = "http://www.google.com/reader/link?url=" + encodeURIComponent(aURL) +
                   "&title=" + encodeURIComponent(aTitle);
-        BrowserUI.addTab(url, Browser.selectedTab);
+        Browser.addTab(url, true, Browser.selectedTab);
       }
     },
     {
       name: "Facebook",
       callback: function callback(aURL, aTitle) {
         let url = "http://www.facebook.com/share.php?u=" + encodeURIComponent(aURL);
-        BrowserUI.newTab(url, Browser.selectedTab);
+        Browser.addTab(url, true, Browser.selectedTab);
       }
     }
   ]
 };
 
 var BadgeHandlers = {
   _handlers: [
     {
--- a/mobile/chrome/content/browser.js
+++ b/mobile/chrome/content/browser.js
@@ -503,17 +503,16 @@ var Browser = {
 
     this.contentScrollboxScroller.scrollTo(x.value, Number.MAX_VALUE);
     this.pageScrollboxScroller.scrollTo(0, Number.MAX_VALUE);
   },
 
   hideSidebars: function scrollSidebarsOffscreen() {
     let rect = Elements.browsers.getBoundingClientRect();
     this.controlsScrollboxScroller.scrollBy(Math.round(rect.left), 0);
-    this.tryUnfloatToolbar();
   },
 
   hideTitlebar: function hideTitlebar() {
     let rect = Elements.browsers.getBoundingClientRect();
     this.pageScrollboxScroller.scrollBy(0, Math.round(rect.top));
     this.tryUnfloatToolbar();
   },