From 174b585bad2a43bab53b42ce2d0aaa105fa7838d Mon Sep 17 00:00:00 2001
From: "mozilla.mano@sent.com" <mozilla.mano@sent.com>
Date: Tue, 21 Aug 2007 22:01:29 -0700
Subject: [PATCH] Bug 343097 - Cleaner arrowscrollbox-clicktoscroll binding.
 r=sspitzer/mconnor.

---
 browser/base/content/tabbrowser.xml | 21 ++++++++++++---------
 1 file changed, 12 insertions(+), 9 deletions(-)
diff --git a/browser/base/content/tabbrowser.xml b/browser/base/content/tabbrowser.xml
index a43df2d97de..2444eac5b59 100644
--- a/browser/base/content/tabbrowser.xml
+++ b/browser/base/content/tabbrowser.xml
@@ -1607,21 +1607,24 @@
         <body>
           <![CDATA[
             if (aDragSession.canDrop && aDragSession.sourceNode) {
-              // autoscroll the tab strip if we drag over the autorepeat
+              var tabStrip = this.mTabContainer.mTabstrip;
+
+              // autoscroll the tab strip if we drag over the scroll
               // buttons, even if we aren't dragging a tab, but then
               // return to avoid drawing the drop indicator
-              var isTabDrag = (aDragSession.sourceNode.parentNode == this.mTabContainer);
               var pixelsToScroll = 0;
-              var tabStrip = this.mTabContainer.mTabstrip;
-              if (aEvent.originalTarget.localName == "autorepeatbutton") {
-                if (aEvent.originalTarget.getAttribute("class") == 
-                    "autorepeatbutton-up")
-                  pixelsToScroll = tabStrip.scrollIncrement * -1;
-                else
-                  pixelsToScroll = tabStrip.scrollIncrement;
+
+              var targetAnonid = aEvent.originalTarget.getAttribute("anonid");
+              if (targetAnonid == "scrollbutton-up") {
+                pixelsToScroll = tabStrip.scrollIncrement * -1;
+                tabStrip.scrollByPixels(pixelsToScroll);
+              }
+              else if (targetAnonid == "scrollbutton-down") {
+                pixelsToScroll = tabStrip.scrollIncrement;
                 tabStrip.scrollByPixels(pixelsToScroll);
               }
 
+              var isTabDrag = (aDragSession.sourceNode.parentNode == this.mTabContainer);
               if (!isTabDrag)
                 return;
 
