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

Unified Diff: net/data/websocket/multiple-connections.html

Issue 436493002: Handle multiple simultanous wss: connections. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Use a pointer instead of a reference to scoped_ptr. Created 6 years, 5 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: net/data/websocket/multiple-connections.html
diff --git a/net/data/websocket/multiple-connections.html b/net/data/websocket/multiple-connections.html
new file mode 100644
index 0000000000000000000000000000000000000000..ff5c8576deccbe573ba69096bcd5d03f020ef86f
--- /dev/null
+++ b/net/data/websocket/multiple-connections.html
@@ -0,0 +1,30 @@
+<title>WebSocket is not subject to HTTP(S) connection limit</title>
+<script>
+var protocol = location.protocol.replace('http', 'ws');
+var url = protocol + '//' + location.host + '/echo-with-no-extension';
+
+// TODO(ricea): The limit for ws: is 255, but wss: gets a limit of 30
+// (per-host:port, not ip:port!) because it still uses the HTTP pool code. This
+// should be fixed at some point. When it is, change this number back to 50 (or
+// even larger, if it works).
+var socketCount = 30;
+var connected = 0;
+
+for (i = 0; i < socketCount; ++i) {
+ var ws = new WebSocket(url);
+ ws.onopen = function() {
+ ++connected;
+ if (connected == socketCount) {
+ document.title = "PASS";
+ }
+ };
+ ws.onclose = function() {
+ document.title = "FAIL";
+ };
+}
+
+setTimeout(function() {
+ console.log("Got stuck after " + connected + " socket(s) connected");
+ document.title = "FAIL";
+}, 5000);
+</script>

Powered by Google App Engine
This is Rietveld 408576698