Context Menu Refactor

chrome/android/java/src/org/chromium/chrome/browser/omnibox/UrlBar.java

Index: chrome/android/java/src/org/chromium/chrome/browser/omnibox/UrlBar.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/omnibox/UrlBar.java b/chrome/android/java/src/org/chromium/chrome/browser/omnibox/UrlBar.java
index 596d18a48075b6aa0170168e46c1e502f938b914..65d9a1677831ee905955a4392969e172d204438c 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/omnibox/UrlBar.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/omnibox/UrlBar.java
@@ -40,7 +40,6 @@ import org.chromium.chrome.browser.omnibox.LocationBarLayout.OmniboxLivenessList
 import org.chromium.chrome.browser.tab.Tab;
 import org.chromium.chrome.browser.util.UrlUtilities;
 import org.chromium.chrome.browser.widget.VerticallyFixedEditText;
-import org.chromium.content.browser.ContentViewCore;
 import org.chromium.ui.UiUtils;
 
 import java.net.MalformedURLException;
@@ -519,13 +518,6 @@ public class UrlBar extends VerticallyFixedEditText {
             return true;
         }
 
-        Tab currentTab = mUrlBarDelegate.getCurrentTab();
-        if (event.getAction() == MotionEvent.ACTION_DOWN && currentTab != null) {
-            // Make sure to hide the current ContentView ActionBar.
-            ContentViewCore viewCore = currentTab.getContentViewCore();
-            if (viewCore != null) viewCore.hideSelectActionMode();

[no sievers, 2016/06/23 23:16:41]

How does the select action mode get hidden now when you focus the omnibox?

[amaralp, 2016/06/24 00:45:49]

To be honest, I don't understand why this code was ever necessary.
Focusing on the omnibox clears selection which in turn hides the
select action mode. There might be some edge case that I'm not 
considering though.

-        }
-
         return super.onTouchEvent(event);
     }