Connecting webrtc-internals WebUI frontend with the backend

content/browser/renderer_host/media/peer_connection_tracker_host.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 #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 {
 
 PeerConnectionTrackerHost::PeerConnectionTrackerHost() {
 }
 
 bool PeerConnectionTrackerHost::OnMessageReceived(const IPC::Message& message,
                                                   bool* message_was_ok) {
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP_EX(PeerConnectionTrackerHost, message, *message_was_ok)
     IPC_MESSAGE_HANDLER(PeerConnectionTrackerHost_AddPeerConnection,
                         OnAddPeerConnection)
     IPC_MESSAGE_HANDLER(PeerConnectionTrackerHost_RemovePeerConnection,
                         OnRemovePeerConnection)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP_EX()
   return handled;
 }
 
 PeerConnectionTrackerHost::~PeerConnectionTrackerHost() {
 }
 
 void PeerConnectionTrackerHost::OnAddPeerConnection(
     const PeerConnectionInfo& info) {
-  WebRTCInternals::GetInstance()->AddPeerConnection(
-      base::GetProcId(peer_handle()), info);
+  BrowserThread::PostTask(
+      BrowserThread::UI, FROM_HERE,
+      base::Bind(&PeerConnectionTrackerHost::OnAddPeerConnectionOnUIThread,

[jam, 2013/01/18 18:27:33]

why are you doing the dispatching manually instead of using
OverrideThreadForMessage?

[jiayl, 2013/01/18 19:20:00]

I tried using OverrideThreadForMessage and it didn't work and I thought it needs
to be override on the sender side. But then I looked more and found I checked
the message type wrongly. :P
Now It works.

+                 this, info));
+}
+
+void PeerConnectionTrackerHost::OnAddPeerConnectionOnUIThread(
+    const PeerConnectionInfo& 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);
+  BrowserThread::PostTask(
+      BrowserThread::UI, FROM_HERE,
+      base::Bind(&PeerConnectionTrackerHost::OnRemovePeerConnectionOnUIThread,
+                 this, lid));
+}
+
+void PeerConnectionTrackerHost::OnRemovePeerConnectionOnUIThread(int lid) {
+  GetContentClient()->browser()->GetWebRTCInternals()->
+      RemovePeerConnection(base::GetProcId(peer_handle()), lid);
 }
 
 }  // namespace content