Author: [email protected]
Date: Wed Jun 10 10:49:08 2009
New Revision: 5532

Modified:
    trunk/dev/core/src/com/google/gwt/core/linker/IFrameLinker.java

Log:
Fixes missing <head> element in the script frame on Safari 3 (issue 3738).
Review: http://gwt-code-reviews.appspot.com/34834


Modified: trunk/dev/core/src/com/google/gwt/core/linker/IFrameLinker.java
==============================================================================
--- trunk/dev/core/src/com/google/gwt/core/linker/IFrameLinker.java      
(original)
+++ trunk/dev/core/src/com/google/gwt/core/linker/IFrameLinker.java     Wed Jun 
 
10 10:49:08 2009
@@ -208,7 +208,7 @@
      out.print("if ($wnd." + context.getModuleFunctionName() + ") $wnd."
          + context.getModuleFunctionName() + ".onScriptLoad();");
      out.newline();
-    out.print("--></script>");
+    out.print("--></script></body></html>");
      out.newlineOpt();

      return out.toString();
@@ -249,10 +249,11 @@
    private String getModulePrefix(LinkerContext context, String strongName,
        boolean supportRunAsync) {
      DefaultTextOutput out = new  
DefaultTextOutput(context.isOutputCompact());
+    out.print("<html>");
      out.newlineOpt();

      // Setup the well-known variables.
-    out.print("<script>");
+    out.print("<head><script>");
      out.newlineOpt();
      out.print("var $gwt_version = \"" + About.getGwtVersionNum() + "\";");
      out.newlineOpt();
@@ -305,8 +306,9 @@
          + "',subSystem:'startup',evtGroup:'moduleStartup'"
          + ",millis:(new Date()).getTime(),type:'moduleEvalStart'});");
      out.newlineOpt();
-    out.print("</script>");
+    out.print("</script></head>");
      out.newlineOpt();
+    out.print("<body>");
      out.newlineOpt();

      // Begin a script block inside the body. It's commented out so that the

--~--~---------~--~----~------------~-------~--~----~
http://groups.google.com/group/Google-Web-Toolkit-Contributors
-~----------~----~----~----~------~----~------~--~---

Reply via email to