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

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

Issue 11876007: Connecting webrtc-internals WebUI frontend with the backend (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@main
Patch Set: Fixing jam's comments Created 7 years, 11 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/peer_connection_tracker_host.cc
diff --git a/content/browser/renderer_host/media/peer_connection_tracker_host.cc b/content/browser/renderer_host/media/peer_connection_tracker_host.cc
index d4a8ac6b66e58097840d7be6acab4f58d63a57d5..9452c42dbdc39a6a99799b8a60ffa1ebea5b13b6 100644
--- a/content/browser/renderer_host/media/peer_connection_tracker_host.cc
+++ b/content/browser/renderer_host/media/peer_connection_tracker_host.cc
@@ -4,9 +4,9 @@
#include "content/browser/renderer_host/media/peer_connection_tracker_host.h"
#include "base/process_util.h"
-#include "content/browser/media/webrtc_internals.h"
#include "content/common/media/peer_connection_tracker_messages.h"
#include "content/public/browser/content_browser_client.h"
+#include "content/public/browser/webrtc_internals.h"
namespace content {
@@ -26,18 +26,29 @@ bool PeerConnectionTrackerHost::OnMessageReceived(const IPC::Message& message,
return handled;
}
+void PeerConnectionTrackerHost::OverrideThreadForMessage(
+ const IPC::Message& message, BrowserThread::ID* thread) {
+ if (message.type() == PeerConnectionTrackerHost_AddPeerConnection::ID ||
jam 2013/01/18 22:23:45 since you're doing this for all PeerConnection mes
jiayl 2013/01/18 22:39:51 Done.
+ message.type() == PeerConnectionTrackerHost_RemovePeerConnection::ID)
+ *thread = BrowserThread::UI;
+}
+
PeerConnectionTrackerHost::~PeerConnectionTrackerHost() {
}
void PeerConnectionTrackerHost::OnAddPeerConnection(
const PeerConnectionInfo& info) {
- WebRTCInternals::GetInstance()->AddPeerConnection(
- base::GetProcId(peer_handle()), info);
+ GetContentClient()->browser()->GetWebRTCInternals()->
+ AddPeerConnection(base::GetProcId(peer_handle()),
+ info.lid,
+ info.url,
+ info.servers,
+ info.constraints);
}
void PeerConnectionTrackerHost::OnRemovePeerConnection(int lid) {
- WebRTCInternals::GetInstance()->RemovePeerConnection(
- base::GetProcId(peer_handle()), lid);
+ GetContentClient()->browser()->GetWebRTCInternals()->
+ RemovePeerConnection(base::GetProcId(peer_handle()), lid);
}
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698