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

Unified Diff: content/browser/renderer_host/p2p/socket_host.h

Issue 13584008: Send notification about outgoing p2p packets from browser to renderer. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 8 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/browser/renderer_host/p2p/socket_host.h
diff --git a/content/browser/renderer_host/p2p/socket_host.h b/content/browser/renderer_host/p2p/socket_host.h
index 0abb74e4f6d38f8acb1431bd9564bba36e9bf0f8..9fe985dc717d7334f2def746f01179cfa0d1865a 100644
--- a/content/browser/renderer_host/p2p/socket_host.h
+++ b/content/browser/renderer_host/p2p/socket_host.h
@@ -62,15 +62,6 @@ class CONTENT_EXPORT P2PSocketHost {
STATE_ERROR,
};
- // Maximum size of send buffers. Must be big enough to fit data for
- // one data burst. Send buffers size needs to be limited to prevent
- // from consuming too much memory with misbehaving renderer process.
- //
- // TODO(sergeyu): Consider implementing congestion notifications to
- // minimize buffering. This will require some fixes in libjingle,
- // see crbug.com/91495 .
- static const int kMaxSendBufferSize = 256 * 1024;
-
P2PSocketHost(IPC::Sender* message_sender, int id);
// Verifies that the packet |data| has a valid STUN header. In case
« no previous file with comments | « no previous file | content/browser/renderer_host/p2p/socket_host_tcp.h » ('j') | content/renderer/p2p/ipc_socket_factory.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698