xpidl allows scriptable 'attribute nsID', which is a Bad Idea

NEW
Unassigned

Status

()

Core
XPCOM
3 years ago
3 years ago

People

(Reporter: nwgh, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(firefox43 affected)

Details

The crash reported in bug 1198058 was caused by the patch in bug 1127618, which added an 'attribute nsID' to nsILoadGroup, a scriptable interface. According to http://mxr.mozilla.org/mozilla-central/source/xpcom/base/nsrootidl.idl#57, one shouldn't do that, and xpidl will throw an error if you do! Indeed, it is true that one should not do that, but alas, it is not true that xpidl will throw an error if you do :( Xpidl really should, as that would've saved me some trouble, and made that comment fully correct.
You need to log in before you can comment on or make changes to this bug.