Closed Bug 1306708 Opened 6 years ago Closed 6 years ago

Rename prio() annotation to nesting; add new prio() annotation


(Core :: IPC, defect)

Not set



Tracking Status
firefox52 --- fixed


(Reporter: billm, Assigned: billm)




(2 files)

This will allow us to use prio() for event queue priority, which makes more sense. The nesting annotation can be something like:
  nesting(inside_sync) // formerly prio(high)
  nesting(inside_cpow) // formerly prio(urgent)
This is a pretty straightforward renaming. The only hard part was trying to get English phrases in comments to make sense using the new language. I sort of settled on "nested level" as a replacement for priority. I think "nested" is a little better than "nesting" because it makes it clear that it's about what the message can nest in rather than what can nest inside it.

I left space in the message for a priority bit, which will be used in the next patch. In this patch it's always unset.
Attachment #8796772 - Flags: review?(dvander)
This patch adds a prio() annotation to IPDL that is orthogonal to nested(). Right now it doesn't do anything. In bug 1306591 Thinker is going to use it to post messages to a special event queue that will be processed preferentially.
Attachment #8796773 - Flags: review?(dvander)
Attachment #8796772 - Flags: review?(dvander) → review+
Attachment #8796773 - Flags: review?(dvander) → review+
Pushed by
Rename prio to nested in IPDL (r=dvander)
Add prio() for message priority to IPDL (r=dvander)
Pushed by
Revert "Bug 1306708 - Fix rebasing bustage on a CLOSED TREE"
Revert "Bug 1306708 - Add prio() for message priority to IPDL (r=dvander)"
Revert "Bug 1306708 - Rename prio to nested in IPDL (r=dvander)" on a CLOSED TREE
Pushed by
Rename prio to nested in IPDL (r=dvander)
Add prio() for message priority to IPDL (r=dvander)
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.