From cece22a1d1919d8da459b6dab18c19e53c6e7f04 Mon Sep 17 00:00:00 2001
From: Leif Åstrand <leif@vaadin.com>
Date: Thu, 7 Jun 2012 12:07:16 +0000
Subject: Remove document.write from bootstrap (#8924)

svn changeset:23905/svn branch:6.8
---
 .../gwt/server/AbstractApplicationPortlet.java     | 70 ++++++++++++++++------
 .../gwt/server/AbstractApplicationServlet.java     | 12 ++--
 2 files changed, 59 insertions(+), 23 deletions(-)

diff --git a/src/com/vaadin/terminal/gwt/server/AbstractApplicationPortlet.java b/src/com/vaadin/terminal/gwt/server/AbstractApplicationPortlet.java
index 9f1bd4e9ee..26b0d3df1d 100644
--- a/src/com/vaadin/terminal/gwt/server/AbstractApplicationPortlet.java
+++ b/src/com/vaadin/terminal/gwt/server/AbstractApplicationPortlet.java
@@ -879,6 +879,30 @@ public abstract class AbstractApplicationPortlet extends GenericPortlet
         return null;
     }
 
+    /**
+     * Returns the URL from which the widgetset is served on the portal.
+     * 
+     * @param request
+     * @return
+     */
+    protected String getWidgetsetURL(RenderRequest request) {
+        String requestWidgetset = getApplicationOrSystemProperty(
+                PARAMETER_WIDGETSET, null);
+        String sharedWidgetset = getPortalProperty(
+                PORTAL_PARAMETER_VAADIN_WIDGETSET, request.getPortalContext());
+
+        String widgetset;
+        if (requestWidgetset != null) {
+            widgetset = requestWidgetset;
+        } else if (sharedWidgetset != null) {
+            widgetset = sharedWidgetset;
+        } else {
+            widgetset = DEFAULT_WIDGETSET;
+        }
+        String widgetsetURL = getWidgetsetURL(widgetset, request);
+        return widgetsetURL;
+    }
+
     /**
      * Returns the URL from which the widgetset is served on the portal.
      * 
@@ -1037,12 +1061,32 @@ public abstract class AbstractApplicationPortlet extends GenericPortlet
 
         writer.write("</script>\n");
 
+        writeAjaxPageWidgetset(request, writer);
+
         writeAjaxPageHtmlTheme(request, writer, themeName, themeURI,
                 portalTheme);
 
         // TODO Warn if widgetset has not been loaded after 15 seconds
     }
 
+    /**
+     * Writes the script to load the widgetset on the HTML fragment created by
+     * the portlet.
+     * 
+     * @param request
+     * @param writer
+     * @throws IOException
+     */
+    protected void writeAjaxPageWidgetset(RenderRequest request,
+            BufferedWriter writer) throws IOException {
+        String widgetsetURL = getWidgetsetURL(request);
+        writer.write("<iframe tabIndex=\"-1\" id=\"__gwt_historyFrame\" "
+                + "style=\"position:absolute;width:0;height:0;border:0;overflow:"
+                + "hidden;opacity:0;top:-100px;left:-100px;\" src=\"javascript:false\"></iframe>'\n");
+        writer.write("<script language='javascript' src='" + widgetsetURL
+                + "'></script>\n");
+    }
+
     /**
      * Writes the script to load the widgetset on the HTML fragment created by
      * the portlet.
@@ -1051,29 +1095,19 @@ public abstract class AbstractApplicationPortlet extends GenericPortlet
      * @param response
      * @param writer
      * @throws IOException
+     * 
+     * @Deprecated As of 6.8, use
+     *             {@link #writeAjaxPageWidgetset(RenderRequest, BufferedWriter)}
+     *             instead to avoid using document.write
      */
+    @Deprecated
     protected void writeAjaxPageScriptWidgetset(RenderRequest request,
             RenderResponse response, final BufferedWriter writer)
             throws IOException {
-        String requestWidgetset = getApplicationOrSystemProperty(
-                PARAMETER_WIDGETSET, null);
-        String sharedWidgetset = getPortalProperty(
-                PORTAL_PARAMETER_VAADIN_WIDGETSET, request.getPortalContext());
+        // No longer used as of #8924, but retained to maintain compatibility
 
-        String widgetset;
-        if (requestWidgetset != null) {
-            widgetset = requestWidgetset;
-        } else if (sharedWidgetset != null) {
-            widgetset = sharedWidgetset;
-        } else {
-            widgetset = DEFAULT_WIDGETSET;
-        }
-        String widgetsetURL = getWidgetsetURL(widgetset, request);
-        writer.write("document.write('<iframe tabIndex=\"-1\" id=\"__gwt_historyFrame\" "
-                + "style=\"position:absolute;width:0;height:0;border:0;overflow:"
-                + "hidden;opacity:0;top:-100px;left:-100px;\" src=\"javascript:false\"></iframe>');\n");
-        writer.write("document.write(\"<script language='javascript' src='"
-                + widgetsetURL + "'><\\/script>\");\n}\n");
+        // But we still need to close this one block, for compatibility
+        writer.append("\\n}\\n");
     }
 
     /**
diff --git a/src/com/vaadin/terminal/gwt/server/AbstractApplicationServlet.java b/src/com/vaadin/terminal/gwt/server/AbstractApplicationServlet.java
index c6d286ed03..ffa34b11cd 100644
--- a/src/com/vaadin/terminal/gwt/server/AbstractApplicationServlet.java
+++ b/src/com/vaadin/terminal/gwt/server/AbstractApplicationServlet.java
@@ -1929,11 +1929,7 @@ public abstract class AbstractApplicationServlet extends HttpServlet implements
         if (!isProductionMode()) {
             page.write("vaadin.debug = true;\n");
         }
-        page.write("document.write('<iframe tabIndex=\"-1\" id=\"__gwt_historyFrame\" "
-                + "style=\"position:absolute;width:0;height:0;border:0;overflow:"
-                + "hidden;\" src=\"javascript:false\"></iframe>');\n");
-        page.write("document.write(\"<script language='javascript' src='"
-                + widgetsetFilePath + "'><\\/script>\");\n}\n");
+        page.write("}\n");
 
         page.write("vaadin.vaadinConfigurations[\"" + appId + "\"] = {");
         page.write("appUri:'" + appUrl + "', ");
@@ -1991,6 +1987,12 @@ public abstract class AbstractApplicationServlet extends HttpServlet implements
         }
         page.write("};\n//]]>\n</script>\n");
 
+        page.write("<iframe tabIndex='-1' id='__gwt_historyFrame' "
+                + "style='position:absolute;width:0;height:0;border:0;overflow:"
+                + "hidden;' src='javascript:false'></iframe>\n");
+        page.write("<script language='javascript' src='" + widgetsetFilePath
+                + "'></script>\n");
+
         if (themeName != null) {
             // Custom theme's stylesheet, load only once, in different
             // script
-- 
cgit v1.2.3