AW: do not convert support lib param types

No expected change to behavior, but this should have a slight
performance boost to the support library.

Previously, for each method call on a boundary interface, we would
iterate over method parameter types and convert each type to the
corresponding type visible by delegateLoader. This work is redundant,
because we only ever pass pre-L frameworks types (e.g., WebView, String,
InvocationHandler) and primitives.

This work would not be redundant if we passed BoundaryInterfaces as
parameters to some BoundaryInterface methods, however we don't ever
intend to do so.

Bug: 837820
Test: ./gradlew :webkit:connectedAndroidTest
Change-Id: Icd3bfc620a45f9acd437da514ae5a02b3d8a6b02
Reviewed-on: https://chromium-review.googlesource.com/1054122Reviewed-by: Gustav Sennton <gsennton@chromium.org>
Commit-Queue: Nate Fischer <ntfschr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#558037}

android_webview/support_library/boundary_interfaces/src/org/chromium/support_lib_boundary/util/BoundaryInterfaceReflectionUtil.java

@@ -24,17 +24,9 @@ public class BoundaryInterfaceReflectionUtil {
             throws ClassNotFoundException, NoSuchMethodException {
         Class<?> declaringClass =
                 Class.forName(method.getDeclaringClass().getName(), true, delegateLoader);
-        Class[] otherSideParameterClasses = method.getParameterTypes();
-        Class[] parameterClasses = new Class[otherSideParameterClasses.length];
-        for (int n = 0; n < parameterClasses.length; n++) {
-            Class<?> clazz = otherSideParameterClasses[n];
-            // Primitive classes are shared between the classloaders - so we can use the same
-            // primitive class declarations on either side. Non-primitive classes must be looked up
-            // by name.
-            parameterClasses[n] = clazz.isPrimitive()
-                    ? clazz
-                    : Class.forName(clazz.getName(), true, delegateLoader);
-        }
+        // We do not need to convert parameter types across ClassLoaders because we never pass
+        // BoundaryInterfaces in methods, but pass InvocationHandlers instead.
+        Class[] parameterClasses = method.getParameterTypes();
         return declaringClass.getDeclaredMethod(method.getName(), parameterClasses);
     }