commit b41fe30cf653c4f850ed071dd1a9d86729fdbba8
author Walter Jang <wjang@google.com> Mon Jul 25 14:59:12 2016 -0700
committer Walter Jang <wjang@google.com> Mon Jul 25 15:05:29 2016 -0700
tree 22dbe17e07633958171ae605702693c6ba8f3c7d
parent b4e7e9fe440e194dcdc393bdd93fc514bf3fb31d

Hide "Remove contacts" menu item for empty groups

Bug 30369711

Change-Id: I0e80955a3ea657e2bac67c7e39ae4b2755badae5

src/com/android/contacts/activities/GroupMembersActivity.java

diff --git a/src/com/android/contacts/activities/GroupMembersActivity.java b/src/com/android/contacts/activities/GroupMembersActivity.java
index f687728..edd93dd 100644
--- a/src/com/android/contacts/activities/GroupMembersActivity.java
+++ b/src/com/android/contacts/activities/GroupMembersActivity.java
@@ -336,13 +336,19 @@
         setVisible(menu, R.id.menu_add, isGroupEditable && !isSelectionMode);
         setVisible(menu, R.id.menu_rename_group, !isGroupReadOnly && !isSelectionMode);
         setVisible(menu, R.id.menu_delete_group, !isGroupReadOnly && !isSelectionMode);
-        setVisible(menu, R.id.menu_edit_group, isGroupEditable && !mIsEditMode && !isSelectionMode);
+        setVisible(menu, R.id.menu_edit_group, isGroupEditable && !mIsEditMode && !isSelectionMode
+                && !isGroupEmpty());
         setVisible(menu, R.id.menu_remove_from_group, isGroupEditable && isSelectionMode &&
                 !mIsEditMode);
 
         return true;
     }
 
+    private boolean isGroupEmpty() {
+        return mMembersFragment != null && mMembersFragment.getAdapter() != null &&
+                mMembersFragment.getAdapter().isEmpty();
+    }
+
     private static void setVisible(Menu menu, int id, boolean visible) {
         final MenuItem menuItem = menu.findItem(id);
         if (menuItem != null) {