Context Menu Refactor

content/public/android/java/src/org/chromium/content/browser/input/SelectActionMode.java

Index: content/public/android/java/src/org/chromium/content/browser/input/SelectActionMode.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/input/SelectActionMode.java b/content/public/android/java/src/org/chromium/content/browser/input/SelectActionMode.java
new file mode 100644
index 0000000000000000000000000000000000000000..17337b11e2139714f875ad46e88423f2ac8f9586
--- /dev/null
+++ b/content/public/android/java/src/org/chromium/content/browser/input/SelectActionMode.java
@@ -0,0 +1,38 @@
+// 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.content.browser;
+
+import org.chromium.base.annotations.CalledByNative;
+
+/**
+ * This class is an implementation of Select Action Mode UI.
+ */
+class SelectActionMode {
+    private SelectActionMode(int left, int top, int right, int bottom,
+            boolean hasSelection, boolean isEditable, boolean isPassword) {
+    }
+
+    @CalledByNative
+    private static SelectActionMode create(int left, int top, int right,
+            int bottom, boolean hasSelection, boolean isEditable, boolean isPassword) {
+        return new SelectActionMode(left, top, right, bottom, hasSelection, isEditable, isPassword);
+    }
+
+    @CalledByNative
+    private void show() {

[aelias_OOO_until_Jul13, 2016/06/24 21:37:29]

So I assume this patch only compiles but doesn't work yet because methods like
this have empty implementations, and I don't see the Blink change mentioned in
the description?

At a glance, I have no particular problem to any part of this patch, but I
prefer to do detailed code review only on working changes, so please ping again
when you have something that doesn't break any behavior (to your knowledge). 
Also, in the future, if you have a nonworking WIP, please specify that it
doesn't work in the patch description since that's different from the default
assumption (more commonly, WIPs work but are factored ugly -- this is the
reverse :).

+    }
+
+    @CalledByNative
+    private void hide() {
+    }
+
+    @CalledByNative
+    private void close() {
+    }
+
+    @CalledByNative
+    private void move(int left, int top, int right, int bottom) {
+    }
+}