[Android] Adding Smart GO/NEXT feature in Chrome

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

 // Copyright 2012 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.input;
 
 import android.annotation.SuppressLint;
 import android.content.res.Configuration;
 import android.graphics.Rect;
 import android.os.Build;
@@ skipping 602 matching lines @@
             case android.R.id.paste:
                 mWebContents.paste();
                 return true;
             default:
                 return false;
         }
     }
 
     boolean performEditorAction(int actionCode) {
         if (!isValid()) return false;
-        if (actionCode == EditorInfo.IME_ACTION_NEXT) {
-            sendSyntheticKeyPress(KeyEvent.KEYCODE_TAB,
-                    KeyEvent.FLAG_SOFT_KEYBOARD | KeyEvent.FLAG_KEEP_TOUCH_MODE
-                    | KeyEvent.FLAG_EDITOR_ACTION);
-        } else {
-            sendSyntheticKeyPress(KeyEvent.KEYCODE_ENTER,
-                    KeyEvent.FLAG_SOFT_KEYBOARD | KeyEvent.FLAG_KEEP_TOUCH_MODE
-                    | KeyEvent.FLAG_EDITOR_ACTION);
+        switch (actionCode) {
+            case EditorInfo.IME_ACTION_NEXT:
+                restartInput();

[Changwan Ryu, 2017/04/27 16:34:53]

I don't think you need to restart input here at all. Focus change in
FocusController should finish composing text on the previous input and cause
restart input after next input is focused. If you saw evidence that it did not
work, let me know.

[AKVT, 2017/05/03 14:35:11]

Done.

+                advanceFocusInForm(true);
+                break;
+            case EditorInfo.IME_ACTION_PREVIOUS:
+                restartInput();
+                advanceFocusInForm(false);
+                break;
+            default:
+                sendSyntheticKeyPress(KeyEvent.KEYCODE_ENTER,
+                        KeyEvent.FLAG_SOFT_KEYBOARD | KeyEvent.FLAG_KEEP_TOUCH_MODE
+                                | KeyEvent.FLAG_EDITOR_ACTION);
+                break;
         }
         return true;
     }
 
+    /**
+     * Advances the focus to next input field in the current form.
+     *
+     * @param forward indicates whether to advance forward or backward direction.
+     */
+    public void advanceFocusInForm(boolean forward) {

[Changwan Ryu, 2017/04/27 16:34:53]

public -> private

[AKVT, 2017/05/03 14:35:11]

Done.

+        if (mNativeImeAdapterAndroid == 0) return;
+        nativeAdvanceFocusInForm(mNativeImeAdapterAndroid, forward);
+    }
+
     void notifyUserAction() {
         mInputMethodManagerWrapper.notifyUserAction();
     }
 
     @VisibleForTesting
     protected void sendSyntheticKeyPress(int keyCode, int flags) {
         long eventTime = SystemClock.uptimeMillis();
         sendKeyEvent(new KeyEvent(eventTime, eventTime,
                 KeyEvent.ACTION_DOWN, keyCode, 0, 0,
                 KeyCharacterMap.VIRTUAL_KEYBOARD, 0,
@@ skipping 255 matching lines @@
     private native void nativeSetEditableSelectionOffsets(long nativeImeAdapterAndroid,
             int start, int end);
     private native void nativeSetComposingRegion(long nativeImeAdapterAndroid, int start, int end);
     private native void nativeDeleteSurroundingText(long nativeImeAdapterAndroid,
             int before, int after);
     private native void nativeDeleteSurroundingTextInCodePoints(
             long nativeImeAdapterAndroid, int before, int after);
     private native boolean nativeRequestTextInputStateUpdate(long nativeImeAdapterAndroid);
     private native void nativeRequestCursorUpdate(long nativeImeAdapterAndroid,
             boolean immediateRequest, boolean monitorRequest);
+    private native void nativeAdvanceFocusInForm(long nativeImeAdapterAndroid, boolean forward);
 }