Bug 1330009 - Remove unused setToolBarButtonsAlpha() method. r=sebastian
authorFrancisco Aguiar <franciscomaguiar@gmail.com>
Mon, 16 Jan 2017 00:57:53 +0000
changeset 374699 3f18ac5367e0a6f482299f829b85eca22a91b33c
parent 374698 3e2eddbfeaef6075cf337bdcd2997a8238e989c9
child 374700 529788a8225ac3417669cea916a32c9a906ccaab
push id6996
push userjlorenzo@mozilla.com
push dateMon, 06 Mar 2017 20:48:21 +0000
treeherdermozilla-beta@d89512dab048 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssebastian
bugs1330009
milestone53.0a1
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
Bug 1330009 - Remove unused setToolBarButtonsAlpha() method. r=sebastian MozReview-Commit-ID: A7PIZDck82h
mobile/android/base/java/org/mozilla/gecko/toolbar/BrowserToolbar.java
mobile/android/base/java/org/mozilla/gecko/toolbar/BrowserToolbarTablet.java
--- a/mobile/android/base/java/org/mozilla/gecko/toolbar/BrowserToolbar.java
+++ b/mobile/android/base/java/org/mozilla/gecko/toolbar/BrowserToolbar.java
@@ -651,23 +651,16 @@ public abstract class BrowserToolbar ext
             }
         }
 
         if (needsNewFocus) {
             requestFocus();
         }
     }
 
-    public void setToolBarButtonsAlpha(float alpha) {
-        ViewHelper.setAlpha(tabsCounter, alpha);
-        if (!HardwareUtils.isTablet()) {
-            ViewHelper.setAlpha(menuIcon, alpha);
-        }
-    }
-
     public void onEditSuggestion(String suggestion) {
         if (!isEditing()) {
             return;
         }
 
         urlEditLayout.onEditSuggestion(suggestion);
     }
 
--- a/mobile/android/base/java/org/mozilla/gecko/toolbar/BrowserToolbarTablet.java
+++ b/mobile/android/base/java/org/mozilla/gecko/toolbar/BrowserToolbarTablet.java
@@ -162,22 +162,16 @@ class BrowserToolbarTablet extends Brows
     }
 
     @Override
     public void triggerTabsPanelTransition(final PropertyAnimator animator, final boolean areTabsShown) {
         // Do nothing.
     }
 
     @Override
-    public void setToolBarButtonsAlpha(float alpha) {
-        // Do nothing.
-    }
-
-
-    @Override
     public void startEditing(final String url, final PropertyAnimator animator) {
         // We already know the forward button state - no need to store it here.
         backButtonWasEnabledOnStartEditing = backButton.isEnabled();
 
         backButton.setEnabled(false);
         forwardButton.setEnabled(false);
 
         super.startEditing(url, animator);