Closed Bug 910020 Opened 12 years ago Closed 12 years ago

Rename "RPC" To "Interrupt"

Categories

(Core :: IPC, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla27

People

(Reporter: dvander, Assigned: dvander)

References

Details

(Keywords: dev-doc-needed)

Attachments

(1 file, 1 obsolete file)

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.
Attached patch interrupt.patchSplinter Review
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+
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
Keywords: dev-doc-needed
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: