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

Unified Diff: chrome/browser/renderer_host/chrome_render_message_filter.cc

Issue 2144533002: Convert network hints to Mojo (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address sammc's review comments Created 4 years, 5 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/renderer_host/chrome_render_message_filter.cc
diff --git a/chrome/browser/renderer_host/chrome_render_message_filter.cc b/chrome/browser/renderer_host/chrome_render_message_filter.cc
index 1a48364b54c7839d39063d124d3d35240d9f2a58..d8952d9ac7e514fb4b3038b689748338757b557c 100644
--- a/chrome/browser/renderer_host/chrome_render_message_filter.cc
+++ b/chrome/browser/renderer_host/chrome_render_message_filter.cc
@@ -25,7 +25,6 @@
#include "components/content_settings/content/common/content_settings_messages.h"
#include "components/content_settings/core/browser/cookie_settings.h"
#include "components/network_hints/common/network_hints_common.h"
-#include "components/network_hints/common/network_hints_messages.h"
#include "components/rappor/rappor_service.h"
#include "components/rappor/rappor_utils.h"
#include "components/web_cache/browser/web_cache_manager.h"
@@ -42,9 +41,8 @@ using content::BrowserThread;
namespace {
-const uint32_t kFilteredMessageClasses[] = {
- ChromeMsgStart, ContentSettingsMsgStart, NetworkHintsMsgStart,
-};
+const uint32_t kFilteredMessageClasses[] = {ChromeMsgStart,
+ ContentSettingsMsgStart};
} // namespace
@@ -64,8 +62,6 @@ ChromeRenderMessageFilter::~ChromeRenderMessageFilter() {
bool ChromeRenderMessageFilter::OnMessageReceived(const IPC::Message& message) {
bool handled = true;
IPC_BEGIN_MESSAGE_MAP(ChromeRenderMessageFilter, message)
- IPC_MESSAGE_HANDLER(NetworkHintsMsg_DNSPrefetch, OnDnsPrefetch)
- IPC_MESSAGE_HANDLER(NetworkHintsMsg_Preconnect, OnPreconnect)
IPC_MESSAGE_HANDLER(ChromeViewHostMsg_UpdatedCacheStats,
OnUpdatedCacheStats)
IPC_MESSAGE_HANDLER(ChromeViewHostMsg_AllowDatabase, OnAllowDatabase)
@@ -106,17 +102,17 @@ void ChromeRenderMessageFilter::OverrideThreadForMessage(
}
}
-void ChromeRenderMessageFilter::OnDnsPrefetch(
+void ChromeRenderMessageFilter::DNSPrefetch(
const network_hints::LookupRequest& request) {
if (predictor_)
predictor_->DnsPrefetchList(request.hostname_list);
}
-void ChromeRenderMessageFilter::OnPreconnect(const GURL& url,
- bool allow_credentials,
- int count) {
+void ChromeRenderMessageFilter::Preconnect(const GURL& url,
+ bool allow_credentials,
+ int count) {
if (count < 1) {
- LOG(WARNING) << "NetworkHintsMsg_Preconnect IPC with invalid count: "
+ LOG(WARNING) << "NetworkHints::Preconnect IPC with invalid count: "
<< count;
return;
}
@@ -128,6 +124,11 @@ void ChromeRenderMessageFilter::OnPreconnect(const GURL& url,
}
}
+void ChromeRenderMessageFilter::BindNetworkHints(
+ network_hints::mojom::NetworkHintsRequest request) {
+ bindings_.AddBinding(this, std::move(request));
+}
+
void ChromeRenderMessageFilter::OnUpdatedCacheStats(
uint64_t min_dead_capacity,
uint64_t max_dead_capacity,

Powered by Google App Engine
This is Rietveld 408576698