Conversation
|
FYI, you need to use backticks, with single quotes |
47c25cf to
8f47b63
Compare
jdm
left a comment
There was a problem hiding this comment.
Implementation looks good; I just have a few questions about the unit tests.
fe1abca to
a2bdb97
Compare
a2bdb97 to
3d4521e
Compare
❌ 1 Tests Failed:
View the top 1 failed test(s) by shortest run time
To view more test analytics, go to the Test Analytics Dashboard |
|
The UT is stuck.. I've rerun multiple times but still... |
|
I clicked "update branch" in UI. UT finally starts to run. |
|
Emm the same test is still failing. |
|
Ok let me check out the test. |
120ae5f to
723b929
Compare
|
I added your suggestion and reverted the wpt expected values. Perhaps there is something wrong with the WPT testing on my machine. Can you update the expectations? |
We add some channels and wait for at least one of the channels to send a message. Under the hood it uses crossbeam select and IpcReceiverSet. We changes the implementation of IpcReceiverSet to use GenericReceiverSet in the CoreResourceThread setup. Signed-off-by: Narfinger <[email protected]>
Signed-off-by: Narfinger <[email protected]>
a905bc7 to
d1da93d
Compare
Signed-off-by: Narfinger <[email protected]>
Signed-off-by: Narfinger <[email protected]>
Signed-off-by: Narfinger <[email protected]>
Signed-off-by: Narfinger <[email protected]>
Signed-off-by: Narfinger <[email protected]>
Signed-off-by: Narfinger <[email protected]>
Signed-off-by: Narfinger <[email protected]>
Signed-off-by: Narfinger <[email protected]>
Signed-off-by: Narfinger <[email protected]>
|
🔨 Triggering try run (#20755628182) for Linux (WPT) |
|
Test results for linux-wpt from try job (#20755628182): Flaky unexpected result (31)
Stable unexpected results that are known to be intermittent (25)
Stable unexpected results (4)
|
|
|
Signed-off-by: Josh Matthews <[email protected]>
|
FWIW, it's now possible to do |
Signed-off-by: Josh Matthews <[email protected]>
Signed-off-by: Josh Matthews <[email protected]>
This implements GenericReceiverSet similar to IpcReceiverSet. This allows us to wait on a group of channels.
IpcReceiverSet was allowed to use IpcReceivers of different type, i.e.,
IpcReceiver<Foo>andIpcReceiver<Bar>in the same select query. This changes with GenericReceiverSet to only allow one type, i.e.,GenericReceiver<Foo>. As this functionality was only used in the CoreResourceThread, we changed the setup slightly for the memory reporter.With this we also change the implementation of CoreResourceThread to now use the GenericReceiverSet.
Signed-off-by: Narfinger [email protected]
Testing: New testcases were added to GenericReceiverSet and browsing works normally.