Allow doc.written scripts with a matching domain and registry to execute.

third_party/WebKit/Source/platform/network/NetworkUtilsTest.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "platform/network/NetworkUtils.h"
 
 #include "net/base/ip_address.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "wtf/text/WTFString.h"
 
@@ skipping 43 matching lines @@
                 String::fromUTF8(addressString.data(),
                     addressString.length())));
         } else {
             EXPECT_FALSE(NetworkUtils::isReservedIPAddress(
                 String::fromUTF8(addressString.data(),
                     addressString.length())));
         }
     }
 }
 
+TEST(NetworkUtilsTest, GetDomainAndRegistry)
+{
+    EXPECT_EQ("", NetworkUtils::getDomainAndRegistry("", true));
+    EXPECT_EQ("", NetworkUtils::getDomainAndRegistry(".", true));
+    EXPECT_EQ("", NetworkUtils::getDomainAndRegistry("..", true));
+    EXPECT_EQ("", NetworkUtils::getDomainAndRegistry("com", true));
+    EXPECT_EQ("", NetworkUtils::getDomainAndRegistry(".com", true));
+    EXPECT_EQ("", NetworkUtils::getDomainAndRegistry("www.example.com:8000", true));
+
+    EXPECT_EQ("", NetworkUtils::getDomainAndRegistry("localhost", true));
+    EXPECT_EQ("", NetworkUtils::getDomainAndRegistry("127.0.0.1", true));
+
+    EXPECT_EQ("example.com", NetworkUtils::getDomainAndRegistry("example.com", true));
+    EXPECT_EQ("example.com", NetworkUtils::getDomainAndRegistry("www.example.com", true));
+    EXPECT_EQ("example.com", NetworkUtils::getDomainAndRegistry("multilevel.www.example.com", true));
+    EXPECT_EQ("example.co.uk", NetworkUtils::getDomainAndRegistry("www.example.co.uk", true));
+
+    // Verify proper handling of 'private registries'.
+    EXPECT_EQ("foo.appspot.com", NetworkUtils::getDomainAndRegistry("www.foo.appspot.com", true));
+    EXPECT_EQ("appspot.com", NetworkUtils::getDomainAndRegistry("www.foo.appspot.com", false));

[shivanisha, 2016/08/01 16:35:55]

Can we also include the example mentioned in the comment in
ShouldDisallowFetchForMainFrameScript:
static.example.com and www.example.com to test both of them return example.com.
As per the test on www.foo.appspot.com returning foo.appspot.com, won't
static.example.com return static.example.com?

[Bryan McQuade, 2016/08/01 18:43:51]

Ah, replied to the overall change rather than the comment. Repeating here:

Yes, good idea. I just added static.example.com, and www.example.com was already
there, above. Both return example.com. The getDomainAndRegistry function is
backed by a lookup trie that has a bunch of hostnames encoded in it, so it knows
that appspot.com is a dedicated private registry which it will handle
differently from example.com which is not a registry. Thanks!

+
+    // Verify that unknown registries are included.
+    EXPECT_EQ("example.notarealregistry", NetworkUtils::getDomainAndRegistry("www.example.notarealregistry", true));
+}
+
 } // namespace blink