[fsp] Let extensions decide about the file system id.

chrome/browser/chromeos/extensions/file_system_provider/provider_function.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 "chrome/browser/chromeos/extensions/file_system_provider/file_system_provider_api.h"
 
 #include "chrome/browser/chromeos/file_system_provider/provided_file_system_interface.h"
 #include "chrome/browser/chromeos/file_system_provider/request_manager.h"
 #include "chrome/browser/chromeos/file_system_provider/request_value.h"
 #include "chrome/browser/chromeos/file_system_provider/service.h"
 #include "chrome/common/extensions/api/file_system_provider_internal.h"
 
 using chromeos::file_system_provider::ProvidedFileSystemInterface;
 using chromeos::file_system_provider::RequestManager;
 using chromeos::file_system_provider::RequestValue;
 using chromeos::file_system_provider::Service;
 
 namespace extensions {
 
 const char kNotFoundErrorName[] = "NotFoundError";
 const char kSecurityErrorName[] = "SecurityError";
 
 const char kEmptyNameErrorMessage[] = "Empty display name is not allowed.";
+const char kEmptyIdErrorMessage[] = "Empty file system Id s not allowed.";
 const char kMountFailedErrorMessage[] = "Mounting the file system failed.";
 const char kUnmountFailedErrorMessage[] = "Unmounting the file system failed.";
 const char kResponseFailedErrorMessage[] =
     "Sending a response for the request failed.";
 
 base::DictionaryValue* CreateError(const std::string& name,
                                    const std::string& message) {
   base::DictionaryValue* error = new base::DictionaryValue();
   error->SetString("name", name);
   error->SetString("message", message);
@@ skipping 63 matching lines @@
 bool FileSystemProviderInternalFunction::RunSync() {
   DCHECK(args_);
   if (!Parse())
     return true;
 
   SendResponse(RunWhenValid());
   return true;
 }
 
 bool FileSystemProviderInternalFunction::Parse() {
-  int file_system_id = 0;
+  std::string file_system_id;
 
-  if (!args_->GetInteger(0, &file_system_id) ||
+  if (!args_->GetString(0, &file_system_id) ||
       !args_->GetInteger(1, &request_id_)) {
     bad_message_ = true;
     SendResponse(false);
     return false;
   }
 
   Service* service = Service::Get(GetProfile());
   if (!service) {
     SendResponse(false);
     return false;
@@ skipping 13 matching lines @@
 
 void FileSystemProviderInternalFunction::SetErrorResponse(
     const std::string& name,
     const std::string& message) {
   base::ListValue* result = new base::ListValue();
   result->Append(CreateError(name, message));
   SetResult(result);
 }
 
 }  // namespace extensions