Closed
Bug 1554195
Opened 5 years ago
Closed 5 years ago
Remove the MSG_THIS_DOES_NOT_IMPLEMENT_INTERFACE error message
Categories
(Core :: DOM: Bindings (WebIDL), task)
Core
DOM: Bindings (WebIDL)
Tracking
()
RESOLVED
FIXED
mozilla69
Tracking | Status | |
---|---|---|
firefox69 | --- | fixed |
People
(Reporter: bzbarsky, Assigned: bzbarsky)
Details
Attachments
(1 file)
I meant to switch consumers over to MSG_DOES_NOT_IMPLEMENT_INTERFACE back in bug 882653, but apparently failed to...
Assignee | ||
Comment 1•5 years ago
|
||
I was clearly trying to do that in bug 882653 part 3 and failed to. Our
current behavior of passing two args to this error message (which only takes
one arg) is silly, and the only thing that makes it at all sane is that we only
use it in class hooks, which can never have the wrong sort of object, so it's
unreached code.
The comment cleanup is just to make the role of CGAbstractBindingMethod. and
the validity of the changes to it, clearer.
Pushed by bzbarsky@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d715c6ac79a5 Remove MSG_THIS_DOES_NOT_IMPLEMENT_INTERFACE. r=mccr8
Comment 3•5 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
You need to log in
before you can comment on or make changes to this bug.
Description
•