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)

task
Not set
normal

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...

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
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.

Attachment

General

Created:
Updated:
Size: