Invert host/domain checks in SameDomainOrHost

net/base/registry_controlled_domains/registry_controlled_domain.h

Index: net/base/registry_controlled_domains/registry_controlled_domain.h
diff --git a/net/base/registry_controlled_domains/registry_controlled_domain.h b/net/base/registry_controlled_domains/registry_controlled_domain.h
index 9e631f6f130f99159bbaaba77b7c5519e3dd935f..429a5770cc2b26522d6f516a120750b57f03379b 100644
--- a/net/base/registry_controlled_domains/registry_controlled_domain.h
+++ b/net/base/registry_controlled_domains/registry_controlled_domain.h
@@ -190,9 +190,9 @@ NET_EXPORT std::string GetDomainAndRegistry(base::StringPiece host,
 
 // These convenience functions return true if the two GURLs or Origins both have
 // hosts and one of the following is true:
+// * The hosts are identical.
 // * They each have a known domain and registry, and it is the same for both
 //   URLs.  Note that this means the trailing dot, if any, must match too.

[Peter Kasting, 2016/11/14 23:21:03]

This comment reordering makes me wonder if the function should be renamed
SameHostOrDomain() or maybe just OnSameSite() or something.

Probably that just mucks with a lot of code to little benefit, though.

[Charlie Harrison, 2016/11/14 23:48:23]

Yeah I don't think the rename is necessary, though I do agree it would be
slightly more accurate. The method is pretty much logical OR so I don't think a
re-ordering is super confusing.

-// * They don't have known domains/registries, but the hosts are identical.
 // Effectively, callers can use this function to check whether the input URLs
 // represent hosts "on the same site".
 NET_EXPORT bool SameDomainOrHost(const GURL& gurl1, const GURL& gurl2,