Add an option to specify requested fields for fetching metadata.

content/browser/fileapi/file_system_operation_impl_unittest.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 "storage/browser/fileapi/file_system_operation_impl.h"
 
 #include "base/bind.h"
 #include "base/files/file_util.h"
 #include "base/files/scoped_temp_dir.h"
 #include "base/logging.h"
@@ skipping 367 matching lines @@
     operation_runner()->CreateDirectory(
         url,
         exclusive,
         recursive,
         RecordStatusCallback(run_loop.QuitClosure(), &status));
     run_loop.Run();
     update_observer_.Disable();
     return status;
   }
 
-  base::File::Error GetMetadata(const FileSystemURL& url) {
+  base::File::Error GetMetadata(const FileSystemURL& url, int fields) {
     base::File::Error status;
     base::RunLoop run_loop;
     update_observer_.Enable();
     operation_runner()->GetMetadata(
-        url, RecordMetadataCallback(run_loop.QuitClosure(), &status));
+        url, fields, RecordMetadataCallback(run_loop.QuitClosure(), &status));
     run_loop.Run();
     update_observer_.Disable();
     return status;
   }
 
   base::File::Error ReadDirectory(const FileSystemURL& url) {
     base::File::Error status;
     base::RunLoop run_loop;
     update_observer_.Enable();
     operation_runner()->ReadDirectory(
@@ skipping 527 matching lines @@
   // Dir doesn't exist.
   EXPECT_EQ(base::File::FILE_OK,
             CreateDirectory(URLForPath("new"), true, false));
   EXPECT_TRUE(DirectoryExists("new"));
   EXPECT_EQ(1, change_observer()->get_and_reset_create_directory_count());
   EXPECT_TRUE(change_observer()->HasNoChange());
 }
 
 TEST_F(FileSystemOperationImplTest, TestExistsAndMetadataFailure) {
   EXPECT_EQ(base::File::FILE_ERROR_NOT_FOUND,
-            GetMetadata(URLForPath("nonexistent")));
+            GetMetadata(URLForPath("nonexistent"),
+                        storage::FileSystemOperation::GET_METADATA_FIELD_NONE));
 
   EXPECT_EQ(base::File::FILE_ERROR_NOT_FOUND,
             FileExists(URLForPath("nonexistent")));
 
   EXPECT_EQ(base::File::FILE_ERROR_NOT_FOUND,
             DirectoryExists(URLForPath("nonexistent")));
   EXPECT_TRUE(change_observer()->HasNoChange());
 }
 
 TEST_F(FileSystemOperationImplTest, TestExistsAndMetadataSuccess) {
   FileSystemURL dir(CreateDirectory("dir"));
   FileSystemURL file(CreateFile("dir/file"));
   int read_access = 0;
 
   EXPECT_EQ(base::File::FILE_OK, DirectoryExists(dir));
   ++read_access;
 
-  EXPECT_EQ(base::File::FILE_OK, GetMetadata(dir));
+  EXPECT_EQ(
+      base::File::FILE_OK,
+      GetMetadata(
+          dir, storage::FileSystemOperation::GET_METADATA_FIELD_IS_DIRECTORY));
   EXPECT_TRUE(info().is_directory);
   ++read_access;
 
   EXPECT_EQ(base::File::FILE_OK, FileExists(file));
   ++read_access;
 
-  EXPECT_EQ(base::File::FILE_OK, GetMetadata(file));
+  EXPECT_EQ(
+      base::File::FILE_OK,
+      GetMetadata(
+          file, storage::FileSystemOperation::GET_METADATA_FIELD_IS_DIRECTORY));
   EXPECT_FALSE(info().is_directory);
   ++read_access;
 
   EXPECT_EQ(read_access,
             quota_manager_proxy()->notify_storage_accessed_count());
   EXPECT_TRUE(change_observer()->HasNoChange());
 }
 
 TEST_F(FileSystemOperationImplTest, TestTypeMismatchErrors) {
   FileSystemURL dir(CreateDirectory("dir"));
@@ skipping 93 matching lines @@
 TEST_F(FileSystemOperationImplTest, TestTruncate) {
   FileSystemURL file(CreateFile("file"));
   base::FilePath platform_path = PlatformPath("file");
 
   char test_data[] = "test data";
   int data_size = static_cast<int>(sizeof(test_data));
   EXPECT_EQ(data_size,
             base::WriteFile(platform_path, test_data, data_size));
 
   // Check that its length is the size of the data written.
-  EXPECT_EQ(base::File::FILE_OK, GetMetadata(file));
+  EXPECT_EQ(
+      base::File::FILE_OK,
+      GetMetadata(
+          file, storage::FileSystemOperation::GET_METADATA_FIELD_IS_DIRECTORY |
+                    storage::FileSystemOperation::GET_METADATA_FIELD_SIZE));
   EXPECT_FALSE(info().is_directory);
   EXPECT_EQ(data_size, info().size);
 
   // Extend the file by truncating it.
   int length = 17;
   EXPECT_EQ(base::File::FILE_OK, Truncate(file, length));
 
   EXPECT_EQ(1, change_observer()->get_and_reset_modify_file_count());
   EXPECT_TRUE(change_observer()->HasNoChange());
 
@@ skipping 195 matching lines @@
   FileSystemURL src_file(CreateFile("src"));
   FileSystemURL dest_file(CreateFile("dest"));
 
   EXPECT_EQ(base::File::FILE_OK, Truncate(dest_file, 6));
   EXPECT_EQ(base::File::FILE_OK,
             Copy(src_file, dest_file, FileSystemOperation::OPTION_NONE));
   EXPECT_EQ(0, GetFileSize("dest"));
 }
 
 }  // namespace content