852220 - nsIConsoleService does not work properly

Status: RESOLVED WONTFIX

(Core :: General, defect)

Description

[Alice0775 White]

Build Identifier:
http://hg.mozilla.org/mozilla-central/rev/b03bb3ce8cee
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:22.0) Gecko/20130318 Firefox/22.0 ID:20130318030947

Executing the following code in ScratchPad(chrome privilege) should appear an alert box.
However since Aurora20.0a2, alert box does not pop up anymore.

//begin---------------------------
function myLogToConsole(aMessage, aSourceName, aSourceLine, aLineNumber,
                        aColumnNumber, aFlags, aCategory) {
  var consoleService = Components.classes["@mozilla.org/consoleservice;1"]
                                 .getService(Components.interfaces.nsIConsoleService);
  var scriptError = Components.classes["@mozilla.org/scripterror;1"]
                              .createInstance(Components.interfaces.nsIScriptError);
  scriptError.init(aMessage, aSourceName, aSourceLine, aLineNumber,
                   aColumnNumber, aFlags, aCategory);
  consoleService.logMessage(scriptError);
}

var errorListener = {
  QueryInterface: XPCOMUtils.generateQI([Components.interfaces.nsIConsoleListener,
                                         Components.interfaces.nsISupports]),
  observe: function(message) {
    if ("QueryInterface" in message) {
      var error = message.QueryInterface(Components.interfaces.nsIScriptError);
      alert(error.errorMessage);
    }
  }
}

var consoleService = Components.classes["@mozilla.org/consoleservice;1"]
                     .getService(Components.interfaces.nsIConsoleService);
consoleService.registerListener(errorListener);

myLogToConsole("my message", "", "", "",  "", 1, "myerror");

consoleService.unregisterListener(errorListener);
//end---------------------------



If I changed last line
>> consoleService.unregisterListener(errorListener);
to
>> setTimeout(function(){ consoleService.unregisterListener(errorListener);}, 0);

then it works as expected.

Comment 1

[Alice0775 White]

Err
s/However since Aurora20.0a2/However since Aurora21.0a2/

Comment 2

[Boris Zbarsky [:bzbarsky]]

This was a purposeful change with bug 831428.  See in particular bug 831428 comment 9.

I believe this bug as filed is wontfix.

Comment 3

[Alice0775 White]

Stylish 1.3.1( https://addons.mozilla.org/en-US/firefox/addon/stylish/ ) is affected.
See forum http://forums.mozillazine.org/viewtopic.php?f=23&t=2680325

Comment 4

[Boris Zbarsky [:bzbarsky]]

Hmm.  We didn't have any addon messaging about this, did we?  We probably should.

Comment 5

[Jorge Villalobos [:jorgev] (he/him)]

(In reply to Boris Zbarsky (:bz) from comment #4)
> Hmm.  We didn't have any addon messaging about this, did we?  We probably
> should.

Not yet, no. I flagged bug 831428 for addon-compat and we will definitely include it in our messaging.

Comment 6

[Benjamin Smedberg]

Yes, this was an intentional change. I didn't expect that anything other than unit tests could possibly depend on the current behavior, but apparently they do!