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

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

Issue 637953008: Improve video capture columns in chrome://media-internals (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Move to impl, update comments, remove pixel format Created 6 years, 2 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/video_capture_device.cc
diff --git a/media/video/capture/video_capture_device.cc b/media/video/capture/video_capture_device.cc
index 3b74417cbf823c0e9d235953809fb40a2cadaaef..a7adf2edff35fd58f78a5825e7ccaed77f012ce1 100644
--- a/media/video/capture/video_capture_device.cc
+++ b/media/video/capture/video_capture_device.cc
@@ -57,6 +57,36 @@ VideoCaptureDevice::Name::Name(const std::string& name,
VideoCaptureDevice::Name::~Name() {}
+#if defined(OS_WIN)
+const char* VideoCaptureDevice::Name::GetCaptureApiTypeString() const {
+ switch(capture_api_type()) {
+ case MEDIA_FOUNDATION:
+ return "MEDIA_FOUNDATION";
+ case DIRECT_SHOW:
+ return "DIRECT_SHOW";
+ case DIRECT_SHOW_WDM_CROSSBAR:
+ return "DIRECT_SHOW_WDM_CROSSBAR";
+ default:
+ NOTREACHED() << "Unknown Video Capture API type!";
+ return "API_TYPE_UNKNOWN";
xhwang 2014/10/21 19:07:49 ditto about readable names.
burnik 2014/10/21 19:37:11 Done.
+ }
+}
+#elif defined(OS_MACOSX)
+const char* VideoCaptureDevice::Name::GetCaptureApiTypeString() const {
+ switch(capture_api_type()) {
+ case AVFOUNDATION:
+ return "AVFOUNDATION";
+ case QTKIT:
+ return "QTKIT";
+ case DECKLINK:
+ return "DECKLINK";
+ default:
+ NOTREACHED() << "Unknown Video Capture API type!";
+ return "API_TYPE_UNKNOWN";
+ }
+}
+#endif
+
VideoCaptureDevice::~VideoCaptureDevice() {}
int VideoCaptureDevice::GetPowerLineFrequencyForLocation() const {

Powered by Google App Engine
This is Rietveld 408576698