Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(287)

Unified Diff: media/video/capture/file_video_capture_device_factory.cc

Issue 235353002: Extract VideoCaptureDeviceFactory out of VideoCaptureDevice and use for File and FakeVCD. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: perkj@s comments Created 6 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/video/capture/file_video_capture_device_factory.cc
diff --git a/media/video/capture/file_video_capture_device_factory.cc b/media/video/capture/file_video_capture_device_factory.cc
new file mode 100644
index 0000000000000000000000000000000000000000..0cf92d4be927a9bc096608bc645c5ded839f39e7
--- /dev/null
+++ b/media/video/capture/file_video_capture_device_factory.cc
@@ -0,0 +1,63 @@
+// Copyright 2014 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 "media/video/capture/file_video_capture_device_factory.h"
+
+#include "base/command_line.h"
+#include "base/files/file_path.h"
+#include "base/strings/sys_string_conversions.h"
+#include "media/base/media_switches.h"
+#include "media/video/capture/file_video_capture_device.h"
+
+namespace media {
+
+const char* kFileVideoCaptureDeviceName =
+ "/dev/placeholder-for-file-backed-fake-capture-device";
+
+// Inspects the command line and retrieves the file path parameter.
+base::FilePath GetFilePathFromCommandLine() {
+ base::FilePath command_line_file_path =
+ CommandLine::ForCurrentProcess()->GetSwitchValuePath(
+ switches::kUseFileForFakeVideoCapture);
+ CHECK(!command_line_file_path.empty());
+ return command_line_file_path;
+}
+
+VideoCaptureDevice* FileVideoCaptureDeviceFactory::Create(
+ const VideoCaptureDevice::Name& device_name) {
+#if defined(OS_WIN)
+ return new FileVideoCaptureDevice(
+ base::FilePath(base::SysUTF8ToWide(device_name.name())));
+#else
+ return new FileVideoCaptureDevice(base::FilePath(device_name.name()));
+#endif
+}
+
+void FileVideoCaptureDeviceFactory::GetDeviceNames(
+ VideoCaptureDevice::Names* const device_names) {
+ DCHECK(device_names->empty());
+ base::FilePath command_line_file_path = GetFilePathFromCommandLine();
+#if defined(OS_WIN)
+ device_names->push_back(VideoCaptureDevice::Name(
+ base::SysWideToUTF8(command_line_file_path.value()),
+ kFileVideoCaptureDeviceName));
+#else
+ device_names->push_back(VideoCaptureDevice::Name(
+ command_line_file_path.value(),
+ kFileVideoCaptureDeviceName));
+#endif
+}
+
+void FileVideoCaptureDeviceFactory::GetDeviceSupportedFormats(
+ const VideoCaptureDevice::Name& device,
+ VideoCaptureFormats* supported_formats) {
+ base::File file =
+ FileVideoCaptureDevice::OpenFileForRead(GetFilePathFromCommandLine());
+ VideoCaptureFormat capture_format;
+ FileVideoCaptureDevice::ParseFileAndExtractVideoFormat(&file,
+ &capture_format);
+ supported_formats->push_back(capture_format);
+}
+
+} // namespace media

Powered by Google App Engine
This is Rietveld 408576698