Reviewed by Maciej.
authorthatcher <thatcher@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 15 Apr 2006 19:27:58 +0000 (19:27 +0000)
committerthatcher <thatcher@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 15 Apr 2006 19:27:58 +0000 (19:27 +0000)
        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=8405
          REGRESSION: Web Inspector's Style pane is blank

        * bindings/js/kjs_window.cpp:
        (KJS::Window::isSafeScript): Use isEmpty() instead of isNull() for checking
        the domain to determine if the document in a local file.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@13880 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog
WebCore/bindings/js/kjs_window.cpp

index 9cff70f..4237200 100644 (file)
@@ -1,3 +1,14 @@
+2006-04-15  Mitz Pettel  <opendarwin.org@mitzpettel.com>
+
+        Reviewed by Maciej.
+
+        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=8405
+          REGRESSION: Web Inspector's Style pane is blank
+
+        * bindings/js/kjs_window.cpp:
+        (KJS::Window::isSafeScript): Use isEmpty() instead of isNull() for checking
+        the domain to determine if the document in a local file. 
+
 2006-04-14  David Hyatt  <hyatt@apple.com>
 
         CSS vendor-specific property/value cleanup.  Properly qualify background-clip,
index bfd86e2..6e25d8d 100644 (file)
@@ -1252,7 +1252,7 @@ bool Window::isSafeScript(ExecState *exec) const
   
   // FIXME: this really should be explicitly checking for the "file:" protocol instead
   // Always allow local pages to execute any JS.
-  if (actDomain.isNull())
+  if (actDomain.isEmpty())
     return true;
   
   WebCore::String thisDomain = thisDocument->domain();