Skip to content

/ servo

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement MessagePort and MessageChannel #16622

Closed
wants to merge 15 commits into from
+794 −87
Closed

Implement MessagePort and MessageChannel #16622

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
9edd4ec
Add PortMessageQueue
KiChjang Dec 11, 2016
74bcb9d
Accept transfer argument for StructuredCloneData::write
KiChjang Oct 29, 2017
ea481b7
Allow structured clone reads to return a boolean
KiChjang Nov 1, 2017
055bdd1
Add Transferable trait
KiChjang Mar 11, 2017
5ddcdb5
Add basic skeletons to MessagePort
KiChjang Dec 12, 2016
e99eb47
Implement transfer and transfer-receiving steps on MessagePort
KiChjang Oct 23, 2017
f023385
Use transfer and transfer_receive in StructuredClone callbacks
KiChjang Nov 2, 2017
99842e4
Implement MessageChannel
KiChjang Apr 26, 2017
c20ed6a
Freeze the array object for the MessageEvent ports attribute
KiChjang Nov 3, 2017
e4d075b
Implement transfer argument on window.postMessage
KiChjang Nov 13, 2017
25fa53a
Use ReentrantMutex instead for MessagePortInternal
KiChjang Apr 4, 2018
4da8a00
Accept origin as a parameter in dispatch_jsval
KiChjang Apr 4, 2018
5bc41b5
Fix BorrowMut crash with pending_port_message
KiChjang Apr 6, 2018
498fbef
Detach port on closure and check for detached during transfer
KiChjang May 28, 2018
5693b81
Enable webmessaging tests
KiChjang Apr 27, 2017
File filter...
Filter file types
Clear filters
Jump to…
Jump to file
Failed to load files.

Always

Just for now

Prev
Loading status checks…

Enable webmessaging tests

  • Loading branch information
@KiChjang
KiChjang committed Jun 11, 2019
commit 5693b812073ab31f97cc67cc9c3f8b418627c22e
2 tests/wpt/include.ini
View file
Open in desktop
@@ -137,6 +137,8 @@ skip: true
skip: false
[WebIDL]
skip: false
[webmessaging]
skip: false

This comment has been minimized.

Sign in to view
Copy link
@emilio

emilio Apr 29, 2017

Member

Are we really passing all the tests? Or does this need expectations update?

This comment has been minimized.

Sign in to view
Copy link
@KiChjang

KiChjang Apr 30, 2017

Author Member

It does need update, I still need to investigate issues that make MessagePort not quite work as intended.
[websockets]
skip: false
[webstorage]
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.