Introduce ThreadedInputConnection behind a switch

chrome/android/javatests/src/org/chromium/chrome/browser/autofill/AutofillPopupTest.java

 // Copyright 2013 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.autofill;
 
 import android.test.suitebuilder.annotation.MediumTest;
 import android.text.TextUtils;
 import android.view.View;
 import android.view.ViewGroup;
 
+import org.chromium.base.CommandLine;
 import org.chromium.base.ThreadUtils;
 import org.chromium.base.test.util.Feature;
 import org.chromium.base.test.util.UrlUtils;
 import org.chromium.chrome.R;
 import org.chromium.chrome.browser.ChromeActivity;
 import org.chromium.chrome.browser.autofill.PersonalDataManager.AutofillProfile;
 import org.chromium.chrome.test.ChromeActivityTestCaseBase;
 import org.chromium.content.browser.ContentViewCore;
 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.TestInputMethodManagerWrapper;
 import org.chromium.content.browser.test.util.TouchCommon;
+import org.chromium.content.common.ContentSwitches;
 import org.chromium.content_public.browser.WebContents;
 import org.chromium.ui.autofill.AutofillPopup;
 
 import java.util.ArrayList;
 import java.util.List;
 import java.util.concurrent.ExecutionException;
 import java.util.concurrent.TimeoutException;
 
 /**
  * Integration tests for the AutofillPopup.
@@ skipping 131 matching lines @@
                 LANGUAGE_CODE);
         mHelper.setProfile(profile);
         assertEquals(1, mHelper.getNumberOfProfiles());
 
         // Click the input field for the first name.
         assertTrue(DOMUtils.waitForNonZeroNodeBounds(webContents, "fn"));
         DOMUtils.clickNode(this, viewCore, "fn");
 
         waitForKeyboardShowRequest(immw, 1);
 
-        ThreadUtils.runOnUiThreadBlocking(new Runnable() {
-            @Override
-            public void run() {
-                viewCore.getInputConnectionForTest().setComposingText(
-                        inputText, 1);
-            }
-        });
+        if (CommandLine.getInstance().hasSwitch(ContentSwitches.USE_IME_THREAD)) {

[aelias_OOO_until_Jul13, 2015/09/30 00:10:03]

We shouldn't be checking for a command-line switch in a unit tests, tests should
be orthogonal to that.

[Changwan Ryu, 2016/01/19 07:31:52]

Done.

+            viewCore.getInputConnectionForTest().setComposingText(inputText, 1);
+        } else {
+            ThreadUtils.runOnUiThreadBlocking(new Runnable() {
+                @Override
+                public void run() {
+                    viewCore.getInputConnectionForTest().setComposingText(inputText, 1);
+                }
+            });
+        }
 
         waitForAnchorViewAdd(view);
         View anchorView = view.findViewById(R.id.dropdown_popup_window);
 
         assertTrue(anchorView.getTag() instanceof AutofillPopup);
         final AutofillPopup popup = (AutofillPopup) anchorView.getTag();
 
         waitForAutofillPopopShow(popup);
 
         TouchCommon.singleClickViewRelative(popup.getListView(), 10, 10);
@@ skipping 160 matching lines @@
     private void assertLogged(String autofilledValue, String profileFullName) {
         for (AutofillLogger.LogEntry entry : mAutofillLoggedEntries) {
             if (entry.getAutofilledValue().equals(autofilledValue)
                     && entry.getProfileFullName().equals(profileFullName)) {
                 return;
             }
         }
         fail("Logged entry not found [" + autofilledValue + "," + profileFullName + "]");
     }
 }