Clean up state hierarchy for AbstractFieldState

Change-Id: I33c8c977bf8cd2d4c637d7988586496ce963465f

2 files changed, 3 insertions, 15 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 0dbe3cb331..60966c39a9 100644
--- a/client/src/main/java/com/vaadin/client/connectors/AbstractMultiSelectConnector.java
+++ b/client/src/main/java/com/vaadin/client/connectors/AbstractMultiSelectConnector.java
@@ -26,12 +26,12 @@ 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.AbstractFieldState;
 import com.vaadin.shared.Range;
 import com.vaadin.shared.Registration;
 import com.vaadin.shared.data.selection.MultiSelectServerRpc;
 import com.vaadin.shared.data.selection.SelectionModel;
 import com.vaadin.shared.ui.ListingJsonConstants;
-import com.vaadin.shared.ui.RequiredIndicatorState;
 
 import elemental.json.JsonObject;
 
@@ -158,8 +158,8 @@ public abstract class AbstractMultiSelectConnector
     }
 
     @Override
-    public RequiredIndicatorState getState() {
-        return (RequiredIndicatorState) super.getState();
+    public AbstractFieldState getState() {
+        return (AbstractFieldState) super.getState();
     }
 
     @Override
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 015b805ee1..fde1f548a0 100644
--- a/client/src/main/java/com/vaadin/client/ui/AbstractFieldConnector.java
+++ b/client/src/main/java/com/vaadin/client/ui/AbstractFieldConnector.java
@@ -26,27 +26,15 @@ public abstract class AbstractFieldConnector extends AbstractComponentConnector
         return (AbstractFieldState) super.getState();
     }
 
-    public boolean isModified() {
-        return getState().modified;
-    }
-
     @Override
     public boolean isRequiredIndicatorVisible() {
         return getState().required && !isReadOnly();
     }
 
     @Override
-    public boolean isErrorIndicatorVisible() {
-        return super.isErrorIndicatorVisible() && !getState().hideErrors;
-    }
-
-    @Override
     protected void updateWidgetStyleNames() {
         super.updateWidgetStyleNames();
 
-        // add / remove modified style name to Fields
-        setWidgetStyleName(StyleConstants.MODIFIED, isModified());
-
         // add / remove error style name to Fields
         setWidgetStyleNameWithPrefix(getWidget().getStylePrimaryName(),
                 StyleConstants.REQUIRED_EXT, isRequiredIndicatorVisible());