Mostly manual refactoring of ServiceWorkerRequestHandler into a base class and two derived classes,…

content/browser/service_worker/service_worker_provider_host.h

Index: content/browser/service_worker/service_worker_provider_host.h
diff --git a/content/browser/service_worker/service_worker_provider_host.h b/content/browser/service_worker/service_worker_provider_host.h
index 1ca21b063eafeb3599d5cc1fdba487002379b771..10977facf731a10360ed3b0c3782278c64935b11 100644
--- a/content/browser/service_worker/service_worker_provider_host.h
+++ b/content/browser/service_worker/service_worker_provider_host.h
@@ -20,6 +20,7 @@ namespace content {
 
 class ServiceWorkerContextCore;
 class ServiceWorkerDispatcherHost;
+class ServiceWorkerRequestHandler;
 class ServiceWorkerVersion;
 
 // This class is the browser-process representation of a serice worker
@@ -79,9 +80,10 @@ class CONTENT_EXPORT ServiceWorkerProviderHost
   // our process state. That would be indicative of a bad IPC message.
   bool SetHostedVersionId(int64 versions_id);
 
-  // Returns true if this provider host should handle requests for
-  // |resource_type|.
-  bool ShouldHandleRequest(ResourceType::Type resource_type) const;
+  // Returns a handler for the request, may return NULL if the request
+  // doesn't require special handling.
+  ServiceWorkerRequestHandler* CreateRequestHandler(
+      ResourceType::Type resource_type);

[kinuko, 2014/04/24 05:15:00]

nit: this could probably return already scoped_ptr'ed one

[michaeln, 2014/04/24 19:01:19]

Done

 
  private:
   const int process_id_;