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

Unified Diff: content/browser/renderer_host/media/web_contents_video_capture_device.cc

Issue 17846002: Refactor the VideoCaptureDevice::Name struct. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address comments Created 7 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
« no previous file with comments | « content/browser/renderer_host/media/video_capture_manager.cc ('k') | media/media.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/renderer_host/media/web_contents_video_capture_device.cc
diff --git a/content/browser/renderer_host/media/web_contents_video_capture_device.cc b/content/browser/renderer_host/media/web_contents_video_capture_device.cc
index 42868c0a85fa83fd1f4203bacbf6c4075bc5c273..5cc0f42cb7022a8178be09811912afa0f3267162 100644
--- a/content/browser/renderer_host/media/web_contents_video_capture_device.cc
+++ b/content/browser/renderer_host/media/web_contents_video_capture_device.cc
@@ -1237,14 +1237,13 @@ media::VideoCaptureDevice* WebContentsVideoCaptureDevice::Create(
&render_view_id))
return NULL;
- media::VideoCaptureDevice::Name name;
- base::SStringPrintf(&name.device_name,
+ std::string device_name;
+ base::SStringPrintf(&device_name,
"WebContents[%.*s]",
static_cast<int>(device_id.size()), device_id.data());
- name.unique_id = device_id;
-
return new WebContentsVideoCaptureDevice(
- name, render_process_id, render_view_id);
+ media::VideoCaptureDevice::Name(device_name, device_id),
+ render_process_id, render_view_id);
}
void WebContentsVideoCaptureDevice::Allocate(
« no previous file with comments | « content/browser/renderer_host/media/video_capture_manager.cc ('k') | media/media.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698