Bug 897701 - Don't entrain the target of the pan longer than necessary. r=jlebar, a=leo+
authorKyle Huey <khuey@kylehuey.com>
Wed, 24 Jul 2013 16:27:45 -0700
changeset 119813 3258648233dcc4a61cca5d97813646ca0264eec9
parent 119812 c196ea4390f8ffa610f492ee9750ac2cf696ebb4
child 119814 e8ded8d2e5cf277515661963b55f437d9feb6100
push id1006
push userryanvm@gmail.com
push dateFri, 26 Jul 2013 16:57:26 +0000
reviewersjlebar, leo
bugs897701
milestone18.1
Bug 897701 - Don't entrain the target of the pan longer than necessary. r=jlebar, a=leo+
dom/browser-element/BrowserElementPanning.js
--- a/dom/browser-element/BrowserElementPanning.js
+++ b/dom/browser-element/BrowserElementPanning.js
@@ -211,16 +211,19 @@ const ContentPanning = {
     if (this.target && click && (this.panning || this.preventNextClick)) {
       let target = this.target;
       let view = target.ownerDocument ? target.ownerDocument.defaultView
                                       : target;
       view.addEventListener('click', this, true, true);
     }
 
     this._finishPanning();
+
+    // Now that we're done, avoid entraining the thing we just panned.
+    this.pointerDownTarget = null;
   },
 
   // True when there's an async pan-zoom controll watching the
   // outermost scrollable frame, and we're waiting to see whether
   // we're going to take over from it and synchronously scroll an
   // inner scrollable frame.
   detectingScrolling: false,