Small improvements to MixedContentChecker.

third_party/WebKit/Source/platform/network/NetworkUtils.h

Index: third_party/WebKit/Source/platform/network/NetworkUtils.h
diff --git a/third_party/WebKit/Source/platform/network/NetworkUtils.h b/third_party/WebKit/Source/platform/network/NetworkUtils.h
index bb4b5f9fa5fccc85936dfa11cbdd2fb689e0d5ee..98e733a6ef63e5645e5ab3e99085fb8a07b95043 100644
--- a/third_party/WebKit/Source/platform/network/NetworkUtils.h
+++ b/third_party/WebKit/Source/platform/network/NetworkUtils.h
@@ -14,6 +14,8 @@ namespace NetworkUtils {
 
 PLATFORM_EXPORT bool isReservedIPAddress(const String& host);
 
+PLATFORM_EXPORT bool isLocalHostname(const String& host, bool* isLocal6);

[Mike West, 2016/07/14 05:50:17]

Should we use this for `SecurityOrigin::isLocalhost()` as well?

[carlosk, 2016/07/18 09:38:36]

Possibly but I didn't have a use for that one in this change so I didn't add it.

My objective here was to being bringing Blink concepts closer to what we use
browser side. I recently noticed many differences around networking and
security. This seemed like a simple first step.

[Mike West, 2016/07/18 12:24:17]

I'd suggest changing it. In fact, I'd suggest replacing the whole thing with a
call to `net::IsLocalhost()`.
If you want to keep this patch small, then please just replace the `if (m_host
== "localhost")` piece of `SecurityOrigin::isLocalhost()` and add a TODO to pipe
through `net::IsLocalhost`. That looks like the only other place in C++ where
we've hardcoded the name.

[carlosk, 2016/07/18 13:14:02]

Done, the simpler version.

+
 } // NetworkUtils
 
 } // namespace blink