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

content/common/frame_messages.h

Index: content/common/frame_messages.h
diff --git a/content/common/frame_messages.h b/content/common/frame_messages.h
index 9f65e60eef8ec63bb6acc6f7023c84ff02e0a4d0..c2fd763c4291774f5f54e4ade7888606ff4f9bb9 100644
--- a/content/common/frame_messages.h
+++ b/content/common/frame_messages.h
@@ -38,6 +38,7 @@
 #include "content/public/common/page_importance_signals.h"
 #include "content/public/common/page_state.h"
 #include "content/public/common/referrer.h"
+#include "content/public/common/request_context_type.h"
 #include "content/public/common/resource_response.h"
 #include "content/public/common/stop_find_action.h"
 #include "content/public/common/three_d_api_types.h"
@@ -46,6 +47,7 @@
 #include "ppapi/features/features.h"
 #include "third_party/WebKit/public/platform/WebFocusType.h"
 #include "third_party/WebKit/public/platform/WebInsecureRequestPolicy.h"
+#include "third_party/WebKit/public/platform/WebMixedContent.h"

[jam, 2017/01/09 21:15:46]

nit: shouldn't be needed because the header that declares the struct should be
including the file

[carlosk, 2017/01/10 19:13:11]

Done.

 #include "third_party/WebKit/public/web/WebFindOptions.h"
 #include "third_party/WebKit/public/web/WebFrameOwnerProperties.h"
 #include "third_party/WebKit/public/web/WebFrameSerializerCacheControlPolicy.h"
@@ -347,6 +349,7 @@ IPC_STRUCT_TRAITS_BEGIN(content::BeginNavigationParams)
   IPC_STRUCT_TRAITS_MEMBER(has_user_gesture)
   IPC_STRUCT_TRAITS_MEMBER(skip_service_worker)
   IPC_STRUCT_TRAITS_MEMBER(request_context_type)
+  IPC_STRUCT_TRAITS_MEMBER(mixed_content_context_type)
   IPC_STRUCT_TRAITS_MEMBER(searchable_form_url)
   IPC_STRUCT_TRAITS_MEMBER(searchable_form_encoding)
   IPC_STRUCT_TRAITS_MEMBER(initiator_origin)
@@ -932,6 +935,22 @@ IPC_MESSAGE_ROUTED0(FrameMsg_SetHasReceivedUserGesture)
 IPC_MESSAGE_ROUTED1(FrameMsg_RunFileChooserResponse,
                     std::vector<content::FileChooserFileInfo>)
 
+// Updates the renderer with a list of unique blink::UseCounter::Feature values
+// representing Blink features "used" (or "performed") by the browser during the
+// current page load happening on the frame.
+IPC_MESSAGE_ROUTED1(FrameMsg_BlinkFeatureUsageReport,
+                    std::set<int>) /* features */
+
+// Informs the renderer that mixed content was found by the browser. The
+// included data is used for instance to report to the CSP policy and to log to
+// the frame console.
+IPC_MESSAGE_ROUTED5(FrameMsg_MixedContentFoundByTheBrowser,

[jam, 2017/01/09 21:15:46]

nit: ByTheBrowser is redundant since this is an IPC coming from the browser

[carlosk, 2017/01/10 19:13:11]

Done.

+                    GURL,                        /* main_resource_url */
+                    GURL,                        /* mixed_content_url */
+                    content::RequestContextType, /* request_context_type */
+                    bool,                        /* was_allowed */
+                    bool)                        /* had_redirect */
+
 // -----------------------------------------------------------------------------
 // Messages sent from the renderer to the browser.