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

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

Issue 2778173002: Revert of "Speculatively launch Service Workers on mouse/touch events." (Closed)
Patch Set: rebase Created 3 years, 9 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 b866521f6284232c4374fd6afb6489ec4f8aaa1b..25d08d97f8c6970613ec011a0139e71afa8f58b5 100644
--- a/chrome/browser/renderer_host/chrome_render_message_filter.cc
+++ b/chrome/browser/renderer_host/chrome_render_message_filter.cc
@@ -27,7 +27,6 @@
#include "components/web_cache/browser/web_cache_manager.h"
#include "content/public/browser/notification_service.h"
#include "content/public/browser/render_process_host.h"
-#include "content/public/browser/service_worker_context.h"
#include "extensions/features/features.h"
#include "ppapi/features/features.h"
@@ -45,23 +44,16 @@ const uint32_t kFilteredMessageClasses[] = {
ChromeMsgStart, NetworkHintsMsgStart,
};
-void DidStartServiceWorkerForNavigationHint(bool success) {
- DCHECK_CURRENTLY_ON(BrowserThread::UI);
-}
-
} // namespace
-ChromeRenderMessageFilter::ChromeRenderMessageFilter(
- int render_process_id,
- Profile* profile,
- content::ServiceWorkerContext* service_worker_context)
+ChromeRenderMessageFilter::ChromeRenderMessageFilter(int render_process_id,
+ Profile* profile)
: BrowserMessageFilter(kFilteredMessageClasses,
arraysize(kFilteredMessageClasses)),
render_process_id_(render_process_id),
profile_(profile),
predictor_(profile_->GetNetworkPredictor()),
- cookie_settings_(CookieSettingsFactory::GetForProfile(profile)),
- service_worker_context_(service_worker_context) {}
+ cookie_settings_(CookieSettingsFactory::GetForProfile(profile)) {}
ChromeRenderMessageFilter::~ChromeRenderMessageFilter() {
}
@@ -71,7 +63,6 @@ bool ChromeRenderMessageFilter::OnMessageReceived(const IPC::Message& message) {
IPC_BEGIN_MESSAGE_MAP(ChromeRenderMessageFilter, message)
IPC_MESSAGE_HANDLER(NetworkHintsMsg_DNSPrefetch, OnDnsPrefetch)
IPC_MESSAGE_HANDLER(NetworkHintsMsg_Preconnect, OnPreconnect)
- IPC_MESSAGE_HANDLER(NetworkHintsMsg_NavigationHint, OnNavigationHint)
IPC_MESSAGE_HANDLER(ChromeViewHostMsg_UpdatedCacheStats,
OnUpdatedCacheStats)
IPC_MESSAGE_HANDLER(ChromeViewHostMsg_AllowDatabase, OnAllowDatabase)
@@ -95,7 +86,6 @@ bool ChromeRenderMessageFilter::OnMessageReceived(const IPC::Message& message) {
void ChromeRenderMessageFilter::OverrideThreadForMessage(
const IPC::Message& message, BrowserThread::ID* thread) {
switch (message.type()) {
- case NetworkHintsMsg_NavigationHint::ID:
#if BUILDFLAG(ENABLE_PLUGINS)
case ChromeViewHostMsg_IsCrashReportingEnabled::ID:
#endif
@@ -129,17 +119,6 @@ void ChromeRenderMessageFilter::OnPreconnect(const GURL& url,
}
}
-void ChromeRenderMessageFilter::OnNavigationHint(
- const GURL& url,
- blink::WebNavigationHintType type) {
- // TODO(horo): We don't need to have this method in //chrome. Move it to
- // //content while mojoifing network_hints. (crbug.com/610750)
- DCHECK_CURRENTLY_ON(BrowserThread::UI);
- service_worker_context_->StartServiceWorkerForNavigationHint(
- url, type, render_process_id_,
- base::Bind(&DidStartServiceWorkerForNavigationHint));
-}
-
void ChromeRenderMessageFilter::OnUpdatedCacheStats(uint64_t capacity,
uint64_t size) {
web_cache::WebCacheManager::GetInstance()->ObserveStats(render_process_id_,
« no previous file with comments | « chrome/browser/renderer_host/chrome_render_message_filter.h ('k') | components/network_hints/common/network_hints_messages.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698