[PR #9710/b40265e backport][3.11] Remove redundant protocol.should_close check in Connection._release (#9712)

Author: bdraco

aiohttp/connector.py

@@ -186,9 +186,7 @@ def release(self) -> None:
         self._notify_release()
 
         if self._protocol is not None:
-            self._connector._release(
-                self._key, self._protocol, should_close=self._protocol.should_close
-            )
+            self._connector._release(self._key, self._protocol)
             self._protocol = None
 
     @property

tests/test_client_connection.py

@@ -112,7 +112,7 @@ def test_release(connector, key, protocol, loop) -> None:
     conn.release()
     assert not protocol.transport.close.called
     assert conn._protocol is None
-    connector._release.assert_called_with(key, protocol, should_close=False)
+    connector._release.assert_called_with(key, protocol)
     assert conn.closed
 
 
@@ -123,7 +123,7 @@ def test_release_proto_should_close(connector, key, protocol, loop) -> None:
     conn.release()
     assert not protocol.transport.close.called
     assert conn._protocol is None
-    connector._release.assert_called_with(key, protocol, should_close=True)
+    connector._release.assert_called_with(key, protocol)
     assert conn.closed