PlzNavigate: Move navigation-level mixed content checks to the browser.

content/browser/frame_host/mixed_content_navigation_throttle_unittest.cc

+// Copyright 2017 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 "base/macros.h"
+#include "content/browser/frame_host/mixed_content_navigation_throttle.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace content {
+
+// Tests that MixedContentNavigationThrottle correctly detects or ignores many
+// cases where there is or there is not mixed content, respectively.
+// Note: Browser side version of MixedContentCheckerTest.IsMixedContent. Must be
+// kept in sync manually!
+TEST(MixedContentNavigationThrottleTest, IsMixedContent) {
+  struct TestCase {
+    const char* origin;
+    const char* target;
+    const bool expectation;

[nasko, 2017/01/23 22:32:38]

nit: What is the meaning of "expectation"? Maybe rename it more explicitly
"expected_mixed_content" or something like that.

[carlosk, 2017/02/08 02:59:03]

Done.

+  } cases[] = {
+      {"http://example.com/foo", "http://example.com/foo", false},
+      {"http://example.com/foo", "https://example.com/foo", false},
+      {"http://example.com/foo", "data:text/html,<p>Hi!</p>", false},
+      {"http://example.com/foo", "about:blank", false},
+      {"https://example.com/foo", "https://example.com/foo", false},
+      {"https://example.com/foo", "wss://example.com/foo", false},
+      {"https://example.com/foo", "data:text/html,<p>Hi!</p>", false},
+      {"https://example.com/foo", "http://127.0.0.1/", false},
+      {"https://example.com/foo", "http://[::1]/", false},
+      {"https://example.com/foo", "blob:https://example.com/foo", false},
+      {"https://example.com/foo", "blob:http://example.com/foo", false},

[nasko, 2017/01/23 22:32:38]

nit: I know this is how the code works, but this just feels wrong :(.

[carlosk, 2017/02/08 02:59:03]

Acknowledged.

+      {"https://example.com/foo", "blob:null/foo", false},
+      {"https://example.com/foo", "filesystem:https://example.com/foo", false},
+      {"https://example.com/foo", "filesystem:http://example.com/foo", false},
+      {"https://example.com/foo", "filesystem:null/foo", false},
+
+      {"https://example.com/foo", "http://example.com/foo", true},
+      {"https://example.com/foo", "http://google.com/foo", true},
+      {"https://example.com/foo", "ws://example.com/foo", true},
+      {"https://example.com/foo", "ws://google.com/foo", true},
+      {"https://example.com/foo", "http://192.168.1.1/", true},
+      {"https://example.com/foo", "http://localhost/", true},
+  };
+
+  for (const auto& test : cases) {
+    SCOPED_TRACE(::testing::Message() << "Origin: " << test.origin
+                                      << ", Target: " << test.target
+                                      << ", Expectation: " << test.expectation);
+    GURL origin_url(test.origin);
+    GURL target_url(test.target);
+    EXPECT_EQ(test.expectation,
+              MixedContentNavigationThrottle::IsMixedContentForTesting(
+                  origin_url, target_url));
+  }
+}
+
+}  // content