Remove redundant interface already implemented by super class. (#10258)

12 files changed, 13 insertions, 27 deletions

diff --git a/client/src/main/java/com/vaadin/client/connectors/AbstractMultiSelectConnector.java b/client/src/main/java/com/vaadin/client/connectors/AbstractMultiSelectConnector.java
index ee85b50ebe..b78ea914d7 100644
--- a/client/src/main/java/com/vaadin/client/connectors/AbstractMultiSelectConnector.java
+++ b/client/src/main/java/com/vaadin/client/connectors/AbstractMultiSelectConnector.java
@@ -25,7 +25,6 @@ import com.google.gwt.user.client.ui.Composite;
 import com.google.gwt.user.client.ui.IsWidget;
 import com.google.gwt.user.client.ui.Widget;
 import com.vaadin.client.data.DataSource;
-import com.vaadin.client.ui.HasRequiredIndicator;
 import com.vaadin.shared.Range;
 import com.vaadin.shared.Registration;
 import com.vaadin.shared.data.selection.MultiSelectServerRpc;
@@ -45,7 +44,7 @@ import elemental.json.JsonObject;
  * @since 8.0
  */
 public abstract class AbstractMultiSelectConnector
-        extends AbstractListingConnector implements HasRequiredIndicator {
+        extends AbstractListingConnector {
 
     /**
      * Abstraction layer to help populate different multiselect widgets based on
diff --git a/client/src/main/java/com/vaadin/client/connectors/AbstractSingleSelectConnector.java b/client/src/main/java/com/vaadin/client/connectors/AbstractSingleSelectConnector.java
index 9df4afa084..2f0a02f5af 100644
--- a/client/src/main/java/com/vaadin/client/connectors/AbstractSingleSelectConnector.java
+++ b/client/src/main/java/com/vaadin/client/connectors/AbstractSingleSelectConnector.java
@@ -17,7 +17,6 @@ package com.vaadin.client.connectors;
 
 import com.google.gwt.event.dom.client.HasAllFocusHandlers;
 import com.google.gwt.user.client.ui.Widget;
-import com.vaadin.client.ui.HasRequiredIndicator;
 import com.vaadin.shared.ui.AbstractSingleSelectState;
 
 /**
@@ -27,8 +26,7 @@ import com.vaadin.shared.ui.AbstractSingleSelectState;
  * @since 8.0
  */
 public abstract class AbstractSingleSelectConnector<WIDGET extends Widget & HasAllFocusHandlers>
-        extends AbstractFocusableListingConnector<WIDGET>
-        implements HasRequiredIndicator {
+        extends AbstractFocusableListingConnector<WIDGET> {
 
     @Override
     public AbstractSingleSelectState getState() {
diff --git a/client/src/main/java/com/vaadin/client/ui/AbstractComponentConnector.java b/client/src/main/java/com/vaadin/client/ui/AbstractComponentConnector.java
index 76f72b6043..ec8c005da5 100644
--- a/client/src/main/java/com/vaadin/client/ui/AbstractComponentConnector.java
+++ b/client/src/main/java/com/vaadin/client/ui/AbstractComponentConnector.java
@@ -31,7 +31,6 @@ import com.google.gwt.user.client.ui.Focusable;
 import com.google.gwt.user.client.ui.HasEnabled;
 import com.google.gwt.user.client.ui.Widget;
 import com.vaadin.client.BrowserInfo;
-import com.vaadin.client.ComponentConnector;
 import com.vaadin.client.HasComponentsConnector;
 import com.vaadin.client.LayoutManager;
 import com.vaadin.client.MouseEventDetailsBuilder;
@@ -63,7 +62,7 @@ import com.vaadin.shared.ui.TabIndexState;
 import com.vaadin.shared.ui.ui.UIState;
 
 public abstract class AbstractComponentConnector extends AbstractConnector
-        implements ComponentConnector, HasErrorIndicator {
+        implements HasErrorIndicator {
 
     private HandlerRegistration contextHandler = null;
 
diff --git a/client/src/main/java/com/vaadin/client/ui/AbstractFieldConnector.java b/client/src/main/java/com/vaadin/client/ui/AbstractFieldConnector.java
index 7c6c681485..1650baee8d 100644
--- a/client/src/main/java/com/vaadin/client/ui/AbstractFieldConnector.java
+++ b/client/src/main/java/com/vaadin/client/ui/AbstractFieldConnector.java
@@ -19,7 +19,7 @@ import com.vaadin.client.StyleConstants;
 import com.vaadin.shared.AbstractFieldState;
 
 public abstract class AbstractFieldConnector extends AbstractComponentConnector
-        implements HasRequiredIndicator, HasErrorIndicator {
+        implements HasRequiredIndicator {
 
     @Override
     public AbstractFieldState getState() {
diff --git a/client/src/main/java/com/vaadin/client/ui/VAbstractPopupCalendar.java b/client/src/main/java/com/vaadin/client/ui/VAbstractPopupCalendar.java
index 67ac878552..76ebe85c28 100644
--- a/client/src/main/java/com/vaadin/client/ui/VAbstractPopupCalendar.java
+++ b/client/src/main/java/com/vaadin/client/ui/VAbstractPopupCalendar.java
@@ -61,7 +61,7 @@ import com.vaadin.shared.ui.datefield.TextualDateFieldState;
  */
 public abstract class VAbstractPopupCalendar<PANEL extends VAbstractCalendarPanel<R>, R extends Enum<R>>
         extends VAbstractTextualDate<R>
-        implements Field, ClickHandler, CloseHandler<PopupPanel>, SubPartAware {
+        implements ClickHandler, CloseHandler<PopupPanel> {
 
     /** For internal use only. May be removed or replaced in the future. */
     public final Button calendarToggle = new Button();
diff --git a/client/src/main/java/com/vaadin/client/ui/VAbstractTextualDate.java b/client/src/main/java/com/vaadin/client/ui/VAbstractTextualDate.java
index 8206f2f862..2fd153c458 100644
--- a/client/src/main/java/com/vaadin/client/ui/VAbstractTextualDate.java
+++ b/client/src/main/java/com/vaadin/client/ui/VAbstractTextualDate.java
@@ -52,7 +52,8 @@ import com.vaadin.shared.EventId;
  * @since 8.0
  */
 public abstract class VAbstractTextualDate<R extends Enum<R>>
-        extends VDateField<R> implements Field, ChangeHandler, Focusable,
+        extends VDateField<R>
+        implements ChangeHandler, Focusable,
         SubPartAware, HandlesAriaCaption, HandlesAriaInvalid,
         HandlesAriaRequired, KeyDownHandler {
 
diff --git a/client/src/main/java/com/vaadin/client/ui/VColorPicker.java b/client/src/main/java/com/vaadin/client/ui/VColorPicker.java
index 48947b764c..65b19781a1 100644
--- a/client/src/main/java/com/vaadin/client/ui/VColorPicker.java
+++ b/client/src/main/java/com/vaadin/client/ui/VColorPicker.java
@@ -16,7 +16,6 @@
 package com.vaadin.client.ui;
 
 import com.google.gwt.event.dom.client.ClickEvent;
-import com.google.gwt.event.dom.client.ClickHandler;
 import com.google.gwt.user.client.ui.HTML;
 
 /**
@@ -24,7 +23,7 @@ import com.google.gwt.user.client.ui.HTML;
  *
  * @since 7.0.0
  */
-public class VColorPicker extends VButton implements ClickHandler {
+public class VColorPicker extends VButton {
 
     private String color = null;
 
diff --git a/client/src/main/java/com/vaadin/client/ui/VNativeSelect.java b/client/src/main/java/com/vaadin/client/ui/VNativeSelect.java
index a7e3097bdf..0791cc0304 100644
--- a/client/src/main/java/com/vaadin/client/ui/VNativeSelect.java
+++ b/client/src/main/java/com/vaadin/client/ui/VNativeSelect.java
@@ -17,7 +17,6 @@ package com.vaadin.client.ui;
 
 import java.util.Objects;
 
-import com.google.gwt.event.dom.client.HasAllFocusHandlers;
 import com.google.gwt.user.client.ui.ListBox;
 import com.vaadin.client.widgets.FocusableFlowPanelComposite;
 import com.vaadin.shared.ui.nativeselect.NativeSelectState;
@@ -27,8 +26,7 @@ import com.vaadin.shared.ui.nativeselect.NativeSelectState;
  *
  * @author Vaadin Ltd.
  */
-public class VNativeSelect extends FocusableFlowPanelComposite
-        implements HasAllFocusHandlers {
+public class VNativeSelect extends FocusableFlowPanelComposite {
 
     private final ListBox listBox = new ListBox();
 
diff --git a/client/src/main/java/com/vaadin/client/ui/VTextArea.java b/client/src/main/java/com/vaadin/client/ui/VTextArea.java
index 18727361fe..4ca0948e0c 100644
--- a/client/src/main/java/com/vaadin/client/ui/VTextArea.java
+++ b/client/src/main/java/com/vaadin/client/ui/VTextArea.java
@@ -35,7 +35,7 @@ import com.vaadin.client.ui.dd.DragImageModifier;
  *
  */
 public class VTextArea extends VTextField
-        implements DragImageModifier, AbstractTextFieldWidget {
+        implements DragImageModifier {
 
     public static final String CLASSNAME = "v-textarea";
 
diff --git a/client/src/main/java/com/vaadin/client/ui/VTwinColSelect.java b/client/src/main/java/com/vaadin/client/ui/VTwinColSelect.java
index 4797aab311..023b6395b2 100644
--- a/client/src/main/java/com/vaadin/client/ui/VTwinColSelect.java
+++ b/client/src/main/java/com/vaadin/client/ui/VTwinColSelect.java
@@ -31,7 +31,6 @@ import com.google.gwt.event.dom.client.ClickEvent;
 import com.google.gwt.event.dom.client.ClickHandler;
 import com.google.gwt.event.dom.client.DoubleClickEvent;
 import com.google.gwt.event.dom.client.DoubleClickHandler;
-import com.google.gwt.event.dom.client.HasDoubleClickHandlers;
 import com.google.gwt.event.dom.client.KeyCodes;
 import com.google.gwt.event.dom.client.KeyDownEvent;
 import com.google.gwt.event.dom.client.KeyDownHandler;
@@ -112,8 +111,7 @@ public class VTwinColSelect extends Composite implements MultiSelectWidget,
     /**
      * A multiselect ListBox which catches double clicks.
      */
-    public class DoubleClickListBox extends ListBox
-            implements HasDoubleClickHandlers {
+    public class DoubleClickListBox extends ListBox {
         /**
          * Constructs a new DoubleClickListBox.
          */
diff --git a/client/src/main/java/com/vaadin/client/ui/combobox/ComboBoxConnector.java b/client/src/main/java/com/vaadin/client/ui/combobox/ComboBoxConnector.java
index 2b3db9aa65..683776f2f0 100644
--- a/client/src/main/java/com/vaadin/client/ui/combobox/ComboBoxConnector.java
+++ b/client/src/main/java/com/vaadin/client/ui/combobox/ComboBoxConnector.java
@@ -23,11 +23,8 @@ import com.vaadin.client.Profiler;
 import com.vaadin.client.annotations.OnStateChange;
 import com.vaadin.client.communication.StateChangeEvent;
 import com.vaadin.client.connectors.AbstractListingConnector;
-import com.vaadin.client.connectors.data.HasDataSource;
 import com.vaadin.client.data.DataChangeHandler;
 import com.vaadin.client.data.DataSource;
-import com.vaadin.client.ui.HasErrorIndicator;
-import com.vaadin.client.ui.HasRequiredIndicator;
 import com.vaadin.client.ui.SimpleManagedLayout;
 import com.vaadin.client.ui.VComboBox;
 import com.vaadin.client.ui.VComboBox.ComboBoxSuggestion;
@@ -47,8 +44,7 @@ import elemental.json.JsonObject;
 
 @Connect(ComboBox.class)
 public class ComboBoxConnector extends AbstractListingConnector
-        implements HasRequiredIndicator, HasDataSource, SimpleManagedLayout,
-        HasErrorIndicator {
+        implements SimpleManagedLayout {
 
     private ComboBoxServerRpc rpc = getRpcProxy(ComboBoxServerRpc.class);
     private SelectionServerRpc selectionRpc = getRpcProxy(
diff --git a/client/src/main/java/com/vaadin/client/ui/optiongroup/CheckBoxGroupConnector.java b/client/src/main/java/com/vaadin/client/ui/optiongroup/CheckBoxGroupConnector.java
index 1ab5c1d591..b04266f00f 100644
--- a/client/src/main/java/com/vaadin/client/ui/optiongroup/CheckBoxGroupConnector.java
+++ b/client/src/main/java/com/vaadin/client/ui/optiongroup/CheckBoxGroupConnector.java
@@ -26,7 +26,6 @@ import com.vaadin.client.TooltipInfo;
 import com.vaadin.client.communication.StateChangeEvent;
 import com.vaadin.client.connectors.AbstractFocusableListingConnector;
 import com.vaadin.client.data.DataSource;
-import com.vaadin.client.ui.HasRequiredIndicator;
 import com.vaadin.client.ui.VCheckBoxGroup;
 import com.vaadin.shared.data.selection.MultiSelectServerRpc;
 import com.vaadin.shared.ui.Connect;
@@ -47,8 +46,7 @@ import elemental.json.JsonObject;
 // TODO refactor to extend AbstractMultiSelectConnector, maybe when
 // SelectionModel is removed from client side framwork8-issues#421
 public class CheckBoxGroupConnector
-        extends AbstractFocusableListingConnector<VCheckBoxGroup>
-        implements HasRequiredIndicator {
+        extends AbstractFocusableListingConnector<VCheckBoxGroup> {
 
     @Override
     protected void init() {