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

Unified Diff: remoting/protocol/webrtc_transport.cc

Issue 2398293002: Small fixes for WevRTC protocol in host (Closed)
Patch Set: Created 4 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
« no previous file with comments | « remoting/codec/webrtc_video_encoder_vpx.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/protocol/webrtc_transport.cc
diff --git a/remoting/protocol/webrtc_transport.cc b/remoting/protocol/webrtc_transport.cc
index b3f759ac652728238ec39569e5ad9ce69f814dce..afa408592e47af22a31e257e45f5ac5c0018ad31 100644
--- a/remoting/protocol/webrtc_transport.cc
+++ b/remoting/protocol/webrtc_transport.cc
@@ -336,9 +336,9 @@ bool WebrtcTransport::ProcessTransportInfo(XmlElement* transport_info) {
}
}
- // Set bitrate range to 1-20 Mbps.
+ // Set bitrate range to 1-100 Mbps.
Irfan 2016/10/07 14:59:12 Does this provide any actual gains ?
// - Setting min bitrate here enables padding.
- // - The default max bitrate is 600 kbps. Setting it to 20 Mbps allows to
+ // - The default max bitrate is 600 kbps. Setting it to 100 Mbps allows to
// use higher bandwidth when it's available.
//
// TODO(sergeyu): Padding needs to be enabled to workaround BW estimator not
@@ -351,7 +351,7 @@ bool WebrtcTransport::ProcessTransportInfo(XmlElement* transport_info) {
std::string vp8line_with_bitrate =
vp8line +
"a=fmtp:100 x-google-min-bitrate=1000\n"
- "a=fmtp:100 x-google-max-bitrate=20000\n";
+ "a=fmtp:100 x-google-max-bitrate=100000\n";
base::ReplaceSubstringsAfterOffset(&sdp, 0, vp8line, vp8line_with_bitrate);
webrtc::SdpParseError error;
« no previous file with comments | « remoting/codec/webrtc_video_encoder_vpx.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698