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

Unified Diff: chrome/renderer/chrome_content_renderer_client.cc

Issue 15741003: Moving WebRTC logging related files from content to chrome. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Public content API for setting the log delegate. Created 7 years, 7 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/renderer/chrome_content_renderer_client.cc
diff --git a/chrome/renderer/chrome_content_renderer_client.cc b/chrome/renderer/chrome_content_renderer_client.cc
index 2ace978b9fc6631e39009a547d62dee2465ebe6a..cbb4135dc482bd1014db64a3262bd674b69f5049 100644
--- a/chrome/renderer/chrome_content_renderer_client.cc
+++ b/chrome/renderer/chrome_content_renderer_client.cc
@@ -103,6 +103,11 @@
#include "chrome/renderer/automation/automation_renderer_helper.h"
#endif
+#if defined(ENABLE_WEBRTC)
+#include "chrome/renderer/media/webrtc_logging_handler_impl.h"
+#include "chrome/renderer/media/webrtc_logging_message_filter.h"
+#endif
+
using autofill::AutofillAgent;
using autofill::PasswordAutofillAgent;
using autofill::PasswordGenerationManager;
@@ -224,6 +229,10 @@ void ChromeContentRendererClient::RenderThreadStarted() {
phishing_classifier_.reset(safe_browsing::PhishingClassifierFilter::Create());
#endif
prerender_dispatcher_.reset(new prerender::PrerenderDispatcher());
+#if defined(ENABLE_WEBRTC)
+ webrtc_logging_message_filter_ = new WebRtcLoggingMessageFilter(
+ content::RenderThread::Get()->GetIOMessageLoopProxy());
+#endif
RenderThread* thread = RenderThread::Get();
@@ -236,6 +245,10 @@ void ChromeContentRendererClient::RenderThreadStarted() {
thread->AddObserver(visited_link_slave_.get());
thread->AddObserver(prerender_dispatcher_.get());
+#if defined(ENABLE_WEBRTC)
+ thread->AddFilter(webrtc_logging_message_filter_.get());
+#endif
+
thread->RegisterExtension(extensions_v8::ExternalExtension::Get());
thread->RegisterExtension(extensions_v8::LoadTimesExtension::Get());
@@ -1185,4 +1198,26 @@ bool ChromeContentRendererClient::AllowBrowserPlugin(
tag_name.equals(WebString::fromUTF8(kAdViewTagName));
}
+#if defined(ENABLE_WEBRTC)
+void ChromeContentRendererClient::InitWebRtcLogging(
+ const std::string& app_session_id) {
+ RenderThread::Get()->GetIOMessageLoopProxy()->PostTask(
+ FROM_HERE, base::Bind(
+ &ChromeContentRendererClient::DoInitWebRtcLogging,
+ base::Unretained(this),
+ webrtc_logging_message_filter_,
+ app_session_id));
+}
+
+void ChromeContentRendererClient::DoInitWebRtcLogging(
+ WebRtcLoggingMessageFilter* filter,
+ const std::string& app_session_id) {
+ WebRtcLoggingHandlerImpl* handler =
+ new WebRtcLoggingHandlerImpl(filter->io_message_loop());
+
+ // TODO(grunell): Give app session id as parameter.
+ filter->InitLogging(handler);
+}
+#endif
+
} // namespace chrome

Powered by Google App Engine
This is Rietveld 408576698