[Android] Update all Robolectric tests to Robolectric 3.0

testing/android/junit/java/src/org/chromium/testing/local/LocalRobolectricTestRunner.java

Index: testing/android/junit/java/src/org/chromium/testing/local/LocalRobolectricTestRunner.java
diff --git a/testing/android/junit/java/src/org/chromium/testing/local/LocalRobolectricTestRunner.java b/testing/android/junit/java/src/org/chromium/testing/local/LocalRobolectricTestRunner.java
index cf6202b0507d863108ed40b2e814e144b7561c21..21a0b4e252c1dbdffa164540450d4d1489f29c3c 100644
--- a/testing/android/junit/java/src/org/chromium/testing/local/LocalRobolectricTestRunner.java
+++ b/testing/android/junit/java/src/org/chromium/testing/local/LocalRobolectricTestRunner.java
@@ -6,44 +6,37 @@ package org.chromium.testing.local;
 
 import org.junit.runners.model.InitializationError;
 
-import org.robolectric.AndroidManifest;
-import org.robolectric.DependencyResolver;
 import org.robolectric.RobolectricTestRunner;
-import org.robolectric.SdkConfig;
 import org.robolectric.annotation.Config;
+import org.robolectric.manifest.AndroidManifest;
 
 /**
- * A custom Robolectric Junit4 Test Runner. This test runner will load the
- * "real" android jars from a local directory rather than use Maven to fetch
- * them from the Maven Central repository. Additionally, it will ignore the
+ * A custom Robolectric Junit4 Test Runner. This test runner will ignore the
  * API level written in the AndroidManifest as that can cause issues if
- * robolectric does not support that API level.
+ * Robolectric does not support that API level. The API level will be grabbed
+ * from the robolectric Config annotation, or just be
+ * |DEFAULT_ANDROID_API_LEVEL|
  */
 public class LocalRobolectricTestRunner extends RobolectricTestRunner {
 
-    private static final int ANDROID_API_LEVEL = 18;
+    private static final int DEFAULT_ANDROID_API_LEVEL = 21;
 
     public LocalRobolectricTestRunner(Class<?> testClass) throws InitializationError {
         super(testClass);
     }
 
     @Override
-    protected final DependencyResolver getJarResolver() {
-        return new RobolectricClasspathDependencyResolver();
-    }
-
-    @Override
-    protected SdkConfig pickSdkVersion(AndroidManifest appManifest, Config config) {
+    protected int pickSdkVersion(Config config, AndroidManifest appManifest) {
         // Pulling from the manifest is dangerous as the apk might target a version of
         // android that robolectric does not yet support. We still allow the API level to
         // be overridden with the Config annotation.
-        return config.emulateSdk() < 0
-                ? new SdkConfig(ANDROID_API_LEVEL) : super.pickSdkVersion(null, config);
-    }
-
-    @Override
-    protected int pickReportedSdkVersion(Config config, AndroidManifest appManifest) {
-        return config.reportSdk() < 0
-                ? ANDROID_API_LEVEL : super.pickReportedSdkVersion(config, appManifest);
+        if (config != null && config.sdk().length > 1) {

[jbudorick, 2016/08/03 15:32:29]

nit: wdyt about

  if (config != null) {
      if (config.sdk().length > 1) {
          throw ...
      } else if (config.sdk().length == 1) {
          return config.sdk()[0];
      }
  }
  return DEFAULT_ANDROID_API_LEVEL;

[mikecase (-- gone --), 2016/08/05 03:17:07]

Done

+            throw new IllegalArgumentException(
+                    "RobolectricTestRunner does not support multiple values for @Config.sdk");
+        } else if (config != null && config.sdk().length == 1) {
+            return config.sdk()[0];
+        } else {
+            return DEFAULT_ANDROID_API_LEVEL;
+        }
     }
-}
+}