2009-11-12 Adam Barth <abarth@webkit.org>
authoreric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 13 Nov 2009 03:05:42 +0000 (03:05 +0000)
committereric@webkit.org <eric@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 13 Nov 2009 03:05:42 +0000 (03:05 +0000)
        Reviewed by Darin Fisher.

        Improve SecurityOrigin::toString comment
        https://bugs.webkit.org/show_bug.cgi?id=31041

        * page/SecurityOrigin.h:

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

WebCore/ChangeLog
WebCore/page/SecurityOrigin.h

index c337832c9060802393bb9dc3c8a6ea59b1cf27df..712459cefe51a4028423f22589d5a3e4af61e279 100644 (file)
@@ -1,3 +1,12 @@
+2009-11-12  Adam Barth  <abarth@webkit.org>
+
+        Reviewed by Darin Fisher.
+
+        Improve SecurityOrigin::toString comment
+        https://bugs.webkit.org/show_bug.cgi?id=31041
+
+        * page/SecurityOrigin.h:
+
 2009-11-12  Shinichiro Hamaji  <hamaji@chromium.org>
 
         Reviewed by Darin Adler.
index 6d4ce1ffb65901147e1e7fbe4ad580906d27dbe0..7c32f03d299eca2684eeffa8ff101527cebc20b8 100644 (file)
@@ -123,8 +123,14 @@ namespace WebCore {
         // Convert this SecurityOrigin into a string.  The string
         // representation of a SecurityOrigin is similar to a URL, except it
         // lacks a path component.  The string representation does not encode
-        // the value of the SecurityOrigin's domain property.  The empty
-        // SecurityOrigin is represented with the string "null".
+        // the value of the SecurityOrigin's domain property.
+        //
+        // When using the string value, it's important to remember that it
+        // might be "null".  This happens when this SecurityOrigin has
+        // noAccess to other SecurityOrigins.  For example, this SecurityOrigin
+        // might have come from a data URL, the SecurityOrigin might be empty,
+        // or we might have explicitly decided that we
+        // shouldTreatURLSchemeAsNoAccess.
         String toString() const;
 
         // Serialize the security origin to a string that could be used as part of