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

RESOLVED FIXED in mozilla36

Status

()

Core
DOM
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jrmuizel, Assigned: baku)

Tracking

unspecified
mozilla36
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
This is probably unintentional.
(Assignee)

Comment 1

3 years ago
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)
(Reporter)

Comment 2

3 years ago
Should that been done with QueryInterface instead of QueryInteface?
Flags: needinfo?(jmuizelaar)
(Assignee)

Updated

3 years ago
Assignee: nobody → amarchesini
(Assignee)

Comment 3

3 years ago
Created attachment 8510298 [details] [diff] [review]
qi.patch
Attachment #8510298 - Flags: review?(ehsan.akhgari)

Updated

3 years ago
Attachment #8510298 - Flags: review?(ehsan.akhgari) → review+
(Assignee)

Comment 4

3 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/e413f8f8c235
https://hg.mozilla.org/mozilla-central/rev/e413f8f8c235
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.