Created attachment 585110 [details] [diff] [review] patch We appear to support singleton XPCWrappedNativeProtos, but I don't think this is ever used. I checked MXR and I added an assertion and got a green on try. So this should be safe. May our invariants grow stronger each day! I'd also like to get rid of these uppercase variable names. They're awful.
Comment on attachment 585110 [details] [diff] [review] patch Looks good! This needs sr, I think. I'm guessing we don't need to rev the IDL here since it's just a constant, but blake would know for sure. Flagging him for sr.
(In reply to Bill McCloskey (:billm) from comment #0) > I'd also like to get rid of these uppercase variable names. They're awful. I'd guess that they'd be going away with the new DOM bindings, so it's probably better to just wait. :-)
No, removing enums is binary compatible.