diff options
author | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-05-07 22:51:31 +0000 |
---|---|---|
committer | brettw@chromium.org <brettw@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-05-07 22:51:31 +0000 |
commit | fcb524ead399b591b4e6587c2fbe712e45126320 (patch) | |
tree | 77da32bc4e3ab916de166e0f9e52e0ad61629efa /media/video | |
parent | 3afef01773fcfe63523ebede8ad5a4dfed10088b (diff) | |
download | chromium_src-fcb524ead399b591b4e6587c2fbe712e45126320.zip chromium_src-fcb524ead399b591b4e6587c2fbe712e45126320.tar.gz chromium_src-fcb524ead399b591b4e6587c2fbe712e45126320.tar.bz2 |
Move FileEnumerator to its own file, do some refactoring.
It creates a class FileInfo to contain the details rather than using a platform-specific typedef. This allows the accessors GetName, GetSize, etc. to be moved directly to this class (previously they were static helpers on the FileEnumerator class) which makes a bunch of code much cleaner. It also gives reasonable getting and initialization which the previous version lacked.
BUG=175002
R=rvargas@chromium.org
Review URL: https://codereview.chromium.org/13165005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@198820 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'media/video')
-rw-r--r-- | media/video/capture/linux/video_capture_device_linux.cc | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/media/video/capture/linux/video_capture_device_linux.cc b/media/video/capture/linux/video_capture_device_linux.cc index cf53ce0..6c9047e 100644 --- a/media/video/capture/linux/video_capture_device_linux.cc +++ b/media/video/capture/linux/video_capture_device_linux.cc @@ -19,6 +19,7 @@ #include "base/bind.h" #include "base/file_util.h" +#include "base/files/file_enumerator.h" #include "base/stringprintf.h" namespace media { @@ -100,15 +101,14 @@ void VideoCaptureDevice::GetDeviceNames(Names* device_names) { device_names->clear(); base::FilePath path("/dev/"); - file_util::FileEnumerator enumerator( - path, false, file_util::FileEnumerator::FILES, "video*"); + base::FileEnumerator enumerator( + path, false, base::FileEnumerator::FILES, "video*"); while (!enumerator.Next().empty()) { - file_util::FileEnumerator::FindInfo info; - enumerator.GetFindInfo(&info); + base::FileEnumerator::FileInfo info = enumerator.GetInfo(); Name name; - name.unique_id = path.value() + info.filename; + name.unique_id = path.value() + info.GetName().value(); if ((fd = open(name.unique_id.c_str() , O_RDONLY)) < 0) { // Failed to open this device. continue; @@ -123,7 +123,7 @@ void VideoCaptureDevice::GetDeviceNames(Names* device_names) { name.device_name = base::StringPrintf("%s", cap.card); device_names->push_back(name); } else { - DVLOG(1) << "No usable formats reported by " << info.filename; + DVLOG(1) << "No usable formats reported by " << info.GetName().value(); } } close(fd); |