Generalize volume configuration over all volumes.

chrome/browser/chromeos/extensions/file_manager/private_api_misc.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_manager/private_api_misc.h"
 
 #include <set>
 #include <vector>
 
 #include "ash/frame/frame_util.h"
@@ skipping 474 matching lines @@
   using chromeos::file_system_provider::Service;
   using chromeos::file_system_provider::ProvidingExtensionInfo;
   Service* const service = Service::Get(chrome_details_.GetProfile());
 
   if (!service->RequestMount(params->extension_id))
     return RespondNow(Error("Failed to request a new mount."));
 
   return RespondNow(NoArguments());
 }
 
-FileManagerPrivateConfigureProvidedFileSystemFunction::
-    FileManagerPrivateConfigureProvidedFileSystemFunction()
+FileManagerPrivateConfigureVolumeFunction::
+    FileManagerPrivateConfigureVolumeFunction()
     : chrome_details_(this) {
 }
 
 ExtensionFunction::ResponseAction
-FileManagerPrivateConfigureProvidedFileSystemFunction::Run() {
-  using extensions::api::file_manager_private::ConfigureProvidedFileSystem::
-      Params;
+FileManagerPrivateConfigureVolumeFunction::Run() {
+  using extensions::api::file_manager_private::ConfigureVolume::Params;
   const scoped_ptr<Params> params(Params::Create(*args_));
   EXTENSION_FUNCTION_VALIDATE(params);
 
   using file_manager::VolumeManager;
   using file_manager::Volume;
   VolumeManager* const volume_manager =
       VolumeManager::Get(chrome_details_.GetProfile());
   base::WeakPtr<Volume> volume =
       volume_manager->FindVolumeById(params->volume_id);
   if (!volume.get())
     return RespondNow(Error("Volume not found."));
+  if (!volume->configurable())
+    return RespondNow(Error("Volume not configurable."));
 
-  using chromeos::file_system_provider::Service;
-  Service* const service = Service::Get(chrome_details_.GetProfile());
-  DCHECK(service);
+  switch (volume->type()) {
+    case file_manager::VOLUME_TYPE_PROVIDED: {
+      using chromeos::file_system_provider::Service;
+      Service* const service = Service::Get(chrome_details_.GetProfile());
+      DCHECK(service);
 
-  using chromeos::file_system_provider::ProvidedFileSystemInterface;
-  ProvidedFileSystemInterface* const file_system =
-      service->GetProvidedFileSystem(volume->extension_id(),
-                                     volume->file_system_id());
-  DCHECK(file_system);
+      using chromeos::file_system_provider::ProvidedFileSystemInterface;
+      ProvidedFileSystemInterface* const file_system =
+          service->GetProvidedFileSystem(volume->extension_id(),
+                                         volume->file_system_id());
+      if (file_system)
+        file_system->Configure(base::Bind(
+            &FileManagerPrivateConfigureVolumeFunction::OnCompleted, this));
+      break;
+    }
+    default:
+      NOTIMPLEMENTED();
+  }
 
-  file_system->Configure(base::Bind(
-      &FileManagerPrivateConfigureProvidedFileSystemFunction::OnCompleted,
-      this));
   return RespondLater();
 }
 
-void FileManagerPrivateConfigureProvidedFileSystemFunction::OnCompleted(
+void FileManagerPrivateConfigureVolumeFunction::OnCompleted(
     base::File::Error result) {
   if (result != base::File::FILE_OK) {
     Respond(Error("Failed to complete configuration."));
     return;
   }
 
   Respond(NoArguments());
 }
 
 }  // namespace extensions