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

content/renderer/render_frame_impl.h

Index: content/renderer/render_frame_impl.h
diff --git a/content/renderer/render_frame_impl.h b/content/renderer/render_frame_impl.h
index f6c27ad5be9a6ff99f5675af0cf27eca931c60c9..4dfac8c35275eac2ddc9c964b92aa3be9a8ff140 100644
--- a/content/renderer/render_frame_impl.h
+++ b/content/renderer/render_frame_impl.h
@@ -35,6 +35,7 @@
 #include "content/public/common/console_message_level.h"
 #include "content/public/common/javascript_message_type.h"
 #include "content/public/common/referrer.h"
+#include "content/public/common/request_context_type.h"
 #include "content/public/common/stop_find_action.h"
 #include "content/public/renderer/render_frame.h"
 #include "content/renderer/frame_blame_context.h"
@@ -884,6 +885,12 @@ class CONTENT_EXPORT RenderFrameImpl
   void OnFileChooserResponse(
       const std::vector<content::FileChooserFileInfo>& files);
   void OnClearFocusedElement();
+  void OnBlinkFeatureUsageReport(const std::set<int>& features);
+  void OnMixedContentFoundByTheBrowser(const GURL& main_resource_url,

[nasko, 2017/01/12 18:32:38]

Let's remove the "ByTheBrowser" as it doesn't really matter who found it, it
matters that it is found and reported. It might not be the browser process in
the future : ).

[carlosk, 2017/01/21 02:54:59]

Done both for these browser side methods and for their renderer counterparts.

+                                       const GURL& mixed_content_url,
+                                       RequestContextType request_context_type,
+                                       bool was_allowed,
+                                       bool had_redirect);
 #if defined(OS_ANDROID)
   void OnActivateNearestFindResult(int request_id, float x, float y);
   void OnGetNearestFindResult(int request_id, float x, float y);