🏡 Disable BottomSheet touches while context menu is open.

chrome/android/java/src/org/chromium/chrome/browser/suggestions/SuggestionsRecyclerView.java

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chrome.browser.suggestions;
 
 import android.animation.Animator;
 import android.animation.AnimatorListenerAdapter;
 import android.animation.AnimatorSet;
 import android.animation.ObjectAnimator;
@@ skipping 13 matching lines @@
 import android.view.View;
 import android.view.ViewGroup;
 import android.view.animation.Interpolator;
 import android.view.inputmethod.EditorInfo;
 import android.view.inputmethod.InputConnection;
 
 import org.chromium.base.ApiCompatibilityUtils;
 import org.chromium.base.Callback;
 import org.chromium.chrome.R;
 import org.chromium.chrome.browser.ntp.ContextMenuManager;
-import org.chromium.chrome.browser.ntp.ContextMenuManager.TouchDisableableView;
 import org.chromium.chrome.browser.ntp.cards.CardViewHolder;
 import org.chromium.chrome.browser.ntp.cards.NewTabPageAdapter;
 import org.chromium.chrome.browser.ntp.cards.NewTabPageViewHolder;
 import org.chromium.chrome.browser.widget.displaystyle.UiConfig;
 
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
 
 /**
  * Simple wrapper on top of a RecyclerView that will acquire focus when tapped.  Ensures the
  * New Tab page receives focus when clicked.
  */
-public class SuggestionsRecyclerView extends RecyclerView implements TouchDisableableView {
+public class SuggestionsRecyclerView extends RecyclerView {
     private static final Interpolator DISMISS_INTERPOLATOR = new FastOutLinearInInterpolator();
     private static final int DISMISS_ANIMATION_TIME_MS = 300;
     /**
      * A single instance of {@link ResetForDismissCallback} that can be reused as it has no
      * state.
      */
     public static final NewTabPageViewHolder.PartialBindCallback RESET_FOR_DISMISS_CALLBACK =
             new ResetForDismissCallback();
 
     private final GestureDetector mGestureDetector;
     private final LinearLayoutManager mLayoutManager;
 
     /**
      * Total height of the items being dismissed.  Tracked to allow the bottom space to compensate
      * for their removal animation and avoid moving the scroll position.
      */
     private int mCompensationHeight;
 
     /**
      * Height compensation value for each item being dismissed. Since dismissals sometimes include
      * sibling elements, and these don't get the standard treatment, we track the total height
      * associated with the element the user interacted with.
      */
     private final Map<ViewHolder, Integer> mCompensationHeightMap = new HashMap<>();
 
-    /** Whether the RecyclerView and its children should react to touch events. */
+    /**
+     * Whether the {@link SuggestionsRecyclerView} and its children should react to touch events.
+     */
     private boolean mTouchEnabled = true;
 
     /** The ui config for this view. */
     private UiConfig mUiConfig;
 
     /** The context menu manager for this view. */
     private ContextMenuManager mContextMenuManager;
 
     public SuggestionsRecyclerView(Context context) {
         this(context, null);
@@ skipping 24 matching lines @@
         setHasFixedSize(true);
 
         ItemTouchHelper helper = new ItemTouchHelper(new ItemTouchCallbacks());
         helper.attachToRecyclerView(this);
     }
 
     public boolean isFirstItemVisible() {
         return mLayoutManager.findFirstVisibleItemPosition() == 0;
     }
 
+    /**
+     * Sets whether the {@link SuggestionsRecyclerView} and its children should react to touch
+     * events.
+     */
+    public void setTouchEnabled(boolean enabled) {
+        mTouchEnabled = enabled;
+    }
+
     @Override
     public boolean onInterceptTouchEvent(MotionEvent ev) {
         mGestureDetector.onTouchEvent(ev);
         if (!mTouchEnabled) return true;
         return super.onInterceptTouchEvent(ev);
     }
 
     @Override
-    public void setTouchEnabled(boolean enabled) {
-        mTouchEnabled = enabled;
-    }
-
-    @Override
     public boolean onTouchEvent(MotionEvent ev) {
         if (!mTouchEnabled) return false;
 
         // Action down would already have been handled in onInterceptTouchEvent
         if (ev.getActionMasked() != MotionEvent.ACTION_DOWN) {
             mGestureDetector.onTouchEvent(ev);
         }
         return super.onTouchEvent(ev);
     }
 
@@ skipping 247 matching lines @@
      * Callback to reset a card's properties affected by swipe to dismiss.
      */
     private static class ResetForDismissCallback extends NewTabPageViewHolder.PartialBindCallback {
         @Override
         public void onResult(NewTabPageViewHolder holder) {
             assert holder instanceof CardViewHolder;
             ((CardViewHolder) holder).getRecyclerView().updateViewStateForDismiss(0, holder);
         }
     }
 }