Make AbstractExtensionConnector.extend abstract (#10378)

Change-Id: I96682d62c632588e05508a50765ac3d029500a74

client/src/com/vaadin/client/JavaScriptExtension.java

@@ -49,4 +49,10 @@ public final class JavaScriptExtension extends AbstractExtensionConnector
         super.onUnregister();
         helper.onUnregister();
     }
+
+    @Override
+    protected void extend(ServerConnector target) {
+        // Nothing to do for JavaScriptExtension here. Everything is done in
+        // javascript.
+    }
 }

client/src/com/vaadin/client/extensions/AbstractExtensionConnector.java

@@ -42,7 +42,5 @@ public abstract class AbstractExtensionConnector extends AbstractConnector {
         }
     }
 
-    protected void extend(ServerConnector target) {
-        // Default does nothing
-    }
+    protected abstract void extend(ServerConnector target);
 }

client/src/com/vaadin/client/extensions/javascriptmanager/JavaScriptManagerConnector.java

@@ -22,6 +22,7 @@ import java.util.Set;
 import com.google.gwt.core.client.JavaScriptObject;
 import com.google.gwt.core.client.JsArray;
 import com.google.gwt.json.client.JSONArray;
+import com.vaadin.client.ServerConnector;
 import com.vaadin.client.communication.StateChangeEvent;
 import com.vaadin.client.extensions.AbstractExtensionConnector;
 import com.vaadin.shared.communication.MethodInvocation;
@@ -131,4 +132,11 @@ public class JavaScriptManagerConnector extends AbstractExtensionConnector {
     public JavaScriptManagerState getState() {
         return (JavaScriptManagerState) super.getState();
     }
+
+    @Override
+    protected void extend(ServerConnector target) {
+        // Nothing to do there as we are not interested in the connector we
+        // extend (Page i.e. UI)
+
+    }
 }

uitest/src/com/vaadin/tests/widgetset/client/SerializerTestConnector.java

@@ -26,6 +26,7 @@ import java.util.Map;
 import java.util.Map.Entry;
 import java.util.Set;
 
+import com.vaadin.client.ServerConnector;
 import com.vaadin.client.communication.StateChangeEvent;
 import com.vaadin.client.extensions.AbstractExtensionConnector;
 import com.vaadin.shared.Connector;
@@ -263,4 +264,10 @@ public class SerializerTestConnector extends AbstractExtensionConnector {
         // TODO do something clever
     }
 
+    @Override
+    protected void extend(ServerConnector target) {
+        // TODO Auto-generated method stub
+
+    }
+
 }

uitest/src/com/vaadin/tests/widgetset/client/helloworldfeature/HelloWorldExtensionConnector.java

@@ -42,8 +42,7 @@ public class HelloWorldExtensionConnector extends AbstractExtensionConnector {
     }
 
     @Override
-    public void setParent(ServerConnector parent) {
-        super.setParent(parent);
+    protected void extend(ServerConnector target) {
         greet();
     }
 

uitest/src/com/vaadin/tests/widgetset/client/minitutorials/v7a3/RefresherConnector.java

@@ -1,6 +1,7 @@
 package com.vaadin.tests.widgetset.client.minitutorials.v7a3;
 
 import com.google.gwt.user.client.Timer;
+import com.vaadin.client.ServerConnector;
 import com.vaadin.client.communication.StateChangeEvent;
 import com.vaadin.client.extensions.AbstractExtensionConnector;
 import com.vaadin.shared.ui.Connect;
@@ -30,6 +31,12 @@ public class RefresherConnector extends AbstractExtensionConnector {
         timer.cancel();
     }
 
+    @Override
+    protected void extend(ServerConnector target) {
+        // Nothing for refresher to do here as it does not need to access the
+        // connector it extends
+    }
+
     @Override
     public RefresherState getState() {
         return (RefresherState) super.getState();