Commit 42315506 authored by hclam@chromium.org's avatar hclam@chromium.org

IpcSocketFactory pending packet limit is too small

Refactoring is underway to notify client of P2P socket that network is
congested. In the mean time pending packet limit is too small and causes
huge amount of packet loss, this makes WebRTC unusable.

This is a temporary fix to increase this pre-defined value to 32 pending
packets which accomodates 720p streams up to 5Mb/s.

BUG=228956, 229737

Review URL: https://codereview.chromium.org/13992004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193389 0039d316-1c4b-4281-b951-d872f2087c98
parent 296969e5
...@@ -16,8 +16,8 @@ namespace content { ...@@ -16,8 +16,8 @@ namespace content {
namespace { namespace {
// TODO(sergeyu): Try adjusting these parameters to achieve optimal performance. // TODO(hclam): This shouldn't be a pre-defined value. Bug: crbug.com/181321.
const int kMaxPendingPackets = 8; const int kMaxPendingPackets = 32;
const int kWritableSignalThreshold = 0; const int kWritableSignalThreshold = 0;
// IpcPacketSocket implements talk_base::AsyncPacketSocket interface // IpcPacketSocket implements talk_base::AsyncPacketSocket interface
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment