1417657 - Add basic test for prio() to the IPDL unit tests

Status: RESOLVED FIXED

(Core :: IPC, enhancement)

Comment 1

[Andrew McCreight [:mccr8]]

Attachment: Bug 1417657 - Add basic test for prio().

Review commit: https://reviewboard.mozilla.org/r/201732/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/201732/

Comment 2

[Ben Kelly [:bkelly, not reviewing]]

Can you provide a bit more context for this?  Is it an existing feature that we just don't use in the system anywhere?

Comment 3

[Andrew McCreight [:mccr8]]

(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

[Ben Kelly [:bkelly, not reviewing]]

Comment on attachment 8930608 [details]
Bug 1417657 - Add basic test for prio().

https://reviewboard.mozilla.org/r/201732/#review207010

Comment 5

[Andrew McCreight [:mccr8]]

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.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f515ace4d2da
Add basic test for prio(). r=bkelly

Comment 7

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/f515ace4d2da