Use QHttp2Connection in qhttp2protocolhandler

After introducing QHttp2Connection earlier we ended up having some duplication. For 6.9 let's be brave and get rid of that :) This patch drops all the code that can now be handled by QHttp2Connection, and uses that directly. Fixes: QTBUG-129211 Change-Id: Ie82b532452010b06f33e29457da8fb698898a29f Reviewed-by: Mate Barany <[email protected]>
qt · Nov 21, 2024 · 3307d59 · 3307d59
1 parent ba5a2fb
commit 3307d59
Show file tree

Hide file tree

Showing 5 changed files with 342 additions and 1,283 deletions.
diff --git a/src/network/access/qhttp2connection_p.h b/src/network/access/qhttp2connection_p.h
@@ -244,7 +244,7 @@ class Q_NETWORK_EXPORT QHttp2Connection : public QObject
         return nullptr;
     }
 
-    void close() { sendGOAWAY(Http2::HTTP2_NO_ERROR); }
+    void close(Http2::Http2Error error = Http2::HTTP2_NO_ERROR) { sendGOAWAY(error); }
 
     bool isGoingAway() const noexcept { return m_goingAway; }