EranMes: Fixed clicking based on mouse coordinates.

r12492
patcadelina · Jun 15, 2011 · 70efdb8 · 70efdb8
1 parent 0c56141
commit 70efdb8
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/java/client/test/org/openqa/selenium/interactions/BasicMouseInterfaceTest.java b/java/client/test/org/openqa/selenium/interactions/BasicMouseInterfaceTest.java
@@ -184,8 +184,12 @@ public void testContextClick() {
   }
 
   @JavascriptEnabled
-  @Ignore({ANDROID, IE, FIREFOX, REMOTE, IPHONE, CHROME, SELENESE, OPERA})
+  @Ignore({ANDROID, IE, REMOTE, IPHONE, CHROME, SELENESE, OPERA})
   public void testMoveAndClick() {
+    if (!supportsNativeEvents()) {
+      System.out.println("Skipping mouse test: needs native events");
+      return;
+    }
     driver.get(pages.javascriptPage);
 
     WebElement toClick = driver.findElement(By.id("clickField"));

diff --git a/javascript/firefox-driver/extension/components/firefoxDriver.js b/javascript/firefox-driver/extension/components/firefoxDriver.js
@@ -1220,7 +1220,7 @@ FirefoxDriver.prototype.mouseClick = function(respond, parameters) {
     var currentPosition = respond.session.getMousePosition();
     var browserOffset = getBrowserSpecificOffset_(respond.session.getBrowser());
 
-    events.click(node, node, currentPosition.x + browserOffset.x,
+    events.click(node, currentPosition.x + browserOffset.x,
         currentPosition.y + browserOffset.y, 1);
 
     var dummyIndicator = {