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

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

Issue 88283002: Reland review 34393006: Refactor MediaStreamManager to not output real device id. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Changes since the revert. Created 7 years 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 119 matching lines...) Expand 10 before | Expand all | Expand 10 after
130 << render_view_id << ", " 130 << render_view_id << ", "
131 << page_request_id << ", [" 131 << page_request_id << ", ["
132 << " audio:" << components.audio_type 132 << " audio:" << components.audio_type
133 << " video:" << components.video_type 133 << " video:" << components.video_type
134 << " ], " 134 << " ], "
135 << security_origin.spec() << ")"; 135 << security_origin.spec() << ")";
136 136
137 const std::string& label = media_stream_manager_->GenerateStream( 137 const std::string& label = media_stream_manager_->GenerateStream(
138 this, render_process_id_, render_view_id, page_request_id, 138 this, render_process_id_, render_view_id, page_request_id,
139 components, security_origin); 139 components, security_origin);
140 if (label.empty()) { 140 CHECK(!label.empty());
141 Send(new MediaStreamMsg_StreamGenerationFailed(render_view_id, 141 StoreRequest(render_view_id, page_request_id, label);
142 page_request_id));
143 } else {
144 StoreRequest(render_view_id, page_request_id, label);
145 }
146 } 142 }
147 143
148 void MediaStreamDispatcherHost::OnCancelGenerateStream(int render_view_id, 144 void MediaStreamDispatcherHost::OnCancelGenerateStream(int render_view_id,
149 int page_request_id) { 145 int page_request_id) {
150 DVLOG(1) << "MediaStreamDispatcherHost::OnCancelGenerateStream(" 146 DVLOG(1) << "MediaStreamDispatcherHost::OnCancelGenerateStream("
151 << render_view_id << ", " 147 << render_view_id << ", "
152 << page_request_id << ")"; 148 << page_request_id << ")";
153 149
154 for (StreamMap::iterator it = streams_.begin(); it != streams_.end(); ++it) { 150 for (StreamMap::iterator it = streams_.begin(); it != streams_.end(); ++it) {
155 if (it->second.render_view_id == render_view_id && 151 if (it->second.render_view_id == render_view_id &&
(...skipping 92 matching lines...) Expand 10 before | Expand all | Expand 10 after
248 MediaStreamDispatcherHost::StreamRequest 244 MediaStreamDispatcherHost::StreamRequest
249 MediaStreamDispatcherHost::PopRequest(const std::string& label) { 245 MediaStreamDispatcherHost::PopRequest(const std::string& label) {
250 StreamMap::iterator it = streams_.find(label); 246 StreamMap::iterator it = streams_.find(label);
251 CHECK(it != streams_.end()); 247 CHECK(it != streams_.end());
252 StreamRequest request = it->second; 248 StreamRequest request = it->second;
253 streams_.erase(it); 249 streams_.erase(it);
254 return request; 250 return request;
255 } 251 }
256 252
257 } // namespace content 253 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698