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

Unified Diff: content/renderer/media/media_stream_dependency_factory.cc

Issue 23691066: Hook up WebRTC logging extension API to the underlying functionality. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 7 years, 2 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/renderer/media/media_stream_dependency_factory.cc
diff --git a/content/renderer/media/media_stream_dependency_factory.cc b/content/renderer/media/media_stream_dependency_factory.cc
index 5db6262d114d75062fccb137f6a720e59c3ca2e4..1a7fd531f868fe6ba2c735e7e6d1b77743e65275 100644
--- a/content/renderer/media/media_stream_dependency_factory.cc
+++ b/content/renderer/media/media_stream_dependency_factory.cc
@@ -21,7 +21,6 @@
#include "content/renderer/media/webaudio_capturer_source.h"
#include "content/renderer/media/webrtc_audio_device_impl.h"
#include "content/renderer/media/webrtc_local_audio_track.h"
-#include "content/renderer/media/webrtc_logging_initializer.h"
#include "content/renderer/media/webrtc_uma_histograms.h"
#include "content/renderer/p2p/ipc_network_manager.h"
#include "content/renderer/p2p/ipc_socket_factory.h"
@@ -50,10 +49,6 @@
namespace content {
-// The constraint key for the PeerConnection constructor for enabling diagnostic
-// WebRTC logging. It's a Google specific key, hence the "goog" prefix.
-const char kWebRtcLoggingConstraint[] = "googLog";
-
// Constant constraint keys which enables default audio constraints on
// mediastreams with audio.
struct {
@@ -572,19 +567,6 @@ MediaStreamDependencyFactory::CreatePeerConnection(
CHECK(web_frame);
CHECK(observer);
- webrtc::MediaConstraintsInterface::Constraints optional_constraints =
- constraints->GetOptional();
- std::string constraint_value;
- if (optional_constraints.FindFirst(kWebRtcLoggingConstraint,
- &constraint_value)) {
- std::string url = web_frame->document().url().spec();
- RenderThreadImpl::current()->GetIOMessageLoopProxy()->PostTask(
- FROM_HERE, base::Bind(
- &InitWebRtcLogging,
- constraint_value,
- url));
- }
-
scoped_refptr<P2PPortAllocatorFactory> pa_factory =
new talk_base::RefCountedObject<P2PPortAllocatorFactory>(
p2p_socket_dispatcher_.get(),
« no previous file with comments | « content/renderer/media/media_stream_dependency_factory.h ('k') | content/renderer/media/webrtc_logging_initializer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698