[ServiceWorker] Support other methods than GET and HEAD.

content/browser/service_worker/service_worker_request_handler.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/service_worker/service_worker_request_handler.h"
 
 #include <string>
 
 #include "content/browser/service_worker/service_worker_context_core.h"
 #include "content/browser/service_worker/service_worker_context_wrapper.h"
@@ skipping 24 matching lines @@
         ServiceWorkerRequestHandler::GetHandler(request);
     if (!handler)
       return NULL;
     return handler->MaybeCreateJob(request, network_delegate);
   }
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ServiceWorkerRequestInterceptor);
 };
 
-bool IsMethodSupportedForAppCache(const std::string& method) {
-  return (method == "GET") || (method == "HEAD");
-}
-
-bool IsSchemeAndMethodSupportedForAppCache(const net::URLRequest* request) {
-  return request->url().SchemeIsHTTPOrHTTPS() &&
-         IsMethodSupportedForAppCache(request->method());
-}
-
 }  // namespace
 
 void ServiceWorkerRequestHandler::InitializeHandler(
     net::URLRequest* request,
     ServiceWorkerContextWrapper* context_wrapper,
     webkit_blob::BlobStorageContext* blob_storage_context,
     int process_id,
     int provider_id,
     ResourceType resource_type) {
-  if (!IsSchemeAndMethodSupportedForAppCache(request)) {

[falken, 2014/08/18 06:31:46]

I was confused by this AppCache name. It was probably a mistake from
https://codereview.chromium.org/344493002

[horo, 2014/08/18 06:49:44]

I think ServiceWorkerRequestHandler was initially created by copying
AppCacheRequestHandler.

+  if (!request->url().SchemeIsHTTPOrHTTPS())
     return;
-  }
 
   if (!context_wrapper || !context_wrapper->context() ||
       provider_id == kInvalidServiceWorkerProviderId) {
     return;
   }
 
   ServiceWorkerProviderHost* provider_host =
       context_wrapper->context()->GetProviderHost(process_id, provider_id);
   if (!provider_host || !provider_host->IsContextAlive())
     return;
@@ skipping 27 matching lines @@
     base::WeakPtr<ServiceWorkerProviderHost> provider_host,
     base::WeakPtr<webkit_blob::BlobStorageContext> blob_storage_context,
     ResourceType resource_type)
     : context_(context),
       provider_host_(provider_host),
       blob_storage_context_(blob_storage_context),
       resource_type_(resource_type) {
 }
 
 }  // namespace content