Bug 460769 - arrowscrollbox destruction should cancel smooth scrolling. r=enndeakin

2008-10-20 18:52:12 +02:00 · 2008-10-20 18:52:12 +02:00 · 1c7e7d7d96
--- a/toolkit/content/widgets/scrollbox.xml
+++ b/toolkit/content/widgets/scrollbox.xml
@ -42,6 +42,10 @@
    </content>

    <implementation>
+      <destructor><![CDATA[
+        this._stopSmoothScroll();
+      ]]></destructor>
+
      <field name="_scrollbox">
        document.getAnonymousElementByAttribute(this, "anonid", "scrollbox");
      </field>
@ -303,8 +307,10 @@

      <method name="_stopSmoothScroll">
        <body><![CDATA[
-          clearInterval(this._smoothScrollTimer);
-          this._isScrolling = 0;
+          if (this._isScrolling) {
+            clearInterval(this._smoothScrollTimer);
+            this._isScrolling = 0;
+          }
        ]]></body>
      </method>