Remove some code, an obsolete criteria, from SecurityOrigin test for uniqueness.

Source/platform/weborigin/KURLTest.cpp

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 390 matching lines @@
     EXPECT_TRUE(kurl.isValid());
     EXPECT_FALSE(kurl.isEmpty());
     EXPECT_STREQ("http://www.google.com:8000/favicon.ico", kurl.string().utf8().data());
 
     // Now let's test that giving an invalid replacement fails. Invalid
     // protocols fail without modifying the URL, which should remain valid.
     EXPECT_FALSE(kurl.setProtocol("f/sj#@"));
     EXPECT_TRUE(kurl.isValid());
 }
 
+TEST(KURLTest, ValidHttpAndFtpUrlsHaveHosts)
+{
+    KURL kurl;
+    EXPECT_TRUE(kurl.setProtocol("http"));
+    EXPECT_TRUE(kurl.protocolIs("http"));
+    EXPECT_FALSE(kurl.isValid());
+
+    EXPECT_TRUE(kurl.setProtocol("https"));
+    EXPECT_TRUE(kurl.protocolIs("https"));
+    EXPECT_FALSE(kurl.isValid());
+
+    EXPECT_TRUE(kurl.setProtocol("ftp"));
+    EXPECT_TRUE(kurl.protocolIs("ftp"));
+    EXPECT_FALSE(kurl.isValid());
+
+    kurl = KURL(KURL(), "http://");

[palmer, 2015/08/27 22:59:03]

Maybe try some weird ones, like http:///noodles/pancakes.php (note 3 leading
/s).

[michaeln, 2015/08/28 23:15:51]

Done, KURL interprets http:///noodles/pancakes.php as
http://noodles/pancakes.php

+    EXPECT_TRUE(kurl.protocolIs("http"));
+    EXPECT_FALSE(kurl.isValid());
+
+    kurl = KURL(KURL(), "https://");
+    EXPECT_TRUE(kurl.protocolIs("https"));
+    EXPECT_FALSE(kurl.isValid());
+
+    kurl = KURL(KURL(), "ftp://");
+    EXPECT_TRUE(kurl.protocolIs("ftp"));
+    EXPECT_FALSE(kurl.isValid());
+
+    kurl = KURL(KURL(), "http://host/");
+    EXPECT_TRUE(kurl.isValid());
+    kurl.setHost("");
+    EXPECT_FALSE(kurl.isValid());
+
+    kurl = KURL(KURL(), "https://host/");
+    EXPECT_TRUE(kurl.isValid());
+    kurl.setHost("");
+    EXPECT_FALSE(kurl.isValid());
+
+    kurl = KURL(KURL(), "ftp://host/");
+    EXPECT_TRUE(kurl.isValid());
+    kurl.setHost("");
+    EXPECT_FALSE(kurl.isValid());
+}
+
 TEST(KURLTest, Path)
 {
     const char initial[] = "http://www.google.com/path/foo";
     KURL kurl(ParsedURLString, initial);
 
     // Clear by setting a null string.
     String nullString;
     EXPECT_TRUE(nullString.isNull());
     kurl.setPath(nullString);
     EXPECT_STREQ("http://www.google.com/", kurl.string().utf8().data());
@@ skipping 274 matching lines @@
     };
 
     for (size_t i = 0; i < arraysize(referrerCases); i++) {
         KURL kurl(ParsedURLString, referrerCases[i].input);
         String referrer = kurl.strippedForUseAsReferrer();
         EXPECT_STREQ(referrerCases[i].output, referrer.utf8().data());
     }
 }
 
 } // namespace blink