Bug 1050780 - Avoid disabled items in GeckoMenu's adapter. r=margaret, a=sledru
authorLucas Rocha <lucasr@mozilla.com>
Tue, 02 Sep 2014 10:47:16 +0100
changeset 216653 7cf512b6b64c
parent 216652 764591e4e7f3
child 216654 4815ff146c57
push id3861
push userryanvm@gmail.com
push date2014-09-03 15:49 +0000
treeherdermozilla-beta@4815ff146c57 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmargaret, sledru
bugs1050780
milestone33.0
Bug 1050780 - Avoid disabled items in GeckoMenu's adapter. r=margaret, a=sledru
mobile/android/base/menu/GeckoMenu.java
mobile/android/base/resources/drawable/action_bar_button.xml
--- a/mobile/android/base/menu/GeckoMenu.java
+++ b/mobile/android/base/menu/GeckoMenu.java
@@ -782,17 +782,19 @@ public class GeckoMenu extends ListView
         public boolean areAllItemsEnabled() {
             // Setting this to true is a workaround to fix disappearing
             // dividers in the menu (bug 963249).
             return true;
         }
 
         @Override
         public boolean isEnabled(int position) {
-            return getItem(position).isEnabled();
+            // Setting this to true is a workaround to fix disappearing
+            // dividers in the menu in L (bug 1050780).
+            return true;
         }
 
         public void addMenuItem(GeckoMenuItem menuItem) {
             if (mItems.contains(menuItem))
                 return;
 
             // Insert it in proper order.
             int index = 0;
--- a/mobile/android/base/resources/drawable/action_bar_button.xml
+++ b/mobile/android/base/resources/drawable/action_bar_button.xml
@@ -1,16 +1,17 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!-- This Source Code Form is subject to the terms of the Mozilla Public
    - 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/. -->
 
 <selector xmlns:android="http://schemas.android.com/apk/res/android">
 
-    <item android:state_pressed="true">
+    <item android:state_pressed="true"
+          android:state_enabled="true">
         <shape>
             <solid android:color="@color/highlight" />
         </shape>
     </item>
 
     <item android:state_focused="true"
           android:state_pressed="false">
         <shape>