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

Unified Diff: webrtc/api/peerconnectionfactory.cc

Issue 1818033002: Embed a cricket::MediaConfig in RTCConfiguration. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 9 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: webrtc/api/peerconnectionfactory.cc
diff --git a/webrtc/api/peerconnectionfactory.cc b/webrtc/api/peerconnectionfactory.cc
index 22e450ca4ec29ed5bd1177d73d7020380928c007..b5694791616ba9440457068d2e8e480d55c40cd1 100644
--- a/webrtc/api/peerconnectionfactory.cc
+++ b/webrtc/api/peerconnectionfactory.cc
@@ -289,24 +289,8 @@ PeerConnectionFactory::CreatePeerConnection(
rtc::scoped_refptr<PeerConnection> pc(
new rtc::RefCountedObject<PeerConnection>(this));
- // We rely on default values when constraints aren't found.
- cricket::MediaConfig media_config;
- media_config.video.disable_prerenderer_smoothing =
- configuration.disable_prerenderer_smoothing;
- if (configuration.enable_dscp) {
- media_config.enable_dscp = *(configuration.enable_dscp);
- }
- if (configuration.cpu_overuse_detection) {
- media_config.video.enable_cpu_overuse_detection =
- *(configuration.cpu_overuse_detection);
- }
- if (configuration.suspend_below_min_bitrate) {
- media_config.video.suspend_below_min_bitrate =
- *(configuration.suspend_below_min_bitrate);
- }
-
- if (!pc->Initialize(media_config, configuration, std::move(allocator),
+ if (!pc->Initialize(configuration, std::move(allocator),
std::move(dtls_identity_store), observer)) {
hta-webrtc 2016/03/21 14:38:21 Nit: Please run git cl format before submitting.
nisse-webrtc 2016/03/22 08:33:20 Done. But no changes at this location.
return nullptr;
}

Powered by Google App Engine
This is Rietveld 408576698