Add an option to specify requested fields for fetching metadata.

chrome/browser/chromeos/file_system_provider/fileapi/provider_async_file_util.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/file_system_provider/fileapi/provider_async_file_util.h"
 
 #include "base/callback.h"
 #include "base/files/file.h"
 #include "base/files/file_path.h"
 #include "base/logging.h"
 #include "chrome/browser/chromeos/file_system_provider/mount_path_util.h"
 #include "chrome/browser/chromeos/file_system_provider/provided_file_system_interface.h"
 #include "content/public/browser/browser_thread.h"
 #include "storage/browser/blob/shareable_file_reference.h"
 #include "storage/browser/fileapi/file_system_operation_context.h"
 #include "storage/browser/fileapi/file_system_url.h"
 
 using content::BrowserThread;
 
 namespace chromeos {
 namespace file_system_provider {
 namespace internal {
 namespace {
 
 // Executes GetFileInfo on the UI thread.
 void GetFileInfoOnUIThread(
     scoped_ptr<storage::FileSystemOperationContext> context,
     const storage::FileSystemURL& url,
+    int /* fields */,
     const ProvidedFileSystemInterface::GetMetadataCallback& callback) {
   util::FileSystemURLParser parser(url);
   if (!parser.Parse()) {
     callback.Run(make_scoped_ptr<EntryMetadata>(NULL),
                  base::File::FILE_ERROR_INVALID_OPERATION);
     return;
   }
 
+  // TODO(mtomasz): Pass fields to FSP extensions so only requested fields are
+  // returned.
   parser.file_system()->GetMetadata(
       parser.file_path(),
       ProvidedFileSystemInterface::METADATA_FIELD_DEFAULT,
       callback);
 }
 
 // Routes the response of GetFileInfo back to the IO thread with a type
 // conversion.
 void OnGetFileInfo(const storage::AsyncFileUtil::GetFileInfoCallback& callback,
                    scoped_ptr<EntryMetadata> metadata,
@@ skipping 263 matching lines @@
                  base::Passed(&context),
                  url,
                  exclusive,
                  recursive,
                  base::Bind(&OnCreateDirectory, exclusive, callback)));
 }
 
 void ProviderAsyncFileUtil::GetFileInfo(
     scoped_ptr<storage::FileSystemOperationContext> context,
     const storage::FileSystemURL& url,
+    int fields,
     const GetFileInfoCallback& callback) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
-  BrowserThread::PostTask(BrowserThread::UI,
-                          FROM_HERE,
-                          base::Bind(&GetFileInfoOnUIThread,
-                                     base::Passed(&context),
-                                     url,
-                                     base::Bind(&OnGetFileInfo, callback)));
+  BrowserThread::PostTask(
+      BrowserThread::UI, FROM_HERE,
+      base::Bind(&GetFileInfoOnUIThread, base::Passed(&context), url, fields,
+                 base::Bind(&OnGetFileInfo, callback)));
 }
 
 void ProviderAsyncFileUtil::ReadDirectory(
     scoped_ptr<storage::FileSystemOperationContext> context,
     const storage::FileSystemURL& url,
     const ReadDirectoryCallback& callback) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   BrowserThread::PostTask(BrowserThread::UI,
                           FROM_HERE,
                           base::Bind(&ReadDirectoryOnUIThread,
@@ skipping 123 matching lines @@
   NOTIMPLEMENTED();
   callback.Run(base::File::FILE_ERROR_INVALID_OPERATION,
                base::File::Info(),
                base::FilePath(),
                scoped_refptr<storage::ShareableFileReference>());
 }
 
 }  // namespace internal
 }  // namespace file_system_provider
 }  // namespace chromeos