Bug 596935 - this._dragger is null into InputHandler.js when dismissing a context menu [r=mfinkle,stechz]
authorVivien Nicolas <21@vingtetun.org>
Thu, 16 Sep 2010 19:16:30 +0200
changeset 66659 feb9a75245377141705f7845c4650f9f53f6f9e6
parent 66658 d9f10f50819a15b62e8401d73c464ec80bbfd6aa
child 66660 39637b457f431f5ac06a5cf5c8078cc42710aad5
push id1
push userroot
push dateMon, 20 Oct 2014 17:29:22 +0000
reviewersmfinkle, stechz
bugs596935
Bug 596935 - this._dragger is null into InputHandler.js when dismissing a context menu [r=mfinkle,stechz]
mobile/chrome/content/InputHandler.js
--- a/mobile/chrome/content/InputHandler.js
+++ b/mobile/chrome/content/InputHandler.js
@@ -439,19 +439,21 @@ MouseModule.prototype = {
 
     this._targetScrollInterface = targetScrollInterface;
     this._dragger = dragger;
     this._clicker = (targetClicker) ? targetClicker.customClicker : null;
 
     if (this._clicker)
       this._clicker.mouseDown(aEvent.clientX, aEvent.clientY);
 
-    let draggable = this._dragger.isDraggable(targetScrollbox, targetScrollInterface);
-    if (this._dragger && (draggable.x || draggable.y))
-      this._doDragStart(aEvent);
+    if (this._dragger) {
+      let draggable = this._dragger.isDraggable(targetScrollbox, targetScrollInterface);
+      if (draggable.x || draggable.y)
+        this._doDragStart(aEvent);
+    }
 
     if (this._targetIsContent(aEvent)) {
       this._recordEvent(aEvent);
     }
     else {
       if (this._clickTimeout) {
         // cancel all pending content clicks
         window.clearTimeout(this._clickTimeout);