1495814 - Remove pointless do_QueryInterface() calls from nsBaseChannel.h

Status: RESOLVED FIXED

(Core :: Networking, enhancement)

Description

[Andrew McCreight [:mccr8]]

nsBaseChannel.h has two do_QueryInterface methods:

  // Helper function for calling QueryInterface on this.
  nsQueryInterface do_QueryInterface() {
    return nsQueryInterface(static_cast<nsIChannel *>(this));
  }
  // MSVC needs this:
  nsQueryInterface do_QueryInterface(nsISupports *obj) {
    return nsQueryInterface(obj);
  }

The first one is never called, and I'm pretty sure the second one can just be replaced by the do_QueryInterface that is defined in nsCOMPtr.h, so it should be easy to delete them.

My motivation is that I'm changing how nsQueryInterface works in bug 1493226 and I'd rather not have to change these two places.

Comment 1

[Andrew McCreight [:mccr8]]

Attachment: Bug 1495814 - Remove pointless do_QueryInterface() functions from nsBaseChannel.h

The first QI is never used. The second one is the same as the one in
nsCOMPtr.h, so we should be able to call that instead, which can be
done simply by deleting the method. The motivation is that I'm
changing how do_QueryInterface works, and I'd like to avoid changing
this place given that it isn't actually needed.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fc4e7aa8f23e
Remove pointless do_QueryInterface() functions from nsBaseChannel.h r=mayhemer

Comment 3

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/fc4e7aa8f23e