Closed Bug 1087619 Opened 7 years ago Closed 7 years ago

content/base/test/chrome/test_bug429785.xul has a function called QueryInteface

Categories

(Core :: DOM: Core & HTML, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla36

People

(Reporter: jrmuizel, Assigned: baku)

Details

Attachments

(1 file)

This is probably unintentional.
Actually it is. That test implements a listener for the nsIConsoleService:

    var listener = {
      QueryInteface : function(iid) {
        if (!iid.equals(Components.interfaces.nsISupports) &&
            !iid.equals(Components.interfaces.nsIConsoleListener)) {
          throw Components.results.NS_NOINTERFACE;
        }
        return this;
      },
      observe : function (msg) { errorLogged = true; }
    };

This seems totally correct.
Am I misunderstanding this bug?
Flags: needinfo?(jmuizelaar)
Should that been done with QueryInterface instead of QueryInteface?
Flags: needinfo?(jmuizelaar)
Assignee: nobody → amarchesini
Attached patch qi.patchSplinter Review
Attachment #8510298 - Flags: review?(ehsan.akhgari)
Attachment #8510298 - Flags: review?(ehsan.akhgari) → review+
https://hg.mozilla.org/mozilla-central/rev/e413f8f8c235
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.