Reading properties of components should not set state to dirty (#14060).

Added Automatic Testcase. The testcase needs a default constructor,
which has been added.
The test also found an edge case in Form.java which has been corrected,
as well as one missing getState(false) in AbstractMedia.

Change-Id: Id764c9e1596123015a84f6c2a9507f03bde383b1

1 files changed, 6 insertions, 3 deletions

diff --git a/server/src/com/vaadin/ui/PopupView.java b/server/src/com/vaadin/ui/PopupView.java
index fc927cf90d..90c60edc6e 100644
--- a/server/src/com/vaadin/ui/PopupView.java
+++ b/server/src/com/vaadin/ui/PopupView.java
@@ -61,6 +61,11 @@ public class PopupView extends AbstractComponent implements HasComponents {
 
     /* Constructors */
 
+    private PopupView() {
+        registerRpc(rpc);
+        setHideOnMouseOut(true);
+    }
+
     /**
      * A simple way to create a PopupPanel. Note that the minimal representation
      * may not be dynamically updated, in order to achieve this create your own
@@ -94,9 +99,7 @@ public class PopupView extends AbstractComponent implements HasComponents {
      *            the PopupView.Content that contains the information for this
      */
     public PopupView(PopupView.Content content) {
-        super();
-        registerRpc(rpc);
-        setHideOnMouseOut(true);
+        this();
         setContent(content);
     }