Make WebView keep keyboard when losing focus

chrome/android/javatests/src/org/chromium/chrome/browser/TabsTest.java

 // Copyright 2015 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;
 
 import static org.chromium.base.test.util.Restriction.RESTRICTION_TYPE_NON_LOW_END_DEVICE;
 
 import android.content.DialogInterface;
 import android.content.pm.ActivityInfo;
 import android.os.Debug;
 import android.os.Environment;
+import android.os.IBinder;
+import android.os.ResultReceiver;
 import android.os.SystemClock;
 import android.test.suitebuilder.annotation.LargeTest;
 import android.test.suitebuilder.annotation.MediumTest;
 import android.test.suitebuilder.annotation.SmallTest;
 import android.test.suitebuilder.annotation.Smoke;
 import android.util.DisplayMetrics;
 import android.util.Log;
 import android.view.View;
 
 import org.chromium.base.ThreadUtils;
@@ skipping 30 matching lines @@
 import org.chromium.chrome.browser.tabmodel.TabbedModeTabPersistencePolicy;
 import org.chromium.chrome.browser.toolbar.ToolbarPhone;
 import org.chromium.chrome.test.ChromeTabbedActivityTestBase;
 import org.chromium.chrome.test.util.ApplicationTestUtils;
 import org.chromium.chrome.test.util.ChromeRestriction;
 import org.chromium.chrome.test.util.ChromeTabUtils;
 import org.chromium.chrome.test.util.MenuUtils;
 import org.chromium.chrome.test.util.NewTabPageTestUtils;
 import org.chromium.chrome.test.util.OverviewModeBehaviorWatcher;
 import org.chromium.content.browser.ContentViewCore;
+import org.chromium.content.browser.input.ImeAdapter;
+import org.chromium.content.browser.input.InputMethodManagerWrapper;
 import org.chromium.content.browser.test.util.CallbackHelper;
 import org.chromium.content.browser.test.util.Criteria;
 import org.chromium.content.browser.test.util.CriteriaHelper;
+import org.chromium.content.browser.test.util.DOMUtils;
 import org.chromium.content.browser.test.util.JavaScriptUtils;
 import org.chromium.content.browser.test.util.UiUtils;
 import org.chromium.content.common.ContentSwitches;
 import org.chromium.content_public.browser.WebContents;
 import org.chromium.content_public.browser.WebContentsObserver;
 import org.chromium.net.test.EmbeddedTestServer;
 
 import java.io.File;
 import java.util.Locale;
 import java.util.concurrent.Callable;
@@ skipping 198 matching lines @@
 
         ChromeTabUtils.closeCurrentTab(getInstrumentation(), getActivity());
         getInstrumentation().runOnMainSync(new Runnable() {
             @Override
             public void run() {
                 assertEquals(tabCount, getActivity().getCurrentTabModel().getCount());
             }
         });
     }
 
+    private void setInputMethodManagerWrapperForTest(
+            final CallbackHelper showSoftInputHelper, final CallbackHelper hideSoftInputHelper) {
+        ImeAdapter imeAdapter =
+                getActivity().getActivityTab().getContentViewCore().getImeAdapterForTest();
+        imeAdapter.setInputMethodManagerWrapperForTest(
+                new InputMethodManagerWrapper(getInstrumentation().getContext()) {
+                    @Override
+                    public void showSoftInput(View view, int flags, ResultReceiver resultReceiver) {
+                        super.showSoftInput(view, flags, resultReceiver);
+                        showSoftInputHelper.notifyCalled();
+                    }
+
+                    @Override
+                    public boolean hideSoftInputFromWindow(
+                            IBinder windowToken, int flags, ResultReceiver resultReceiver) {
+                        super.hideSoftInputFromWindow(windowToken, flags, resultReceiver);
+                        hideSoftInputHelper.notifyCalled();
+                        return true;
+                    }
+                });
+    }
+
+    /**
+     * Verify that opening a new tab, switching to an existing tab and closing current tab hide
+     * keyboard.
+     */
+    @LargeTest
+    @Restriction(ChromeRestriction.RESTRICTION_TYPE_TABLET)
+    @Feature({"Android-TabSwitcher"})
+    @RetryOnFailure
+    public void testHideKeyboard() throws Exception {
+        mTestServer = EmbeddedTestServer.createAndStartFileServer(
+                getInstrumentation().getContext(), Environment.getExternalStorageDirectory());
+
+        // Open a new tab(The 1st tab).
+        ChromeTabUtils.fullyLoadUrlInNewTab(
+                getInstrumentation(), getActivity(), mTestServer.getURL(TEST_FILE_PATH), false);
+
+        final CallbackHelper showSoftInputHelper1 = new CallbackHelper();

[Changwan Ryu, 2016/10/19 05:10:21]

How about naming these variables as showCallback1 / hideCallback1 /
showCallback2 / hideCallback2 for short?

Also, is it possible to avoid keeping callback counts and only asserts when
necessary? That way we can also test that the minimum number of show and hide
calls are invoked. (Only if it works, that is.)

assertEquals(0, showCallback1.getCallCount());

// Click node in the 1st tab.
showCallback1.waitForCallback(0, 1);

// Open a new tab (2nd tab).
assertEquals(0, hideCallback1.getCallCount());
hideCallback1.waitForCallback(0, 1);

...

// Click node in the 2nd tab.
assertEquals(0, showCallback2.getCallCount());
...
showCallback2.waitForCallback(0, 1);

...

+        final CallbackHelper hideSoftInputHelper1 = new CallbackHelper();
+        setInputMethodManagerWrapperForTest(showSoftInputHelper1, hideSoftInputHelper1);
+
+        // Click node in the 1st tab.
+        int callbackCount = showSoftInputHelper1.getCallCount();
+        boolean result = DOMUtils.clickNode(

[Changwan Ryu, 2016/10/19 05:10:21]

Could you make the two lines into one?

assertTrue(DOMUtils.clickNode(...));

[yabinh, 2016/10/24 01:40:57]

Done.

+                this, getActivity().getActivityTab().getContentViewCore(), "input_text");
+        assertEquals("Failed to click node.", true, result);
+        showSoftInputHelper1.waitForCallback(callbackCount, 1);
+
+        // Open a new tab(the 2nd tab).
+        callbackCount = hideSoftInputHelper1.getCallCount();
+        ChromeTabUtils.fullyLoadUrlInNewTab(
+                getInstrumentation(), getActivity(), mTestServer.getURL(TEST_FILE_PATH), false);
+        hideSoftInputHelper1.waitForCallback(callbackCount, 1);
+
+        final CallbackHelper showSoftInputHelper2 = new CallbackHelper();
+        final CallbackHelper hideSoftInputHelper2 = new CallbackHelper();
+        setInputMethodManagerWrapperForTest(showSoftInputHelper2, hideSoftInputHelper2);
+
+        // Click node in the 2nd tab.
+        callbackCount = showSoftInputHelper2.getCallCount();
+        DOMUtils.clickNode(this, getActivity().getActivityTab().getContentViewCore(), "input_text");
+        showSoftInputHelper2.waitForCallback(callbackCount, 1);
+
+        // Switch to the 1st tab.
+        callbackCount = hideSoftInputHelper2.getCallCount();
+        ChromeTabUtils.switchTabInCurrentTabModel(getActivity(), 1);
+        hideSoftInputHelper2.waitForCallback(callbackCount, 1);
+
+        // Click node in the 1st tab.
+        callbackCount = showSoftInputHelper1.getCallCount();
+        DOMUtils.clickNode(this, getActivity().getActivityTab().getContentViewCore(), "input_text");
+        showSoftInputHelper1.waitForCallback(callbackCount, 1);
+
+        // Closes current tab(the 1st tab).
+        callbackCount = hideSoftInputHelper1.getCallCount();
+        ChromeTabUtils.closeCurrentTab(getInstrumentation(), getActivity());
+        hideSoftInputHelper1.waitForCallback(callbackCount, 1);
+    }
+
     /**
      * Verify that opening a new tab and navigating immediately sets a size on the newly created
      * renderer. https://crbug.com/434477.
      * @throws InterruptedException
      * @throws TimeoutException
      */
     @SmallTest
     @RetryOnFailure
     public void testNewTabSetsContentViewSize() throws InterruptedException, TimeoutException {
         ChromeTabUtils.newTabFromMenu(getInstrumentation(), getActivity());
@@ skipping 1433 matching lines @@
             throws InterruptedException {
         CriteriaHelper.pollInstrumentationThread(
                     Criteria.equals(expected, new Callable<Boolean>() {
                         @Override
                         public Boolean call() {
                             return fileToCheck.exists();
                         }
                     }));
     }
 }