[fsp] Decouple file_service_provider::Service.

chrome/browser/chromeos/extensions/file_system_provider/file_system_provider_api.cc

 // Copyright 2013 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 "chrome/browser/chromeos/extensions/file_system_provider/file_system_provider_api.h"
 
 #include <string>
 
 #include "base/values.h"
 #include "chrome/browser/chromeos/file_system_provider/service.h"
@@ skipping 101 matching lines @@
     result->Append(CreateError(kSecurityErrorName, kMountFailedErrorMessage));
     SetResult(result);
     return false;
   }
 
   base::ListValue* result = new base::ListValue();
   result->Append(new base::FundamentalValue(file_system_id));
   // Don't append an error on success.
 
   SetResult(result);
-  SendResponse(true);
   return true;
 }
 
 bool FileSystemProviderUnmountFunction::RunImpl() {
   using api::file_system_provider::Unmount::Params;
   const scoped_ptr<Params> params(Params::Create(*args_));
   EXTENSION_FUNCTION_VALIDATE(params);
 
   chromeos::file_system_provider::Service* service =
       chromeos::file_system_provider::Service::Get(GetProfile());
   DCHECK(service);
 
   if (!service->UnmountFileSystem(extension_id(), params->file_system_id)) {
     // TODO(mtomasz): Pass more detailed errors, rather than just a bool.
     base::ListValue* result = new base::ListValue();
     result->Append(CreateError(kSecurityErrorName, kUnmountFailedErrorMessage));
     SetResult(result);
     return false;
   }
 
   base::ListValue* result = new base::ListValue();
   SetResult(result);
-  SendResponse(true);
   return true;
 }
 
 bool FileSystemProviderInternalUnmountRequestedSuccessFunction::RunImpl() {
   using api::file_system_provider_internal::UnmountRequestedSuccess::Params;
   const scoped_ptr<Params> params(Params::Create(*args_));
   EXTENSION_FUNCTION_VALIDATE(params);
 
   chromeos::file_system_provider::Service* service =
       chromeos::file_system_provider::Service::Get(GetProfile());
   DCHECK(service);
 
-  if (!service->FulfillRequest(extension_id(),
-                               params->file_system_id,
-                               params->request_id,
-                               scoped_ptr<base::DictionaryValue>(),
-                               false /* has_more */)) {
+  if (!service->request_manager()->FulfillRequest(
+          extension_id(),
+          params->file_system_id,
+          params->request_id,
+          scoped_ptr<base::DictionaryValue>(),
+          false /* has_more */)) {
     // TODO(mtomasz): Pass more detailed errors, rather than just a bool.
     base::ListValue* result = new base::ListValue();
     result->Append(
         CreateError(kSecurityErrorName, kResponseFailedErrorMessage));
     SetResult(result);
     return false;
   }
 
   base::ListValue* result = new base::ListValue();
   SetResult(result);
-  SendResponse(true);
   return true;
 }
 
 bool FileSystemProviderInternalUnmountRequestedErrorFunction::RunImpl() {
   using api::file_system_provider_internal::UnmountRequestedError::Params;
   const scoped_ptr<Params> params(Params::Create(*args_));
   EXTENSION_FUNCTION_VALIDATE(params);
 
   chromeos::file_system_provider::Service* service =
       chromeos::file_system_provider::Service::Get(GetProfile());
   DCHECK(service);
 
   // Currently it is not possible to refer to types/enums defined in a different
   // IDL file. Therefore we need to convert DOMString to ProviderError, since
   // UnmountRequestedErrorFunction() is defined in a different namespace than
   // ProvidedError.
   // TODO(mtomasz): Remove this trick, once IDL supports namespaces correctly.
   const api::file_system_provider::ProviderError provider_error =
       api::file_system_provider::ParseProviderError(params->error);
 
-  if (!service->RejectRequest(extension_id(),
-                              params->file_system_id,
-                              params->request_id,
-                              ProviderErrorToFileError(provider_error))) {
+  if (!service->request_manager()->RejectRequest(
+          extension_id(),
+          params->file_system_id,
+          params->request_id,
+          ProviderErrorToFileError(provider_error))) {
     // TODO(mtomasz): Pass more detailed errors, rather than just a bool.
     base::ListValue* result = new base::ListValue();
     result->Append(
         CreateError(kSecurityErrorName, kResponseFailedErrorMessage));
     SetResult(result);
     return false;
   }
 
   base::ListValue* result = new base::ListValue();
   SetResult(result);
-  SendResponse(true);
   return true;
 }
 
 }  // namespace extensions