Closed
Bug 1417657
Opened 7 years ago
Closed 7 years ago
Add basic test for prio() to the IPDL unit tests
Categories
(Core :: IPC, enhancement)
Core
IPC
Tracking
()
RESOLVED
FIXED
mozilla59
| Tracking | Status | |
|---|---|---|
| firefox59 | --- | fixed |
People
(Reporter: mccr8, Assigned: mccr8)
Details
Attachments
(1 file)
No description provided.| Comment hidden (mozreview-request) |
|
||
Comment 2•7 years ago
|
||
Can you provide a bit more context for this? Is it an existing feature that we just don't use in the system anywhere?
|
Assignee | |
Comment 3•7 years ago
|
||
(In reply to Ben Kelly [:bkelly] from comment #2) > Can you provide a bit more context for this? Is it an existing feature that > we just don't use in the system anywhere? We use this in real Firefox IPDL files. I just wanted a little test for it in the IPDL unit tests. We don't run those tests automatically, but ideally at some point we would.
|
|
||
Comment 4•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8930608 [details] Bug 1417657 - Add basic test for prio(). https://reviewboard.mozilla.org/r/201732/#review207010
Attachment #8930608 -
Flags: review?(bkelly) → review+
|
|
Assignee | |
Comment 5•7 years ago
|
||
Another part of the context here is that I am also maintaining an IPDL parser written in Rust ( https://github.com/amccreight/ipdl_parser ), which is used for Searchfox, and it is convenient to have the same set of unit tests for both parsers.
Pushed by amccreight@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/f515ace4d2da Add basic test for prio(). r=bkelly
|
||
Comment 7•7 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/f515ace4d2da
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox59:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
You need to log in
before you can comment on or make changes to this bug.
Description
•