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

content/browser/frame_host/mixed_content_navigation_throttle_unittest.cc

Index: content/browser/frame_host/mixed_content_navigation_throttle_unittest.cc
diff --git a/content/browser/frame_host/mixed_content_navigation_throttle_unittest.cc b/content/browser/frame_host/mixed_content_navigation_throttle_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..244a4a5f3d4ef87ec3713f5524328899ff6fe125
--- /dev/null
+++ b/content/browser/frame_host/mixed_content_navigation_throttle_unittest.cc
@@ -0,0 +1,58 @@
+// 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 "content/browser/frame_host/mixed_content_navigation_throttle.h"

[nasko, 2017/02/15 17:41:39]

nit: Why did this move here? It isn't the header file for this cc, so it should
be together with the rest of the includes.

[carlosk, 2017/02/15 21:46:31]

This complies with the style guide [1] and pre-submit checks cry otherwise.

[1]
https://google.github.io/styleguide/cppguide.html#Names_and_Order_of_Includes

+
+#include "base/macros.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 expected_mixed_content;
+  } 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},
+      {"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.expected_mixed_content);
+    GURL origin_url(test.origin);
+    GURL target_url(test.target);
+    EXPECT_EQ(test.expected_mixed_content,
+              MixedContentNavigationThrottle::IsMixedContentForTesting(
+                  origin_url, target_url));
+  }
+}
+
+}  // content