Small fixes for WevRTC protocol in host

remoting/protocol/webrtc_transport.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "remoting/protocol/webrtc_transport.h"
 
 #include <string>
 #include <utility>
 #include <vector>
 
@@ skipping 318 matching lines @@
 #if !defined(NDEBUG)
       ignore_error = base::CommandLine::ForCurrentProcess()->HasSwitch(
           kDisableAuthenticationSwitchName);
 #endif
       if (!ignore_error) {
         Close(AUTHENTICATION_FAILED);
         return true;
       }
     }
 
-    // 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
     // handling spiky traffic patterns well. This won't be necessary with a
     // better bandwidth estimator.
     // TODO(isheriff): The need for this should go away once we have a proper
     // API to provide max bitrate for the case of handing over encoded
     // frames to webrtc.
     std::string vp8line = "a=rtpmap:100 VP8/90000\n";
     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;
     std::unique_ptr<webrtc::SessionDescriptionInterface> session_description(
         webrtc::CreateSessionDescription(type, sdp, &error));
     if (!session_description) {
       LOG(ERROR) << "Failed to parse the session description: "
                  << error.description << " line: " << error.line;
       return false;
     }
@@ skipping 298 matching lines @@
   // the stack and so it must be destroyed later.
   base::ThreadTaskRunnerHandle::Get()->DeleteSoon(
       FROM_HERE, peer_connection_wrapper_.release());
 
   if (error != OK)
     event_handler_->OnWebrtcTransportError(error);
 }
 
 }  // namespace protocol
 }  // namespace remoting