commit 5e650fbaecc3bc8a3c37eb0711458037938b5dde
author chihhangchuang <chihhangchuang@google.com> Thu May 14 21:25:27 2020 +0800
committer chihhangchuang <chihhangchuang@google.com> Thu May 14 21:25:27 2020 +0800
tree 54eaa95a91278b19bf5cd608deebedb9592416eb
parent 2795478645abb878000e5f1d8143d0d67b1166fe

Cleanup for BottomActionBar

- Remove the duplicated API, we don't need two same APIs.
- Remove the duplicated back key binding, since it's done in BottomActionBarFragment.

Test: Manually
Bug: 155157385
Change-Id: I682ebe67f5b7187a1434960d553063b8c6f9d3f6

src/com/android/customization/picker/grid/GridFullPreviewFragment.java

diff --git a/src/com/android/customization/picker/grid/GridFullPreviewFragment.java b/src/com/android/customization/picker/grid/GridFullPreviewFragment.java
index 68ecc5b..2be77d6 100644
--- a/src/com/android/customization/picker/grid/GridFullPreviewFragment.java
+++ b/src/com/android/customization/picker/grid/GridFullPreviewFragment.java
@@ -137,7 +137,6 @@
 
     @Override
     protected void onBottomActionBarReady(BottomActionBar bottomActionBar) {
-        bottomActionBar.bindBackButtonToSystemBackKey(getActivity());
         bottomActionBar.showActionsOnly(APPLY);
         bottomActionBar.setActionClickListener(APPLY, v -> finishActivityWithResultOk());
         bottomActionBar.show();

src/com/android/customization/picker/theme/ThemeFragment.java

diff --git a/src/com/android/customization/picker/theme/ThemeFragment.java b/src/com/android/customization/picker/theme/ThemeFragment.java
index 84705c2..5528adb 100644
--- a/src/com/android/customization/picker/theme/ThemeFragment.java
+++ b/src/com/android/customization/picker/theme/ThemeFragment.java
@@ -165,7 +165,6 @@
     @Override
     protected void onBottomActionBarReady(BottomActionBar bottomActionBar) {
         mBottomActionBar = bottomActionBar;
-        mBottomActionBar.bindBackButtonToSystemBackKey(getActivity());
         mBottomActionBar.showActionsOnly(APPLY);
         mBottomActionBar.setActionClickListener(APPLY, v -> applyTheme());
     }