Closed
Bug 1627299
Opened 5 years ago
Closed 5 years ago
Remove the separation between API messages and payloads
Categories
(Core :: Graphics: WebRender, task, P3)
Core
Graphics: WebRender
Tracking
()
RESOLVED
FIXED
mozilla77
| Tracking | Status | |
|---|---|---|
| firefox77 | --- | fixed |
People
(Reporter: nical, Assigned: nical)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
It was needed for efficiently serializing API messages into shmems, which we don't support anymore.
Removing it will simplify a bunch of code.
|
Assignee | |
Comment 1•5 years ago
|
||
|
|
Assignee | |
Comment 2•5 years ago
|
||
These just wrap regular std Sender/Receiver without providing any value. Serialize/Deserialize was implement manually for MsgSender/MsgReceiver to assert. Serde being amazing, it provides with annotations to not require the traits to be implemented on some enum variants and assert at runtime which functionally equivalent but less error-prone than the fake trait implementations.
Pushed by nsilva@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/68688c8c3bc8
Remove the separation between API messages and payloads. r=kvark
https://hg.mozilla.org/integration/autoland/rev/b0d79edece13
Remove MsgSender/MsgReceiver. r=kvark
|
||
Comment 4•5 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/68688c8c3bc8
https://hg.mozilla.org/mozilla-central/rev/b0d79edece13
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox77:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla77
You need to log in
before you can comment on or make changes to this bug.
Description
•