Bug 518014: preference about:fennec button doesn't show webpage, r=gavin

mobile/chrome/content/browser-ui.js

@@ -537,10 +537,6 @@ var BrowserUI = {
     this.goToURI(list.selectedItem.value);
   },
 
-  showHistory : function() {
-    // XXX Fix me with a real UI
-  },
-
   showBookmarks : function () {
     BookmarkList.show();
   },
@@ -549,6 +545,8 @@ var BrowserUI = {
     aURI = aURI || "about:blank";
     let tab = Browser.addTab(aURI, true);
 
+    this.hidePanel();
+    
     if (aURI == "about:blank") {
       this.showToolbar(true);
       this.showAutoComplete();

mobile/chrome/content/browser.xul

@@ -358,9 +358,9 @@
 #ifdef MOZ_OFFICIAL_BRANDING
 # these two point to the same page, this just matters for what shows up in the
 # URL bar
-                        oncommand="Browser.addTab('about:firefox', true);"/>
+                        oncommand="BrowserUI.newTab('about:firefox', true);"/>
 #else
-                        oncommand="Browser.addTab('about:fennec', true);"/>
+                        oncommand="BrowserUI.newTab('about:fennec', true);"/>
 #endif
               </setting>
               <setting id="prefs-uilanguage" title="&language.title;" type="control">

mobile/chrome/content/downloads.js

@@ -380,10 +380,8 @@ var DownloadsView = {
 
   showPage: function dv_showPage(aItem) {
     let uri = this._getReferrerOrSource(aItem);
-    if (uri) {
+    if (uri)
       BrowserUI.newTab(uri);
-      BrowserUI.hidePanel();
-    }
   },
 
   observe: function (aSubject, aTopic, aData) {

mobile/chrome/content/extensions.js

@@ -502,10 +502,8 @@ var ExtensionsView = {
 
   showPage: function ev_showPage(aItem) {
     let uri = aItem.getAttribute("homepageURL");
-    if (uri) {
+    if (uri)
       BrowserUI.newTab(uri);
-      BrowserUI.hidePanel();
-    }
   },
 
   resetSearch: function ev_resetSearch() {