Change the type of file_type parameter to int, as the parameter actually takes or-ed bitmasks,

chrome/browser/extensions/sandboxed_unpacker_unittest.cc

 // Copyright (c) 2012 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 "base/file_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/message_loop.h"
 #include "base/path_service.h"
 #include "base/scoped_temp_dir.h"
 #include "base/scoped_temp_dir.h"
@@ skipping 117 matching lines @@
         *unpacker_->parsed_manifest());
   }
 
   FilePath GetInstallPath() {
     return temp_dir_.path().AppendASCII(
         extension_filenames::kTempExtensionName);
   }
 
   bool TempFilesRemoved() {
     // Check that temporary files were cleaned up.
-    file_util::FileEnumerator::FileType files_and_dirs =
-      static_cast<file_util::FileEnumerator::FileType>(
+    int files_and_dirs =

[not at google - send to devlin, 2012/08/06 05:04:56]

should be kFilesAndDirs

[Haruki Sato, 2012/08/06 23:22:18]

Good catch. I will follow up with another CL.

         file_util::FileEnumerator::DIRECTORIES |
-        file_util::FileEnumerator::FILES);
+        file_util::FileEnumerator::FILES;

[jar (doing other things), 2012/08/06 18:27:02]

nit: push either line 140 or 139 onto the end of 138.

[Haruki Sato, 2012/08/06 23:22:18]

Done.

 
     file_util::FileEnumerator temp_iterator(
       temp_path_,
       true,  // recursive
       files_and_dirs
     );
     int items_not_removed = 0;
     FilePath item_in_temp;
     item_in_temp = temp_iterator.Next();
     while (!item_in_temp.value().empty()) {
@@ skipping 64 matching lines @@
   // Check that there is newer _locales/en_US/messages.json file.
   base::PlatformFileInfo new_info;
   EXPECT_TRUE(file_util::GetFileInfo(messages_file, &new_info));
 
   EXPECT_TRUE(new_info.last_modified > old_info.last_modified);
 
   ASSERT_TRUE(TempFilesRemoved());
 }
 
 }  // namespace extensions