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

chrome/browser/extensions/api/serial/serial_port_enumerator_posix.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/browser/extensions/api/serial/serial_port_enumerator.h"
 
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/string_util.h"
 #include "content/public/browser/browser_thread.h"
@@ skipping 48 matching lines @@
 // Caching seems undesirable. Many devices can be dynamically added to and
 // removed from the system, so we really do want to regenerate the set each
 // time.
 //
 // TODO(miket): this might be refactorable into serial_connection.cc, if
 // Windows serial-port enumeration also entails looking through a directory.
 SerialPortEnumerator::StringSet
 SerialPortEnumerator::GenerateValidSerialPortNames() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
   const FilePath DEV_ROOT("/dev");
-  const file_util::FileEnumerator::FileType FILES_AND_SYM_LINKS =
-      static_cast<file_util::FileEnumerator::FileType>(
-          file_util::FileEnumerator::FILES |
-          file_util::FileEnumerator::SHOW_SYM_LINKS);
+  const int FILES_AND_SYM_LINKS =

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

should be kFilesAndSymLinks

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

I agree. Can I do that in another CL?

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

I guess, though, easy enough to do now if at all.

[jar (doing other things), 2012/08/07 01:47:43]

+1  The name is all in this file... so it should be straight forward.  Probably
change DEV_ROOT while your'e at it.

On 2012/08/06 23:28:18, kalman wrote:

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

Done.
Could you double check?

+      file_util::FileEnumerator::FILES |
+      file_util::FileEnumerator::SHOW_SYM_LINKS;
 
   StringSet valid_patterns = GenerateValidPatterns();
   StringSet name_set;
   file_util::FileEnumerator enumerator(
       DEV_ROOT, false, FILES_AND_SYM_LINKS);
   do {
     const FilePath next_device_path(enumerator.Next());
     const std::string next_device = next_device_path.value();
     if (next_device.empty())
       break;
 
     StringSet::const_iterator i = valid_patterns.begin();
     for (; i != valid_patterns.end(); ++i) {
       if (MatchPattern(next_device, *i)) {
         name_set.insert(next_device);
         break;
       }
     }
   } while (true);
 
   return name_set;
 }
 
 }  // namespace extensions