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

chrome/common/extensions/extension_file_util.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 "chrome/common/extensions/extension_file_util.h"
 
 #include <map>
 #include <vector>
 
 #include "base/file_path.h"
@@ skipping 636 matching lines @@
   CR_DEFINE_STATIC_LOCAL(
       std::set<FilePath::StringType>, reserved_underscore_names,
       (reserved_names, reserved_names + arraysize(reserved_names)));
 
   // Enumerate all files and directories in the extension root.
   // There is a problem when using pattern "_*" with FileEnumerator, so we have
   // to cheat with find_first_of and match all.
   file_util::FileEnumerator all_files(
     extension_path,
     false,
-    static_cast<file_util::FileEnumerator::FileType>(
-        file_util::FileEnumerator::DIRECTORIES |
-          file_util::FileEnumerator::FILES));
+    (file_util::FileEnumerator::DIRECTORIES |
+     file_util::FileEnumerator::FILES));

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

extra parens seem unnecessary

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

Ack. replied to another comment.

[not at google - send to devlin, 2012/08/06 23:28:18]

You could assign to a kFilesAndDirectories constant like the other examples?

[Haruki Sato, 2012/08/07 02:29:17]

Done.

 
   FilePath file;
   while (!(file = all_files.Next()).empty()) {
     FilePath::StringType filename = file.BaseName().value();
     // Skip all that don't start with "_".
     if (filename.find_first_of(FILE_PATH_LITERAL("_")) != 0) continue;
     if (reserved_underscore_names.find(filename) ==
         reserved_underscore_names.end()) {
       *error = base::StringPrintf(
           "Cannot load extension with file or directory name %s. "
@@ skipping 118 matching lines @@
     return temp_path;
 
   return FilePath();
 }
 
 void DeleteFile(const FilePath& path, bool recursive) {
   file_util::Delete(path, recursive);
 }
 
 }  // namespace extension_file_util