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

Side by Side Diff: content/browser/renderer_host/media/media_stream_dispatcher_host.cc

Issue 143003031: Allow retrieval of media device ID salt even after ResourceContext has gone away. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add TODO about proper fix. Created 6 years, 10 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 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/browser/renderer_host/media/media_stream_dispatcher_host.h" 5 #include "content/browser/renderer_host/media/media_stream_dispatcher_host.h"
6 6
7 #include "content/browser/browser_main_loop.h" 7 #include "content/browser/browser_main_loop.h"
8 #include "content/browser/renderer_host/media/web_contents_capture_util.h" 8 #include "content/browser/renderer_host/media/web_contents_capture_util.h"
9 #include "content/common/media/media_stream_messages.h" 9 #include "content/common/media/media_stream_messages.h"
10 #include "content/common/media/media_stream_options.h" 10 #include "content/common/media/media_stream_options.h"
(...skipping 113 matching lines...) Expand 10 before | Expand all | Expand 10 after
124 const GURL& security_origin) { 124 const GURL& security_origin) {
125 DVLOG(1) << "MediaStreamDispatcherHost::OnGenerateStream(" 125 DVLOG(1) << "MediaStreamDispatcherHost::OnGenerateStream("
126 << render_view_id << ", " 126 << render_view_id << ", "
127 << page_request_id << ", [" 127 << page_request_id << ", ["
128 << " audio:" << components.audio_requested 128 << " audio:" << components.audio_requested
129 << " video:" << components.video_requested 129 << " video:" << components.video_requested
130 << " ], " 130 << " ], "
131 << security_origin.spec() << ")"; 131 << security_origin.spec() << ")";
132 132
133 media_stream_manager_->GenerateStream( 133 media_stream_manager_->GenerateStream(
134 this, render_process_id_, render_view_id, resource_context_, 134 this, render_process_id_, render_view_id,
135 resource_context_->GetMediaDeviceIDSalt(),
135 page_request_id, 136 page_request_id,
136 components, security_origin); 137 components, security_origin);
137 } 138 }
138 139
139 void MediaStreamDispatcherHost::OnCancelGenerateStream(int render_view_id, 140 void MediaStreamDispatcherHost::OnCancelGenerateStream(int render_view_id,
140 int page_request_id) { 141 int page_request_id) {
141 DVLOG(1) << "MediaStreamDispatcherHost::OnCancelGenerateStream(" 142 DVLOG(1) << "MediaStreamDispatcherHost::OnCancelGenerateStream("
142 << render_view_id << ", " 143 << render_view_id << ", "
143 << page_request_id << ")"; 144 << page_request_id << ")";
144 media_stream_manager_->CancelRequest(render_process_id_, render_view_id, 145 media_stream_manager_->CancelRequest(render_process_id_, render_view_id,
(...skipping 15 matching lines...) Expand all
160 int page_request_id, 161 int page_request_id,
161 MediaStreamType type, 162 MediaStreamType type,
162 const GURL& security_origin) { 163 const GURL& security_origin) {
163 DVLOG(1) << "MediaStreamDispatcherHost::OnEnumerateDevices(" 164 DVLOG(1) << "MediaStreamDispatcherHost::OnEnumerateDevices("
164 << render_view_id << ", " 165 << render_view_id << ", "
165 << page_request_id << ", " 166 << page_request_id << ", "
166 << type << ", " 167 << type << ", "
167 << security_origin.spec() << ")"; 168 << security_origin.spec() << ")";
168 169
169 media_stream_manager_->EnumerateDevices( 170 media_stream_manager_->EnumerateDevices(
170 this, render_process_id_, render_view_id, resource_context_, 171 this, render_process_id_, render_view_id,
172 resource_context_->GetMediaDeviceIDSalt(),
171 page_request_id, type, security_origin); 173 page_request_id, type, security_origin);
172 } 174 }
173 175
174 void MediaStreamDispatcherHost::OnCancelEnumerateDevices( 176 void MediaStreamDispatcherHost::OnCancelEnumerateDevices(
175 int render_view_id, 177 int render_view_id,
176 int page_request_id) { 178 int page_request_id) {
177 DVLOG(1) << "MediaStreamDispatcherHost::OnCancelEnumerateDevices(" 179 DVLOG(1) << "MediaStreamDispatcherHost::OnCancelEnumerateDevices("
178 << render_view_id << ", " 180 << render_view_id << ", "
179 << page_request_id << ")"; 181 << page_request_id << ")";
180 media_stream_manager_->CancelRequest(render_process_id_, render_view_id, 182 media_stream_manager_->CancelRequest(render_process_id_, render_view_id,
181 page_request_id); 183 page_request_id);
182 } 184 }
183 185
184 void MediaStreamDispatcherHost::OnOpenDevice( 186 void MediaStreamDispatcherHost::OnOpenDevice(
185 int render_view_id, 187 int render_view_id,
186 int page_request_id, 188 int page_request_id,
187 const std::string& device_id, 189 const std::string& device_id,
188 MediaStreamType type, 190 MediaStreamType type,
189 const GURL& security_origin) { 191 const GURL& security_origin) {
190 DVLOG(1) << "MediaStreamDispatcherHost::OnOpenDevice(" 192 DVLOG(1) << "MediaStreamDispatcherHost::OnOpenDevice("
191 << render_view_id << ", " 193 << render_view_id << ", "
192 << page_request_id << ", device_id: " 194 << page_request_id << ", device_id: "
193 << device_id.c_str() << ", type: " 195 << device_id.c_str() << ", type: "
194 << type << ", " 196 << type << ", "
195 << security_origin.spec() << ")"; 197 << security_origin.spec() << ")";
196 198
197 media_stream_manager_->OpenDevice( 199 media_stream_manager_->OpenDevice(
198 this, render_process_id_, render_view_id, resource_context_, 200 this, render_process_id_, render_view_id,
201 resource_context_->GetMediaDeviceIDSalt(),
199 page_request_id, device_id, type, security_origin); 202 page_request_id, device_id, type, security_origin);
200 203
201 } 204 }
202 205
203 void MediaStreamDispatcherHost::OnCloseDevice( 206 void MediaStreamDispatcherHost::OnCloseDevice(
204 int render_view_id, 207 int render_view_id,
205 const std::string& label) { 208 const std::string& label) {
206 DVLOG(1) << "MediaStreamDispatcherHost::OnCloseDevice(" 209 DVLOG(1) << "MediaStreamDispatcherHost::OnCloseDevice("
207 << render_view_id << ", " 210 << render_view_id << ", "
208 << label << ")"; 211 << label << ")";
209 212
210 media_stream_manager_->CancelRequest(label); 213 media_stream_manager_->CancelRequest(label);
211 } 214 }
212 215
213 } // namespace content 216 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698