Closed
Bug 910020
Opened 12 years ago
Closed 12 years ago
Rename "RPC" To "Interrupt"
Categories
(Core :: IPC, defect)
Core
IPC
Tracking
()
RESOLVED
FIXED
mozilla27
People
(Reporter: dvander, Assigned: dvander)
References
Details
(Keywords: dev-doc-needed)
Attachments
(1 file, 1 obsolete file)
|
150.23 KB,
patch
|
bent.mozilla
:
review+
|
Details | Diff | Splinter Review |
For E10s we'd like to implement a new protocol that is like RPC, but only in the child -> parent direction. It'll essentially be exactly like a sync message, except that if the child process wakes up for an urgent message, it is allowed to initiate an inner sync message on the same protocol.
This new protocol will need a name.
Chris suggested renaming the existing RPC protocol to "Interrupt" ('intr' in ipdl), and then stealing "rpc" for the new protocol. This shouldn't affect too many people since RPC is discouraged in the IPDL docs. And as a bonus it makes the existing RPC protocol sound a little scarier.
I'll post a patch soon, if anyone has better names I'll take 'em.
I think this sounds fine.
| Assignee | ||
Comment 2•12 years ago
|
||
| Assignee | ||
Comment 3•12 years ago
|
||
Sorry for the gigantic patch. This one is literally just s/RPC/Interrupt/ (and some renamed files) so hopefully it's not as bad as the first refactoring.
Attachment #796873 -
Attachment is obsolete: true
Attachment #800376 -
Flags: review?(cjones.bugs)
Comment on attachment 800376 [details] [diff] [review]
interrupt.patch
If it builds and tests pass then rs=me to rename this stuff.
Attachment #800376 -
Flags: review?(cjones.bugs) → review+
Thanks bent!
| Assignee | ||
Comment 6•12 years ago
|
||
Comment 7•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
Updated•11 years ago
|
Keywords: dev-doc-needed
You need to log in
before you can comment on or make changes to this bug.
Description
•