commit 938a64eed195d43ad84bc11d3111a6904fd4b6bb
author jat@google.com <jat@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Fri Aug 28 03:54:30 2009 +0000
committer jat@google.com <jat@google.com@8db76d5a-ed1c-0410-87a9-c151d255dfc7> Fri Aug 28 03:54:30 2009 +0000
tree e3a0850f5ac66e409ef050aaf18cd2a677fcebfd
parent f134f0e23d55d09862861ec4a4b93ee55795b0c6

Fix a problem on Safari/Chrome where docuemntElement.getFirstChild is not
necessarily the head element.

Patch by: dwolf
Review by: jat


git-svn-id: https://google-web-toolkit.googlecode.com/svn/trunk@6025 8db76d5a-ed1c-0410-87a9-c151d255dfc7

user/src/com/google/gwt/jsonp/client/JsonpRequest.java

diff --git a/user/src/com/google/gwt/jsonp/client/JsonpRequest.java b/user/src/com/google/gwt/jsonp/client/JsonpRequest.java
index 62b2b38..43699ec 100644
--- a/user/src/com/google/gwt/jsonp/client/JsonpRequest.java
+++ b/user/src/com/google/gwt/jsonp/client/JsonpRequest.java
@@ -52,9 +52,9 @@
     return $wnd[name] = new Object();
   }-*/;
   
-  private static native Node getDocumentElement() /*-{
-    return $doc.documentElement;
-  }-*/;
+  private static Node getHeadElement() {
+    return Document.get().getElementsByTagName("head").getItem(0);
+  }
   
   private static String nextCallbackId() {
     return "I" + (callbackCounter++);
@@ -142,7 +142,7 @@
     script.setType("text/javascript");
     script.setId(callbackId);
     script.setSrc(uri.toString());
-    getDocumentElement().getFirstChild().appendChild(script);
+    getHeadElement().appendChild(script);
     timer = new Timer() {
       @Override
       public void run() {
@@ -225,7 +225,7 @@
       public void execute() {
         unregisterCallbacks(CALLBACKS);
         Node script = Document.get().getElementById(callbackId);
-        getDocumentElement().getFirstChild().removeChild(script);
+        getHeadElement().removeChild(script);
       }
     });
   }