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

Unified Diff: media/video/capture/mac/video_capture_device_avfoundation_mac.mm

Issue 366593003: Mac VideoCapture: return empty GetModel() for non-USB non-built-in cameras. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rsesek@ comments Created 6 years, 6 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/mac/video_capture_device_avfoundation_mac.mm
diff --git a/media/video/capture/mac/video_capture_device_avfoundation_mac.mm b/media/video/capture/mac/video_capture_device_avfoundation_mac.mm
index 2412aac91d483384a6bc990a57f2243224a807cb..f29edc11d5e43f6fac43c3b0c90c3109eb03bd95 100644
--- a/media/video/capture/mac/video_capture_device_avfoundation_mac.mm
+++ b/media/video/capture/mac/video_capture_device_avfoundation_mac.mm
@@ -23,7 +23,11 @@
if (([device hasMediaType:AVFoundationGlue::AVMediaTypeVideo()] ||
[device hasMediaType:AVFoundationGlue::AVMediaTypeMuxed()]) &&
![device isSuspended]) {
- [deviceNames setObject:[device localizedName]
+ DeviceNameAndTransportType* name_and_transport_type =
Robert Sesek 2014/07/01 16:23:57 naming: nameAndTransportType, since this is in an
mcasas 2014/07/02 07:37:35 Done.
+ [[[DeviceNameAndTransportType alloc]
+ initWithName:[device localizedName]
+ transportType:[device transportType]] autorelease];
+ [deviceNames setObject:name_and_transport_type
forKey:[device uniqueID]];
}
}

Powered by Google App Engine
This is Rietveld 408576698