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

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

Issue 1871533002: Change WebRTC log callback registration in browser process. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Code review (olka) Created 4 years, 8 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: content/browser/renderer_host/media/media_stream_manager.cc
diff --git a/content/browser/renderer_host/media/media_stream_manager.cc b/content/browser/renderer_host/media/media_stream_manager.cc
index d2e6068524c676be2f3938ff1d6f81cd71384ce0..faa3294fe2d2d6ff36211df1083575586b5d658c 100644
--- a/content/browser/renderer_host/media/media_stream_manager.cc
+++ b/content/browser/renderer_host/media/media_stream_manager.cc
@@ -1810,17 +1810,9 @@ void MediaStreamManager::UnregisterNativeLogCallback(int renderer_host_id) {
}
void MediaStreamManager::AddLogMessageOnIOThread(const std::string& message) {
- // Get render process ids on the IO thread.
DCHECK_CURRENTLY_ON(BrowserThread::IO);
-
- for (const LabeledDeviceRequest& labeled_request : requests_) {
- DeviceRequest* const request = labeled_request.second;
- if (request->request_type == MEDIA_GENERATE_STREAM) {
- const auto& found = log_callbacks_.find(request->requesting_process_id);
- if (found != log_callbacks_.end())
- found->second.Run(message);
- }
- }
+ for (const auto& callback : log_callbacks_)
+ callback.second.Run(message);
}
void MediaStreamManager::HandleAccessRequestResponse(
« no previous file with comments | « chrome/browser/media/webrtc_logging_handler_host.cc ('k') | content/browser/renderer_host/render_process_host_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698