Add a note to Navigator.addView and setErrorView javadoc that views should not be shared between UIs (#10240)

Change-Id: I870bacb1794a222931db0a49a03b840dd66662b0

1 files changed, 6 insertions, 0 deletions

diff --git a/server/src/com/vaadin/navigator/Navigator.java b/server/src/com/vaadin/navigator/Navigator.java
index 72128adddf..1abbcdab3b 100644
--- a/server/src/com/vaadin/navigator/Navigator.java
+++ b/server/src/com/vaadin/navigator/Navigator.java
@@ -635,6 +635,9 @@ public class Navigator implements Serializable {
      * <p>
      * Registering another view with a name that is already registered
      * overwrites the old registration of the same type.
+     * <p>
+     * Note that a view should not be shared between UIs (for instance, it
+     * should not be a static field in a UI subclass).
      * 
      * @param viewName
      *            String that identifies a view (not null nor empty string)
@@ -734,6 +737,9 @@ public class Navigator implements Serializable {
      * the navigation state. This implicitly sets an appropriate error view
      * provider and overrides any previous
      * {@link #setErrorProvider(ViewProvider)} call.
+     * <p>
+     * Note that an error view should not be shared between UIs (for instance,
+     * it should not be a static field in a UI subclass).
      * 
      * @param viewClass
      *            The View class whose instance should be used as the error