[ContextualSearch] Add close button to list of actions that shouldPre…

…ventHandlingCurrentSelectionModification If the panel is dismissed due to the user clicking the close button, a selection modification on the current selection should not cause the panel to reappear. BUG=555775 [email protected] Review URL: https://codereview.chromium.org/1442353002 Cr-Commit-Position: refs/heads/master@{#359718} Review URL: https://codereview.chromium.org/1450413002 . Cr-Commit-Position: refs/branch-heads/2564@{crosswalk-project#19} Cr-Branched-From: 1283eca-refs/heads/master@{#359700}
fujunwei · Nov 17, 2015 · bb6c938 · bb6c938
1 parent ae27c2c
commit bb6c938
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/...ndroid/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchManager.java b/...ndroid/java/src/org/chromium/chrome/browser/contextualsearch/ContextualSearchManager.java
@@ -409,7 +409,8 @@ private boolean shouldPreventHandlingCurrentSelectionModification(StateChangeRea
                 && (reason == StateChangeReason.BACK_PRESS
                 || reason == StateChangeReason.BASE_PAGE_SCROLL
                 || reason == StateChangeReason.SWIPE
-                || reason == StateChangeReason.FLING);
+                || reason == StateChangeReason.FLING
+                || reason == StateChangeReason.CLOSE_BUTTON);
     }
 
     /**