backout 7c458c70cd5c for suspected talos regression
authorMargaret Leibovic <margaret.leibovic@gmail.com>
Thu, 13 Mar 2014 07:48:57 -0700
changeset 191683 80fda301471ebba05b8bc00742da9a4eb8a3d665
parent 191682 f547db6d8bc871d0ded1a832d91c62b36cdda7f1
child 191684 02be46ac2b5547ea7aee6881751cc4dc4359ba4b
push id474
push userasasaki@mozilla.com
push dateMon, 02 Jun 2014 21:01:02 +0000
treeherdermozilla-release@967f4cf1b31c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
milestone30.0a1
backs out7c458c70cd5cfe1238bfe0ee6615273cccaebdcb
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
backout 7c458c70cd5c for suspected talos regression
mobile/android/base/BrowserApp.java
--- a/mobile/android/base/BrowserApp.java
+++ b/mobile/android/base/BrowserApp.java
@@ -1688,32 +1688,26 @@ abstract public class BrowserApp extends
             mLayerView.getLayerMarginsAnimator().showMargins(true);
         }
 
         if (mHomePager == null) {
             final ViewStub homePagerStub = (ViewStub) findViewById(R.id.home_pager_stub);
             mHomePager = (HomePager) homePagerStub.inflate();
 
             final HomeBanner homeBanner = (HomeBanner) findViewById(R.id.home_banner);
-
-            // Never show the home banner in guest mode.
-            if (GeckoProfile.get(this).inGuestMode()) {
-                mHomePagerContainer.removeView(homeBanner);
-            } else {
-                mHomePager.setBanner(homeBanner);
-
-                // Remove the banner from the view hierarchy if it is dismissed.
-                homeBanner.setOnDismissListener(new HomeBanner.OnDismissListener() {
-                    @Override
-                    public void onDismiss() {
-                        mHomePager.setBanner(null);
-                        mHomePagerContainer.removeView(homeBanner);
-                    }
-                });
-            }
+            mHomePager.setBanner(homeBanner);
+
+            // Remove the banner from the view hierarchy if it is dismissed.
+            homeBanner.setOnDismissListener(new HomeBanner.OnDismissListener() {
+                @Override
+                public void onDismiss() {
+                    mHomePager.setBanner(null);
+                    mHomePagerContainer.removeView(homeBanner);
+                }
+            });
         }
 
         mHomePagerContainer.setVisibility(View.VISIBLE);
         mHomePager.load(getSupportLoaderManager(),
                         getSupportFragmentManager(),
                         pageId, animator);
 
         // Hide the web content so it cannot be focused by screen readers.