[webkit-changes] [WebKit/WebKit] 5d1eb8: Stream IPC message send tests do not test with sma...

[Kimmo Kinnunen]

  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 5d1eb8359a73e05a30dff4783ed765418e51e938
      https://github.com/WebKit/WebKit/commit/5d1eb8359a73e05a30dff4783ed765418e51e938
  Author: Kimmo Kinnunen <kkinnunen at apple.com>
  Date:   2023-01-16 (Mon, 16 Jan 2023)

  Changed paths:
    M Source/WebKit/Platform/IPC/StreamServerConnection.cpp
    M Source/WebKit/Platform/IPC/StreamServerConnection.h
    M Tools/TestWebKitAPI/Tests/IPC/StreamConnectionTests.cpp

  Log Message:
  -----------
  Stream IPC message send tests do not test with small connection buffer sizes
https://bugs.webkit.org/show_bug.cgi?id=250669
rdar://104294217

Reviewed by Antti Koivisto.

Separate the message send tests to a new test fixture and
parametrize that with connection buffer size. Instantiate
with 128, 256, 512, 16k sizes. With the smallest buffers,
the buffer will wrap around so the tests exercise that
part of the implementation.

Fix a bug that affected the tests only, where unopened
StreamServerConnection would assert on invalidate().
Add a test for the case.

* Source/WebKit/Platform/IPC/StreamServerConnection.cpp:
(IPC::StreamServerConnection::open):
(IPC::StreamServerConnection::invalidate):
* Source/WebKit/Platform/IPC/StreamServerConnection.h:
* Tools/TestWebKitAPI/Tests/IPC/StreamConnectionTests.cpp:
(TestWebKitAPI::StreamConnectionTestBase::setupBase):
(TestWebKitAPI::StreamConnectionTestBase::teardownBase):
(TestWebKitAPI::StreamConnectionTestBase::localReferenceBarrier):
(TestWebKitAPI::TEST_F):
(TestWebKitAPI::StreamMessageTest::bufferSizeLog2 const):
(TestWebKitAPI::StreamMessageTest::defaultDestinationID):
(TestWebKitAPI::TEST_P):
(TestWebKitAPI::StreamConnectionTest::localReferenceBarrier): Deleted.
(TestWebKitAPI::StreamConnectionTest::serverQueue): Deleted.

Canonical link: https://commits.webkit.org/258950@main