Use dropdown list for admin areas in pr form.

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

Index: chrome/android/javatests/src/org/chromium/chrome/browser/autofill/AutofillTestHelper.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/autofill/AutofillTestHelper.java b/chrome/android/javatests/src/org/chromium/chrome/browser/autofill/AutofillTestHelper.java
index 5892cb417b05a632eb2de0a7f5a87c49e10f9ab8..913c93a22c419b99376a9d850a901cf39b46f1ef 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/autofill/AutofillTestHelper.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/autofill/AutofillTestHelper.java
@@ -25,6 +25,7 @@ public class AutofillTestHelper {
     public AutofillTestHelper() {
         registerDataObserver();
         setNormalizationTimeoutForTesting();

[Mathieu, 2017/03/24 17:24:32]

can we have a special value of normalization timeout which is essentially ==
isTestingMode?

[Parastoo, 2017/03/24 22:42:23]

Like checking what the timeout is set to, and based on that decide if it's on
the testing mode? Like this:

if (PDM.getInstance().getRequestTimeout() == 1) {
  // Do this
}

I consider this as a bad design.

Or in setNormalizationTimeoutForTesting(int t) {
   if (t == 1) then setToTestingMode()
}


How about in setToTestingMode, setting the timeout to 1? And then removing the
setNormalizationTimeoutForTesting()?

[Mathieu, 2017/03/27 19:09:06]

I find it weird to have two calls related to testing. I don't think we use the
pattern "testing mode" in many places in the code. Ideally you would
setNormalizationTimeoutForTesting(0) which would tell the code to not even make
the request and return fake/no data right away. Ideally there would be no
concept of "testing mode"

[Mathieu, 2017/03/28 14:43:32]

Still need to address this. 

[Parastoo, 2017/03/29 19:33:55]

Done.

[Parastoo, 2017/03/29 19:33:55]

Done.

+        setToTestingMode();
     }
 
     void setNormalizationTimeoutForTesting() {
@@ -36,6 +37,15 @@ public class AutofillTestHelper {
         });
     }
 
+    void setToTestingMode() {
+        ThreadUtils.runOnUiThreadBlocking(new Runnable() {
+            @Override
+            public void run() {
+                PersonalDataManager.getInstance().setToTestingMode();
+            }
+        });
+    }
+
     AutofillProfile getProfile(final String guid) throws ExecutionException {
         return ThreadUtils.runOnUiThreadBlocking(new Callable<AutofillProfile>() {
             @Override