Take care of a FIXME in SecurityOrigin.cpp to check the validity of any innerURL during constructio…

Source/platform/weborigin/SecurityOriginTest.cpp

Index: Source/platform/weborigin/SecurityOriginTest.cpp
diff --git a/Source/platform/weborigin/SecurityOriginTest.cpp b/Source/platform/weborigin/SecurityOriginTest.cpp
index aed3b260ed5cb6baa9628213c4ca1857630892af..66678f9e996f7bc22f7c55462f0b7cc1ef8ad187 100644
--- a/Source/platform/weborigin/SecurityOriginTest.cpp
+++ b/Source/platform/weborigin/SecurityOriginTest.cpp
@@ -32,6 +32,7 @@
 #include "platform/weborigin/SecurityOrigin.h"
 
 #include "platform/RuntimeEnabledFeatures.h"
+#include "platform/blob/BlobURL.h"
 #include "platform/weborigin/KURL.h"
 #include "platform/weborigin/SecurityPolicy.h"
 #include "wtf/text/StringBuilder.h"
@@ -385,4 +386,18 @@ TEST_F(SecurityOriginTest, CreateFromTuple)
 
 }
 
+TEST_F(SecurityOriginTest, UniquenessPropagatesToBlobUrls)
+{
+    const char* uniqueOriginUrls[] = {
+        "",
+        "data:text/plain,hello_world",

[palmer, 2015/08/18 00:09:17]

What should happen for file: URLs? And filesystem:, and others?

[michaeln, 2015/08/18 20:22:16]

i've added some more cases, if you'd like to see other cases comment with the {
url, expectedBool, expectedString } lines and i'll plug them in

+    };
+
+    for (const char* url : uniqueOriginUrls) {
+        RefPtr<SecurityOrigin> origin = SecurityOrigin::create(KURL(ParsedURLString, url));
+        EXPECT_TRUE(origin->isUnique());
+        EXPECT_TRUE(SecurityOrigin::create(BlobURL::createPublicURL(origin.get()))->isUnique());
+    }
+}
+
 } // namespace blink