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

chrome/browser/chromeos/login/camera_detector.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/chromeos/login/camera_detector.h"
 
 #include "base/bind.h"
 #include "base/file_util.h"
 #include "base/string_split.h"
 #include "base/string_util.h"
@@ skipping 40 matching lines @@
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
 
   bool present = false;
 
   system::UdevInfoProvider* udev_info = system::UdevInfoProvider::GetInstance();
   // We do a quick check using udev database because opening each /dev/videoX
   // device may trigger costly device initialization.
   using file_util::FileEnumerator;
   FileEnumerator file_enum(
       FilePath(kV4LSubsystemDir),
       false,  // Don't recurse.

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

nit: suggest putting line 61 onto end of line 60.

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

Done.

-      static_cast<FileEnumerator::FileType>(
-          FileEnumerator::FILES | FileEnumerator::SHOW_SYM_LINKS));
+      (FileEnumerator::FILES | FileEnumerator::SHOW_SYM_LINKS));

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

nit: suggest romoving parens around this arg.

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

Done.

   for (FilePath path = file_enum.Next(); !path.empty();
        path = file_enum.Next()) {
     std::string v4l_capabilities;
     if (udev_info->QueryDeviceProperty(path.value(), kV4LCapabilities,
                                        &v4l_capabilities)) {
       std::vector<std::string> caps;
       base::SplitString(v4l_capabilities, kV4LCapabilitiesDelim, &caps);
       if (find(caps.begin(), caps.end(), kV4LCaptureCapability) != caps.end()) {
         present = true;
         break;
       }
     }
   }
 
   camera_presence_ = present ? kCameraPresent : kCameraAbsent;
   presence_check_in_progress_ = false;
 
   DVLOG(1) << "Camera presence state: " << camera_presence_;
 }
 
 }  // namespace chromeos