[ar] Fix reflection initializing AR

The reflection was failing because proguard changes the package of the
ArDelegate.

Bug: 900897
Change-Id: I303c8f97f9eaa89f64edce8cab6e912a2bbefe4a
Reviewed-on: https://chromium-review.googlesource.com/c/1318437Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#605458}

chrome/android/java/src/org/chromium/chrome/browser/vr/ArCoreJavaUtils.java

@@ -66,7 +66,7 @@ public class ArCoreJavaUtils {
     private int mAppMinArCoreApkVersionCode = ARCORE_NOT_INSTALLED_VERSION_CODE;
 
     @UsedByReflection("ArDelegate.java")
-    /* package */ static void installArCoreDeviceProviderFactory() {
+    public static void installArCoreDeviceProviderFactory() {
         nativeInstallArCoreDeviceProviderFactory();
     }
 

chrome/android/java/src/org/chromium/chrome/browser/vr/ArDelegate.java

@@ -18,9 +18,16 @@ public class ArDelegate {
             Class.forName("org.chromium.chrome.browser.vr.ArCoreJavaUtils")
                     .getDeclaredMethod("installArCoreDeviceProviderFactory")
                     .invoke(null);
-        } catch (ClassNotFoundException | IllegalAccessException | IllegalArgumentException
-                | NoSuchMethodException | InvocationTargetException e) {
+        } catch (ClassNotFoundException e) {
             // AR not available. No initialization required.
+        } catch (NoSuchMethodException e) {
+            // This and the exceptions below should not happen. Multiple catch statements to work
+            // around ReflectiveOperationException in API level 19.
+            throw new RuntimeException(e);
+        } catch (IllegalAccessException e) {
+            throw new RuntimeException(e);
+        } catch (InvocationTargetException e) {
+            throw new RuntimeException(e);
         }
     }