HTTP internals: do not open too many sockets when preconnecting

Each pair of (normal request, preconnect request) requires only one
socket. E.g. if there is 1 preconnect request in-flight and 2 normal
requests, we need only 2 sockets in total, and not 3.

Therefore, we need to keep track of whether a request is
preconnecting or a normal one.

Task-number: QTBUG-31594
Change-Id: If92ccc35abadfa6090d64ee92bd466615909c94c
Reviewed-by: Richard J. Moore <rich@kde.org>

1 files changed, 1 insertions, 0 deletions

diff --git a/src/network/access/qhttpnetworkconnectionchannel.cpp b/src/network/access/qhttpnetworkconnectionchannel.cpp
index a756f22937..1c7a61dca6 100644
--- a/src/network/access/qhttpnetworkconnectionchannel.cpp
+++ b/src/network/access/qhttpnetworkconnectionchannel.cpp
@@ -214,6 +214,7 @@ bool QHttpNetworkConnectionChannel::sendRequest()
             state = QHttpNetworkConnectionChannel::IdleState;
             reply->d_func()->state = QHttpNetworkReplyPrivate::AllDoneState;
             allDone();
+            connection->preConnectFinished(); // will only decrease the counter
             reply = 0; // so we can reuse this channel
             return true; // we have a working connection and are done
         }