I submitted my addon to the firefox amo directory and it got reviewed and passed, but the reviewer sent me this message:
2) The following error appears in the Error Console. It's fairly easy to fix.
Error: gBrowser.addProgressListener was called with a second argument, which is not supported. See bug 608628. Source File: chrome://browser/content/tabbrowser.xml Line: 1840
The thing is I have nver touched chrome://browser/content/tabbrowser.xml and dont even know where to find it... so how do I fix this problem?
Thanks!
I submitted my addon to the firefox amo directory and it got reviewed and passed, but the reviewer sent me this message:
2) The following error appears in the Error Console. It's fairly easy to fix.
Error: gBrowser.addProgressListener was called with a second argument, which is not supported. See bug 608628. Source File: chrome://browser/content/tabbrowser.xml Line: 1840
The thing is I have nver touched chrome://browser/content/tabbrowser.xml and dont even know where to find it... so how do I fix this problem?
Thanks!
Share Improve this question edited Feb 11, 2014 at 9:04 Wladimir Palant 57.7k12 gold badges99 silver badges127 bronze badges asked Jun 28, 2011 at 1:56 RyanRyan 10k23 gold badges68 silver badges103 bronze badges2 Answers
Reset to default 5Sounds like you are calling addProgressListener() somewhere in your code with multiple arguments. It is a single argument function - you can find the documentation here:
https://developer.mozilla/en/XUL/tabbrowser#m-addProgressListener
The location chrome://browser/content/tabbrowser.xml is where the function is defined, and line 1840 is the location in the file where the "don't call this function with more than one argument" error is thrown. You can find the file in a check out of the FF source at (I think) browser/base/content/tabbrowser.xml, but you probably don't need to examine it in this case.
That's just a warning about a mon mistake. nsIWebProgress.addProgressListener() supports two parameters. However, <tabbrowser> and <browser> elements don't support this second parameter and ignore it (see tabbrowser.addProgressListener()). People were often using aNotifyMask parameter nevertheless without being aware that it doesn't do anything, so this warning has been added to make sure they notice.