Update mtime of directory when containee changed.

webkit/fileapi/obfuscated_file_util_unittest.cc

 // Copyright (c) 2011 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 <algorithm>
 #include <set>
 #include <string>
 
 #include "base/bind.h"
 #include "base/file_path.h"
@@ skipping 1422 matching lines @@
   EXPECT_EQ(base::PLATFORM_FILE_OK,
             ofu()->GetLocalFilePath(context.get(), kPath[0], &local_path));
   EXPECT_TRUE(file_util::Delete(local_path, false));
 
   context.reset(NewContext(NULL));
   entries.clear();
   EXPECT_EQ(base::PLATFORM_FILE_OK,
             ofu()->ReadDirectory(context.get(), FilePath(), &entries));
   EXPECT_EQ(ARRAYSIZE_UNSAFE(kPath) - 1, entries.size());
 }
+
+TEST_F(ObfuscatedFileUtilTest, TestDirectoryTimestamp) {
+  scoped_ptr<FileSystemOperationContext> context(NewContext(NULL));
+  const FilePath dir_path(FILE_PATH_LITERAL("foo_dir"));
+
+  // Create working directory.
+  EXPECT_EQ(base::PLATFORM_FILE_OK,
+            ofu()->CreateDirectory(context.get(), dir_path, false, false));
+
+  // Reset modification time.
+  context.reset(NewContext(NULL));
+  EXPECT_EQ(base::PLATFORM_FILE_OK,
+            ofu()->Touch(context.get(), dir_path, base::Time(), base::Time()));

[ericu, 2011/11/30 19:16:04]

I'm not sure the round-trip from base::Time to what's stored in the database is
always guaranteed to be lossless.  Please add a call to GetFileInfo and an
EXPECT_EQ(base::Time(), file_info.last_modified) here just to make sure.

[tzik, 2011/12/01 16:28:46]

Done.

+
+  // Create a file in the working directory.
+  bool created = false;
+  context.reset(NewContext(NULL));
+  EXPECT_EQ(base::PLATFORM_FILE_OK,
+            ofu()->EnsureFileExists(context.get(),
+                                    dir_path.AppendASCII("bar file"),
+                                    &created));
+  EXPECT_TRUE(created);
+
+  // Ensure modification time of the parent directory changed.
+  base::PlatformFileInfo file_info;
+  FilePath path;
+  context.reset(NewContext(NULL));
+  EXPECT_EQ(base::PLATFORM_FILE_OK,
+            ofu()->GetFileInfo(context.get(), dir_path, &file_info, &path));
+  EXPECT_NE(base::Time(), file_info.last_modified);

[ericu, 2011/11/30 19:16:04]

Please also add tests for move, mkdir, copy, etc., and check the source
directory of a move as well.

[tzik, 2011/12/01 16:28:46]

Done.

+}