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

chrome/common/zip.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/zip.h"
 
 #include "base/bind.h"
 #include "base/file_util.h"
 #include "base/logging.h"
 #include "base/string16.h"
@@ skipping 113 matching lines @@
   zipFile zip_file = internal::OpenForZipping(dest_file.AsUTF8Unsafe(),
                                               APPEND_STATUS_CREATE);
 
   if (!zip_file) {
     DLOG(WARNING) << "couldn't create file " << dest_file.value();
     return false;
   }
 
   bool success = true;
   file_util::FileEnumerator file_enumerator(
       src_dir, true,  // recursive

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

nit: consider pushing args back onto line 133.

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

Done.

-      static_cast<file_util::FileEnumerator::FileType>(
-          file_util::FileEnumerator::FILES |
-          file_util::FileEnumerator::DIRECTORIES));
+      (file_util::FileEnumerator::FILES |
+       file_util::FileEnumerator::DIRECTORIES));
   for (FilePath path = file_enumerator.Next(); !path.value().empty();
        path = file_enumerator.Next()) {
     if (!filter_cb.Run(path)) {
       continue;
     }
 
     if (!AddEntryToZip(zip_file, path, src_dir)) {
       success = false;
       return false;
     }
@@ skipping 12 matching lines @@
   if (include_hidden_files) {
     return ZipWithFilterCallback(
         src_dir, dest_file, base::Bind(&ExcludeNoFilesFilter));
   } else {
     return ZipWithFilterCallback(
         src_dir, dest_file, base::Bind(&ExcludeHiddenFilesFilter));
   }
 }
 
 }  // namespace zip