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

Side by Side Diff: content/public/common/media_stream_request.cc

Issue 34393006: Refactor MediaStreamManager to never output real device id. It now always output sourceId in the fo… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix broken video_capture_host unittests. Addressed justinlinlins comments. Created 7 years, 1 month 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/public/common/media_stream_request.h" 5 #include "content/public/common/media_stream_request.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 8
9 namespace content { 9 namespace content {
10 10
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
55 name(name), 55 name(name),
56 input(sample_rate, channel_layout, frames_per_buffer) { 56 input(sample_rate, channel_layout, frames_per_buffer) {
57 } 57 }
58 58
59 MediaStreamDevice::~MediaStreamDevice() {} 59 MediaStreamDevice::~MediaStreamDevice() {}
60 60
61 MediaStreamRequest::MediaStreamRequest( 61 MediaStreamRequest::MediaStreamRequest(
62 int render_process_id, 62 int render_process_id,
63 int render_view_id, 63 int render_view_id,
64 int page_request_id, 64 int page_request_id,
65 const std::string& tab_capture_device_id,
66 const GURL& security_origin, 65 const GURL& security_origin,
67 MediaStreamRequestType request_type, 66 MediaStreamRequestType request_type,
68 const std::string& requested_audio_device_id, 67 const std::string& requested_audio_device_id,
69 const std::string& requested_video_device_id, 68 const std::string& requested_video_device_id,
70 MediaStreamType audio_type, 69 MediaStreamType audio_type,
71 MediaStreamType video_type) 70 MediaStreamType video_type)
72 : render_process_id(render_process_id), 71 : render_process_id(render_process_id),
73 render_view_id(render_view_id), 72 render_view_id(render_view_id),
74 page_request_id(page_request_id), 73 page_request_id(page_request_id),
75 tab_capture_device_id(tab_capture_device_id), 74 tab_capture_device_id(),
tommi (sloooow) - chröme 2013/10/28 13:48:23 remove?
perkj_chrome 2013/10/29 08:52:17 Done.
76 security_origin(security_origin), 75 security_origin(security_origin),
77 request_type(request_type), 76 request_type(request_type),
78 requested_audio_device_id(requested_audio_device_id), 77 requested_audio_device_id(requested_audio_device_id),
79 requested_video_device_id(requested_video_device_id), 78 requested_video_device_id(requested_video_device_id),
80 audio_type(audio_type), 79 audio_type(audio_type),
81 video_type(video_type) { 80 video_type(video_type) {
82 } 81 }
83 82
84 MediaStreamRequest::~MediaStreamRequest() {} 83 MediaStreamRequest::~MediaStreamRequest() {}
85 84
86 } // namespace content 85 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698