Bug 644645: don't NS_ASSERT when an extra parameter is passed to tabbrowser.addProgressListener, since popular extensions do that and it's annoying

2011-03-25 14:50:00 -04:00 · 2011-03-25 14:50:00 -04:00 · 10be45c537
--- a/browser/base/content/tabbrowser.xml
+++ b/browser/base/content/tabbrowser.xml
@ -1822,9 +1822,12 @@
        <parameter name="aListener"/>
        <body>
          <![CDATA[
-            NS_ASSERT(arguments.length == 1,
-                      "gBrowser.addProgressListener was called with a second argument, " +
-                      "which is not supported. See bug 608628.");
+            if (arguments.length != 1) {
+              Components.utils.reportError("gBrowser.addProgressListener was " +
+                                           "called with a second argument, " +
+                                           "which is not supported. See bug " +
+                                           "608628.");
+            }

            if (!this.mAddProgressListenerWasCalled) {
              this.mAddProgressListenerWasCalled = true;