Fix regressions in RPC handling (#10549)

* Generate parameter types based on RPC interface instead of defining
interface
* Don't try to find parameter types for JavaScript RPC invocations

Change-Id: I0e61e74f57dd34665c8befe826aa2759aeb01b54

client-compiler/src/com/vaadin/server/widgetsetutils/metadata/ServerRpcVisitor.java

@@ -38,7 +38,7 @@ public class ServerRpcVisitor extends TypeVisitor {
                 for (JMethod method : methods) {
                     ClientRpcVisitor.checkReturnType(logger, method);
                     bundle.setNeedsDelayedInfo(type, method);
-                    bundle.setNeedsParamTypes(subType, method);
+                    bundle.setNeedsParamTypes(type, method);
 
                     JType[] parameterTypes = method.getParameterTypes();
                     for (JType paramType : parameterTypes) {

client/src/com/vaadin/client/ApplicationConnection.java

@@ -2396,7 +2396,8 @@ public class ApplicationConnection {
                 JSONArray paramJson = new JSONArray();
 
                 Type[] parameterTypes = null;
-                if (!isLegacyVariableChange(invocation)) {
+                if (!isLegacyVariableChange(invocation)
+                        && !isJavascriptRpc(invocation)) {
                     try {
                         Type type = new Type(invocation.getInterfaceName(),
                                 null);
@@ -2457,6 +2458,12 @@ public class ApplicationConnection {
         makeUidlRequest(req.toString(), extraParams, forceSync);
     }
 
+    private boolean isJavascriptRpc(MethodInvocation invocation) {
+        String connectorId = invocation.getConnectorId();
+        ServerConnector connector = connectorMap.getConnector(connectorId);
+        return connector instanceof HasJavaScriptConnectorHelper;
+    }
+
     private boolean isLegacyVariableChange(MethodInvocation invocation) {
         return ApplicationConstants.UPDATE_VARIABLE_METHOD.equals(invocation
                 .getInterfaceName())