Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3914)

Unified Diff: chrome/browser/net/net_error_tab_helper.cc

Issue 2310583002: Convert some NetError[Tab]Helper messages to mojom (Closed)
Patch Set: rebase Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/net/net_error_tab_helper.cc
diff --git a/chrome/browser/net/net_error_tab_helper.cc b/chrome/browser/net/net_error_tab_helper.cc
index 416251a5238533810dd8935d5032a467f1c9f654..514024a2a3c0f365e1fee79a11dc5760bf319f7e 100644
--- a/chrome/browser/net/net_error_tab_helper.cc
+++ b/chrome/browser/net/net_error_tab_helper.cc
@@ -21,6 +21,8 @@
#include "content/public/browser/navigation_entry.h"
#include "content/public/browser/navigation_handle.h"
#include "content/public/browser/render_frame_host.h"
+#include "content/public/browser/web_contents_interface_registry.h"
+#include "content/public/common/associated_interface_provider.h"
#include "ipc/ipc_message_macros.h"
#include "net/base/net_errors.h"
#include "url/gurl.h"
@@ -98,10 +100,10 @@ void NetErrorTabHelper::RenderFrameCreated(
// platform's network diagnostics dialog.
if (render_frame_host->GetParent())
return;
- render_frame_host->Send(
- new ChromeViewMsg_SetCanShowNetworkDiagnosticsDialog(
- render_frame_host->GetRoutingID(),
- CanShowNetworkDiagnosticsDialog()));
+
+ mojom::NetworkDiagnosticsClientAssociatedPtr client;
+ render_frame_host->GetRemoteAssociatedInterfaces()->GetInterface(&client);
+ client->SetCanShowNetworkDiagnosticsDialog(CanShowNetworkDiagnosticsDialog());
}
void NetErrorTabHelper::DidStartNavigationToPendingEntry(
@@ -181,21 +183,22 @@ bool NetErrorTabHelper::OnMessageReceived(
content::RenderFrameHost* render_frame_host) {
if (render_frame_host != web_contents()->GetMainFrame())
return false;
+#if BUILDFLAG(ANDROID_JAVA_UI)
bool handled = true;
IPC_BEGIN_MESSAGE_MAP(NetErrorTabHelper, message)
- IPC_MESSAGE_HANDLER(ChromeViewHostMsg_RunNetworkDiagnostics,
- RunNetworkDiagnostics)
-#if BUILDFLAG(ANDROID_JAVA_UI)
IPC_MESSAGE_HANDLER(ChromeViewHostMsg_ShowOfflinePages, ShowOfflinePages)
-#endif // BUILDFLAG(ANDROID_JAVA_UI)
IPC_MESSAGE_UNHANDLED(handled = false)
IPC_END_MESSAGE_MAP()
return handled;
+#else
+ return false;
+#endif // BUILDFLAG(ANDROID_JAVA_UI)
}
NetErrorTabHelper::NetErrorTabHelper(WebContents* contents)
: WebContentsObserver(contents),
+ network_diagnostics_binding_(contents, this),
is_error_page_(false),
dns_error_active_(false),
dns_error_page_committed_(false),
@@ -286,12 +289,18 @@ void NetErrorTabHelper::RunNetworkDiagnostics(const GURL& url) {
// any other schemes, but the renderer is not trusted.
if (!url.is_valid() || !url.SchemeIsHTTPOrHTTPS())
return;
+
// Sanitize URL prior to running diagnostics on it.
RunNetworkDiagnosticsHelper(url.GetOrigin().spec());
}
void NetErrorTabHelper::RunNetworkDiagnosticsHelper(
const std::string& sanitized_url) {
+ if (web_contents()->GetInterfaces()->GetCurrentTargetFrame()
+ != web_contents()->GetMainFrame()) {
+ return;
+ }
+
ShowNetworkDiagnosticsDialog(web_contents(), sanitized_url);
}

Powered by Google App Engine
This is Rietveld 408576698