Commit Graph

1749 Commits

Author SHA1 Message Date
Natalia Csoregi
a1e84a49ec Backed out changeset a1428617e87a (bug 1885948) for causing mbu failures. CLOSED TREE 2024-03-19 07:50:34 +02:00
Barret Rennie
541f3fc622 Bug 1885948 - Pass IPDL sources to ipdl.py in a text file r=nika,ahochheiden
If the path to your source checkout on Windows is sufficiently long enough, you
can hit the Windows maximum commandline argument limit (32k) specifying IPDL
files to ipdl.py during build. Writing the list of files to a file and passing
that to ipdl.py gets around this error completely.

Differential Revision: https://phabricator.services.mozilla.com/D204949
2024-03-18 20:03:13 +00:00
Iulian Moraru
39646463bc Backed out changeset 8a9b63318958 (bug 1729044) for causing linux build bustages. CLOSED TREE 2024-03-19 04:58:49 +02:00
Alex
28e7b97eeb Bug 1729044 - Remove ipdl support for intr r=ipc-reviewers,nika
This removes intr support and updates some stale docs as well.

It's not immediately clear what code in MessageChannel can be removed,
though I expect some things could be simplified (there's just not much
alluding to intr/rpc/urgent).

Differential Revision: https://phabricator.services.mozilla.com/D204813
2024-03-18 17:23:26 +00:00
Alex
6c8a0020a5 Bug 1236717 - Remove unused/unmigrated IPDL tests r=ipc-reviewers,nika
The tests we want to keep have been migrated to gtests and are run by
default.

Differential Revision: https://phabricator.services.mozilla.com/D204818
2024-03-18 17:12:25 +00:00
Alex
b9343c37c9 Bug 1885018 - Remove unnecessary generation code in the IPDL compiler r=ipc-reviewers,mccr8
Differential Revision: https://phabricator.services.mozilla.com/D204657
2024-03-15 15:10:33 +00:00
Alex
19fb5724a7 Bug 1885018 - Disallow sync and return values in destructors r=ipc-reviewers,mccr8
Differential Revision: https://phabricator.services.mozilla.com/D204656
2024-03-15 15:10:32 +00:00
Edgar Chen
fdaf5ca11a Bug 1858627 - Update DataTransfer to utilize GetDataSnapshotSync() for retrieving clipboard formats; r=nika,ipc-reviewers
The actual data retrieval continues to use the original method.

Differential Revision: https://phabricator.services.mozilla.com/D202351
2024-03-13 20:44:06 +00:00
Edgar Chen
239696f93b Bug 1858627 - Add nsIClipboard API to return nsIAsyncSetClipboardData synchrously; r=ipc-reviewers,nika
This is a sync version of `nsIClipboard.asyncGetData`, which can be used for
synchronous clipboard APIs, e.g. DataTransfer, to support the clipboard seqence
number concept, see bug 1879401.

Differential Revision: https://phabricator.services.mozilla.com/D201364
2024-03-13 20:44:06 +00:00
Alex Franchuk
1d4418517f Bug 1751995 - Remove TestUrgentHangs r=ipc-reviewers,nika
This seemed like a lot of effort to port with little gain, since it's testing sync/cpow message
things.

Differential Revision: https://phabricator.services.mozilla.com/D202265
2024-03-13 13:10:25 +00:00
Alex Franchuk
4733f46b7c Bug 1751995 - Migrate TestUrgency r=ipc-reviewers,nika
Differential Revision: https://phabricator.services.mozilla.com/D202264
2024-03-13 13:10:25 +00:00
Alex Franchuk
d0bb4eb142 Bug 1751995 - Migrate TestUniquePtrIPC r=ipc-reviewers,nika
Some of the assertions were changed because the message arguments are passed by const reference.
`MoveOnly` is applied to specific types, and UniquePtr is a built-in so we can't adjust that.

Differential Revision: https://phabricator.services.mozilla.com/D202263
2024-03-13 13:10:24 +00:00
Alex Franchuk
dd17f3a6e5 Bug 1751995 - Remove TestSyncWakeup r=ipc-reviewers,nika
This relies on interrupt messages.

Differential Revision: https://phabricator.services.mozilla.com/D202262
2024-03-13 13:10:24 +00:00
Alex Franchuk
fcc88c75c1 Bug 1751995 - Remove TestSyncHang r=ipc-reviewers,nika
This relies on launching additional child processes.

Differential Revision: https://phabricator.services.mozilla.com/D202261
2024-03-13 13:10:23 +00:00
Alex Franchuk
a7a6fd3230 Bug 1751995 - Migrate TestSyncError r=ipc-reviewers,nika
Differential Revision: https://phabricator.services.mozilla.com/D202260
2024-03-13 13:10:23 +00:00
Alex Franchuk
6943de4e7b Bug 1751995 - Remove TestStackHooks r=ipc-reviewers,nika
This tests how interrupt calls occur relative to others.

Differential Revision: https://phabricator.services.mozilla.com/D202259
2024-03-13 13:10:23 +00:00
Alex Franchuk
325edadda3 Bug 1751995 - Remove TestShutdown r=ipc-reviewers,nika
This relies on crashing (exiting) the child actor, so it isn't suitable for gtest.

Differential Revision: https://phabricator.services.mozilla.com/D202258
2024-03-13 13:10:22 +00:00
Alex Franchuk
92d37140e1 Bug 1751995 - Migrate TestShmem r=ipc-reviewers,nika
Differential Revision: https://phabricator.services.mozilla.com/D202257
2024-03-13 13:10:22 +00:00
Alex Franchuk
466ca66588 Bug 1751995 - Migrate TestSelfManage r=ipc-reviewers,nika
Previously named TestSelfManageRoot (based on the initial actor name), but I though the test is
better named "SelfManage".

Differential Revision: https://phabricator.services.mozilla.com/D202062
2024-03-13 13:10:21 +00:00
Alex Franchuk
922844903c Bug 1751995 - Remove TestSanity r=ipc-reviewers,nika
Tests such as TestJSON and TestDataStructures more thoroughly exercise what this is testing.

Differential Revision: https://phabricator.services.mozilla.com/D202061
2024-03-13 13:10:21 +00:00
Alex Franchuk
befb85aa99 Bug 1751995 - Remove TestRPC r=ipc-reviewers,nika
If I understand the test correctly, this doesn't really test anything that other tests don't cover
already.

Differential Revision: https://phabricator.services.mozilla.com/D202060
2024-03-13 13:10:21 +00:00
Alex Franchuk
6e72f2020b Bug 1751995 - Remove TestRacyUndefer r=ipc-reviewers,nika
This relies on interrupt race behavior.

Differential Revision: https://phabricator.services.mozilla.com/D202059
2024-03-13 13:10:20 +00:00
Alex Franchuk
1267555666 Bug 1751995 - Remove TestRacyReentry r=ipc-reviewers,nika
This relies on interrupt race behavior.

Differential Revision: https://phabricator.services.mozilla.com/D202058
2024-03-13 13:10:20 +00:00
Alex Franchuk
e3d9aeb737 Bug 1751995 - Remove TestRacyInterruptReplies r=ipc-reviewers,nika
This tests interrupt race behavior.

Differential Revision: https://phabricator.services.mozilla.com/D202057
2024-03-13 13:10:19 +00:00
Alex Franchuk
4c90bba5f8 Bug 1751995 - Remove TestRaceDeferral r=ipc-reviewers,nika
This relies on interrupt race behavior.

Differential Revision: https://phabricator.services.mozilla.com/D202056
2024-03-13 13:10:19 +00:00
Alex Franchuk
474b930605 Bug 1751995 - Remove TestRaceDeadlock r=ipc-reviewers,nika
This tests interrupt-specific race mediation.

Differential Revision: https://phabricator.services.mozilla.com/D202055
2024-03-13 13:10:18 +00:00
Alex Franchuk
a5878bc858 Bug 1751995 - Remove TestNestedLoops r=ipc-reviewers,nika
This test relies on/tests `intr` behavior.

Differential Revision: https://phabricator.services.mozilla.com/D202054
2024-03-13 13:10:18 +00:00
Alex Franchuk
997d128b9f Bug 1751995 - Migrate TestMultiMgrs r=ipc-reviewers,nika
Differential Revision: https://phabricator.services.mozilla.com/D201751
2024-03-13 13:10:18 +00:00
Alex Franchuk
4fd8eaed07 Bug 1751995 - Migrate TestManyChildAllocs r=ipc-reviewers,nika
Differential Revision: https://phabricator.services.mozilla.com/D201736
2024-03-13 13:10:17 +00:00
Alex Franchuk
937a56d51d Bug 1751995 - Remove TestLatency r=ipc-reviewers,nika
It's more of a benchmark than a test.

Differential Revision: https://phabricator.services.mozilla.com/D201735
2024-03-13 13:10:17 +00:00
Alex Franchuk
7304bda453 Bug 1751995 - Migrate TestJSON r=ipc-reviewers,nika
Differential Revision: https://phabricator.services.mozilla.com/D201727
2024-03-13 13:10:16 +00:00
Alex Franchuk
18c5a3f748 Bug 1751995 - Remove TestInterruptShutdownRace r=ipc-reviewers,nika
Interrupt messages are no longer supported.

Differential Revision: https://phabricator.services.mozilla.com/D201587
2024-03-13 13:10:16 +00:00
Alex Franchuk
42bb9d5f36 Bug 1751995 - Remove TestInterruptRaces r=ipc-reviewers,nika
Interrupt messages are no longer supported.

Differential Revision: https://phabricator.services.mozilla.com/D201586
2024-03-13 13:10:16 +00:00
Alex Franchuk
282be15c73 Bug 1751995 - Remove TestInterruptErrorCleanup r=ipc-reviewers,nika
Interrupt messages are no longer used.

Differential Revision: https://phabricator.services.mozilla.com/D201585
2024-03-13 13:10:15 +00:00
Alex Franchuk
661281e8c6 Bug 1751995 - Migrate TestMostNested r=ipc-reviewers,nika
Previously called TestHighestPrio.

Differential Revision: https://phabricator.services.mozilla.com/D201584
2024-03-13 13:10:15 +00:00
Alex Franchuk
9ac5014fd6 Bug 1751995 - Migrate TestHangs r=ipc-reviewers,nika
A small bit of refactoring involved for this.

Differential Revision: https://phabricator.services.mozilla.com/D200976
2024-03-13 13:10:14 +00:00
Alex Franchuk
723a971652 Bug 1751995 - Remove TestFailedCtor r=ipc-reviewers,nika
Differential Revision: https://phabricator.services.mozilla.com/D201161
2024-03-13 13:10:14 +00:00
Alex Franchuk
9ac08a5284 Bug 1751995 - Migrate TestEndpointOpens r=ipc-reviewers,nika
Differential Revision: https://phabricator.services.mozilla.com/D201160
2024-03-13 13:10:13 +00:00
Alex Franchuk
e43fdb4492 Bug 1751995 - Remove TestEndpointBridge r=ipc-reviewers,nika
This test creates a subprocess from a subprocess, which is not supported.

Differential Revision: https://phabricator.services.mozilla.com/D200975
2024-03-13 13:10:13 +00:00
Alex Franchuk
0eaaeb97e9 Bug 1751995 - Migrate TestDescendant r=ipc-reviewers,nika
Previously named TestDesc, which, based on the test, I can only assume stands for `descendant`.

Differential Revision: https://phabricator.services.mozilla.com/D200966
2024-03-13 13:10:12 +00:00
Alex Franchuk
6b8d00446c Bug 1751995 - Remove TestDemon r=ipc-reviewers,nika
There are fuzzing tests which better accomplish what this test tries to check.

Differential Revision: https://phabricator.services.mozilla.com/D200965
2024-03-13 13:10:12 +00:00
Alex Franchuk
e5fa657f3e Bug 1751995 - Migrate TestDataStructures r=ipc-reviewers,nika
Differential Revision: https://phabricator.services.mozilla.com/D200964
2024-03-13 13:10:12 +00:00
Alex Franchuk
5326796773 Bug 1751995 - Remove TestCrashCleanup r=ipc-reviewers,nika
Crash cleanup in this manner (racing a sync response) is not possible to test with the gtest setup.
It's also a very unlikely scenario as sync messages are used so infrequently.

Differential Revision: https://phabricator.services.mozilla.com/D200963
2024-03-13 13:10:11 +00:00
Alex Franchuk
4900268dff Bug 1751995 - Migrate TestCancel r=ipc-reviewers,nika
Updated assertions for how things work now:
  * cancelling now prevents followup nested messages

Differential Revision: https://phabricator.services.mozilla.com/D200962
2024-03-13 13:10:11 +00:00
Alex Franchuk
55f6b1f934 Bug 1751995 - Remove TestBadActor r=ipc-reviewers,nika
You can't race deletion.

Differential Revision: https://phabricator.services.mozilla.com/D200961
2024-03-13 13:10:10 +00:00
Alex Franchuk
47c20f8b9b Bug 1751995 - Migrate TestAsyncReturns r=ipc-reviewers,nika
Differential Revision: https://phabricator.services.mozilla.com/D200960
2024-03-13 13:10:10 +00:00
Alex Franchuk
67c1f33507 Bug 1751995 - Remove TestActorPunning r=ipc-reviewers,nika
It's no longer possible to pun actors in this way.

Differential Revision: https://phabricator.services.mozilla.com/D200958
2024-03-13 13:10:10 +00:00
Alex Franchuk
32998dcc13 Bug 1751995 - Fix genipdl function in the ipdl compiler. r=ipc-reviewers,nika
I was debugging some unexpected generated code (which ended up being fine) and fixed the genipdl
function in the process.

Differential Revision: https://phabricator.services.mozilla.com/D200957
2024-03-13 13:10:09 +00:00
Alex Franchuk
3afbb2695e Bug 1751995 - Improve the gtest IPDL unit test features. r=ipc-reviewers,nika
This adds an optional testname to IPDL_TEST, so that you can have multiple tests for a single
protocol.

This kind of takes advantage of hackiness in the c preprocessor where you can do `XYZ##__VA_ARGS__`
and it'll work for exactly 0 or 1 var args (but will error with more).  Since this is just for
tests, I thought it was an acceptable level of hackiness to avoid the need to make up additional
nonsense test names for interfaces which only define one test.

This also adds support for IPDL tests that run only cross-process or cross-thread (using
`IPDL_TEST_ON`).

Differential Revision: https://phabricator.services.mozilla.com/D200956
2024-03-13 13:10:08 +00:00
Cristian Tuns
d5255e21a3 Backed out 4 changesets (bug 1858627) for causing build bustages with "-p style --lib" CLOSED TREE
Backed out changeset 7f3f50c77406 (bug 1858627)
Backed out changeset 96327aab8126 (bug 1858627)
Backed out changeset 0655d0feb163 (bug 1858627)
Backed out changeset d3a1b3ac7611 (bug 1858627)
2024-03-13 05:23:51 -04:00