SeleniumHQ
diff --git a/‎javascript/atoms/mouse.js‎
Lines changed: 22 additions & 19 deletions b/‎javascript/atoms/mouse.js‎
Lines changed: 22 additions & 19 deletions
diff --git a/‎javascript/firefox-driver/js/syntheticMouse.js‎
Lines changed: 5 additions & 2 deletions b/‎javascript/firefox-driver/js/syntheticMouse.js‎
Lines changed: 5 additions & 2 deletions
@@ -298,26 +298,29 @@ bot.Mouse.prototype.releaseButton = function(opt_force, opt_count) {
       bot.dom.isInteractable(this.getElement());
   this.fireMouseEvent_(bot.events.EventType.MOUSEUP, null, null, opt_force, opt_count);
 
-  // TODO: Middle button can also trigger click.
-  if (this.buttonPressed_ == bot.Mouse.Button.LEFT &&
-      this.getElement() == this.elementPressed_) {
-    if (!(bot.userAgent.WINDOWS_PHONE &&
-        bot.dom.isElement(this.elementPressed_, goog.dom.TagName.OPTION))) {
-      this.clickElement(this.clientXY_,
-          this.getButtonValue_(bot.events.EventType.CLICK),
-          /* opt_force */ elementInteractableBeforeMouseup);
-    }
-    this.maybeDoubleClickElement_();
-    if (bot.userAgent.IE_DOC_10 &&
-        this.buttonPressed_ == bot.Mouse.Button.LEFT &&
-        bot.dom.isElement(this.elementPressed_, goog.dom.TagName.OPTION)) {
-      this.fireMSPointerEvent(bot.events.EventType.MSLOSTPOINTERCAPTURE,
-          new goog.math.Coordinate(0, 0), 0, bot.Device.MOUSE_MS_POINTER_ID,
-          MSPointerEvent.MSPOINTER_TYPE_MOUSE, false);
+  try { // https://github.com/SeleniumHQ/selenium/issues/1509
+    // TODO: Middle button can also trigger click.
+    if (this.buttonPressed_ == bot.Mouse.Button.LEFT &&
+        this.getElement() == this.elementPressed_) {
+      if (!(bot.userAgent.WINDOWS_PHONE &&
+            bot.dom.isElement(this.elementPressed_, goog.dom.TagName.OPTION))) {
+        this.clickElement(this.clientXY_,
+                          this.getButtonValue_(bot.events.EventType.CLICK),
+                          /* opt_force */ elementInteractableBeforeMouseup);
+      }
+      this.maybeDoubleClickElement_();
+      if (bot.userAgent.IE_DOC_10 &&
+          this.buttonPressed_ == bot.Mouse.Button.LEFT &&
+          bot.dom.isElement(this.elementPressed_, goog.dom.TagName.OPTION)) {
+        this.fireMSPointerEvent(bot.events.EventType.MSLOSTPOINTERCAPTURE,
+                                new goog.math.Coordinate(0, 0), 0, bot.Device.MOUSE_MS_POINTER_ID,
+                                MSPointerEvent.MSPOINTER_TYPE_MOUSE, false);
+      }
+      // TODO: In Linux, this fires after mousedown event.
+    } else if (this.buttonPressed_ == bot.Mouse.Button.RIGHT) {
+      this.fireMouseEvent_(bot.events.EventType.CONTEXTMENU);
     }
-  // TODO: In Linux, this fires after mousedown event.
-  } else if (this.buttonPressed_ == bot.Mouse.Button.RIGHT) {
-    this.fireMouseEvent_(bot.events.EventType.CONTEXTMENU);
+  } catch (ignored) {
   }
   bot.Device.clearPointerMap();
   this.buttonPressed_ = null;
 
@@ -300,9 +300,12 @@ SyntheticMouse.prototype.click = function(target) {
     bot.action.click(element, this.lastMousePosition, this.getMouse_(), true);
   }
 
-  if (bot.dom.isEditable(element) && element.value !== undefined) {
-    goog.dom.selection.setCursorPosition(
+  try { // https://github.com/SeleniumHQ/selenium/issues/1509
+    if (bot.dom.isEditable(element) && element.value !== undefined) {
+      goog.dom.selection.setCursorPosition(
         element, element.value.length);
+    }
+  } catch (ignored) {
   }
 
   this.lastElement = element;
Original file line number	Diff line number	Diff line change
`@@ -300,9 +300,12 @@ SyntheticMouse.prototype.click = function(target) {`
`300`	`300`	`bot.action.click(element, this.lastMousePosition, this.getMouse_(), true);`
`301`	`301`	`}`
`302`	`302`
`303`		`- if (bot.dom.isEditable(element) && element.value !== undefined) {`
`304`		`- goog.dom.selection.setCursorPosition(`
	`303`	`+ try { // https://github.com/SeleniumHQ/selenium/issues/1509`
	`304`	`+ if (bot.dom.isEditable(element) && element.value !== undefined) {`
	`305`	`+ goog.dom.selection.setCursorPosition(`
`305`	`306`	`element, element.value.length);`
	`307`	`+ }`
	`308`	`+ } catch (ignored) {`
`306`	`309`	`}`
`307`	`310`
`308`	`311`	`this.lastElement = element;`