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
horo
2016/07/25 08:24:26
Done.
nhiroki
2016/07/25 09:26:45
How about storage_partition.h and content_features
|
#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_co
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( |