Speculatively launch Service Workers on mouse/touch events. [3/5]

chrome/browser/renderer_host/chrome_render_message_filter.cc

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 0afbb9429a581889755d620bc2a82f50a07646bd..8a900eae41935d0a683b5b87706c46cd45a5583d 100644
--- a/chrome/browser/renderer_host/chrome_render_message_filter.cc
+++ b/chrome/browser/renderer_host/chrome_render_message_filter.cc
@@ -10,6 +10,7 @@
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
+#include "base/feature_list.h"

[nhiroki, 2016/07/25 06:15:13]

Are this and other header inclusions added by this change necessary?

[horo, 2016/07/25 08:24:26]

Done.

[nhiroki, 2016/07/25 09:26:45]

How about storage_partition.h and content_features.h?

 #include "base/logging.h"
 #include "base/macros.h"
 #include "base/metrics/field_trial.h"
@@ -31,6 +32,8 @@
 #include "components/web_cache/browser/web_cache_manager.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/render_process_host.h"

[nhiroki, 2016/07/25 06:15:13]

#include "content/public/browser/service_worker_context.h"

[horo, 2016/07/25 08:24:26]

Done.

+#include "content/public/browser/storage_partition.h"
+#include "content/public/common/content_features.h"
 
 #if defined(ENABLE_EXTENSIONS)
 #include "extensions/browser/guest_view/web_view/web_view_permission_helper.h"
@@ -46,17 +49,23 @@ const uint32_t kFilteredMessageClasses[] = {
     ChromeMsgStart, ContentSettingsMsgStart, NetworkHintsMsgStart,
 };
 
+void DidStartServiceWorkerForNavigationHint(bool success) {
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+}
+
 }  // namespace
 
-ChromeRenderMessageFilter::ChromeRenderMessageFilter(int render_process_id,
-                                                     Profile* profile)
+ChromeRenderMessageFilter::ChromeRenderMessageFilter(
+    int render_process_id,
+    Profile* profile,
+    content::ServiceWorkerContext* service_worker_context)
     : BrowserMessageFilter(kFilteredMessageClasses,
                            arraysize(kFilteredMessageClasses)),
       render_process_id_(render_process_id),
       profile_(profile),
       predictor_(profile_->GetNetworkPredictor()),
-      cookie_settings_(CookieSettingsFactory::GetForProfile(profile)) {
-}
+      cookie_settings_(CookieSettingsFactory::GetForProfile(profile)),
+      service_worker_context_(service_worker_context) {}
 
 ChromeRenderMessageFilter::~ChromeRenderMessageFilter() {
 }
@@ -94,6 +103,7 @@ 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 defined(ENABLE_PLUGINS)
     case ChromeViewHostMsg_IsCrashReportingEnabled::ID:
 #endif
@@ -134,7 +144,10 @@ void ChromeRenderMessageFilter::OnNavigationHint(
     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)
-  // TODO(horo): Implement this.
+  DCHECK_CURRENTLY_ON(BrowserThread::UI);
+  service_worker_context_->StartServiceWorkerForNavigationHint(
+      url, type, render_process_id_,
+      base::Bind(&DidStartServiceWorkerForNavigationHint));
 }
 
 void ChromeRenderMessageFilter::OnUpdatedCacheStats(