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

Unified Diff: chrome/browser/media/webrtc_internals.cc

Issue 11876007: Connecting webrtc-internals WebUI frontend with the backend (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@main
Patch Set: 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: chrome/browser/media/webrtc_internals.cc
diff --git a/content/browser/media/webrtc_internals.cc b/chrome/browser/media/webrtc_internals.cc
similarity index 65%
rename from content/browser/media/webrtc_internals.cc
rename to chrome/browser/media/webrtc_internals.cc
index afd9d12641b30244b2c286481b92a70cf8faa964..63d75cb56148cb26c41c8b7d15b4e7b5b5247fae 100644
--- a/content/browser/media/webrtc_internals.cc
+++ b/chrome/browser/media/webrtc_internals.cc
@@ -2,16 +2,15 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "content/browser/media/webrtc_internals.h"
+#include "chrome/browser/media/webrtc_internals.h"
-#include "content/browser/media/webrtc_internals_ui_observer.h"
-#include "content/common/media/peer_connection_tracker_messages.h"
+#include "chrome/browser/media/webrtc_internals_ui_observer.h"
#include "content/public/browser/browser_thread.h"
using base::DictionaryValue;
using base::ProcessId;
-
-namespace content{
+using content::BrowserThread;
+using std::string;
WebRTCInternals::WebRTCInternals() {
}
@@ -24,18 +23,21 @@ WebRTCInternals* WebRTCInternals::GetInstance() {
}
void WebRTCInternals::AddPeerConnection(ProcessId pid,
- const PeerConnectionInfo& info) {
+ int lid,
+ const string& url,
+ const string& servers,
+ const string& constraints) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
if (observers_.size()) {
DictionaryValue* dict = new DictionaryValue();
if (dict != NULL) {
dict->SetInteger("pid", static_cast<int>(pid));
- dict->SetInteger("lid", info.lid);
- dict->SetString("servers", info.servers);
- dict->SetString("constraints", info.constraints);
- dict->SetString("url", info.url);
+ dict->SetInteger("lid", lid);
+ dict->SetString("servers", servers);
+ dict->SetString("constraints", constraints);
+ dict->SetString("url", url);
- SendUpdate("updatePeerConnectionAdded", dict);
+ SendUpdate("addPeerConnection", dict);
peer_connection_data_.Append(dict);
}
}
@@ -44,11 +46,6 @@ void WebRTCInternals::AddPeerConnection(ProcessId pid,
void WebRTCInternals::RemovePeerConnection(ProcessId pid, int lid) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
if (observers_.size()) {
- DictionaryValue dict;
- dict.SetInteger("pid", static_cast<int>(pid));
- dict.SetInteger("lid", lid);
- SendUpdate("updatePeerConnectionRemoved", &dict);
-
for (size_t i = 0; i < peer_connection_data_.GetSize(); ++i) {
DictionaryValue* dict = NULL;
peer_connection_data_.GetDictionary(i, &dict);
@@ -57,8 +54,16 @@ void WebRTCInternals::RemovePeerConnection(ProcessId pid, int lid) {
int this_lid = 0;
dict->GetInteger("pid", &this_pid);
dict->GetInteger("lid", &this_lid);
- if (this_pid == static_cast<int>(pid) && this_lid == lid)
+ if (this_pid == static_cast<int>(pid) && this_lid == lid) {
peer_connection_data_.Remove(i, NULL);
+
+ DictionaryValue dict;
+ dict.SetInteger("pid", static_cast<int>(pid));
+ dict.SetInteger("lid", lid);
+ SendUpdate("removePeerConnection", &dict);
+
+ break;
+ }
}
}
}
@@ -73,12 +78,10 @@ void WebRTCInternals::RemoveObserver(WebRTCInternalsUIObserver *observer) {
observers_.RemoveObserver(observer);
}
-void WebRTCInternals::SendUpdate(const std::string& command, Value* value) {
+void WebRTCInternals::SendUpdate(const string& command, Value* value) {
DCHECK(observers_.size());
FOR_EACH_OBSERVER(WebRTCInternalsUIObserver,
observers_,
OnUpdate(command, value));
}
-
-} // namespace content

Powered by Google App Engine
This is Rietveld 408576698