Bug 1282372 - Remove tab count transparency/flickering workaround r?sebastian draft
authorAndrzej Hunt <ahunt@mozilla.com>
Mon, 27 Jun 2016 10:22:18 +0200
changeset 381444 9300d6d253d9beea995709ac98d5c2d4c616c6c8
parent 381443 8f48d2f1fa58dbb978abc28051f3550724800b2f
child 523960 d9b3e501797e7b3704df4997ec60706be43cd061
push id21473
push userahunt@mozilla.com
push dateMon, 27 Jun 2016 08:22:41 +0000
reviewerssebastian
bugs1282372
milestone50.0a1
Bug 1282372 - Remove tab count transparency/flickering workaround r?sebastian This issue has been fixed in the latest Android N preview, we should remove the workaround to simplify the code. MozReview-Commit-ID: 6xwIA5QzzhQ
mobile/android/base/java/org/mozilla/gecko/toolbar/PhoneTabsButton.java
--- a/mobile/android/base/java/org/mozilla/gecko/toolbar/PhoneTabsButton.java
+++ b/mobile/android/base/java/org/mozilla/gecko/toolbar/PhoneTabsButton.java
@@ -2,30 +2,21 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this file,
  * You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 package org.mozilla.gecko.toolbar;
 
 import android.content.Context;
 import android.util.AttributeSet;
 
-import org.mozilla.gecko.AppConstants;
 import org.mozilla.gecko.tabs.TabCurve;
 
 public class PhoneTabsButton extends ShapedButton {
     public PhoneTabsButton(Context context, AttributeSet attrs) {
         super(context, attrs);
-
-        if (!AppConstants.Versions.preLollipop) {
-            // The Android N preview has issues rendering our tabs button during animations
-            // unless we use hardware layers, see bug 1264783. For consistency we should
-            // try and set this across all devices, however on 4.X devices the background
-            // isn't drawn when we use hardware layers - hence we need to disable this there.
-            setLayerType(LAYER_TYPE_HARDWARE, null);
-        }
     }
 
     @Override
     protected void onSizeChanged(int width, int height, int oldWidth, int oldHeight) {
         super.onSizeChanged(width, height, oldWidth, oldHeight);
 
         mPath.reset();