Fixing submit simulation for standard compliant drivers

Author: barancev

java/client/src/org/openqa/selenium/remote/RemoteWebElement.java

@@ -89,7 +89,9 @@ public void submit() {
  execute(DriverCommand.SUBMIT_ELEMENT, ImmutableMap.of("id", id));
  } else {
  WebElement form = findElement(By.xpath("./ancestor-or-self::form"));
- parent.executeScript("arguments[0].submit()", form);
+ parent.executeScript("var e = arguments[0].ownerDocument.createEvent('Event');" +
+ "e.initEvent('submit', true, true);" +
+ "if (arguments[0].dispatchEvent(e)) { arguments[0].submit() }", form);
  }
  }
 

java/client/test/org/openqa/selenium/CorrectEventFiringTest.java

@@ -332,7 +332,6 @@ public void testSendingKeysToAFocusedElementShouldNotBlurThatElement() {
 
  @JavascriptEnabled
  @Test
- @Ignore(MARIONETTE)
  public void testSubmittingFormFromFormElementShouldFireOnSubmitForThatForm() {
  driver.get(pages.javascriptPage);
  WebElement formElement = driver.findElement(By.id("submitListeningForm"));
@@ -341,7 +340,6 @@ public void testSubmittingFormFromFormElementShouldFireOnSubmitForThatForm() {
  }
 
  @JavascriptEnabled
- @Ignore({MARIONETTE})
  @Test
  public void testSubmittingFormFromFormInputSubmitElementShouldFireOnSubmitForThatForm() {
  driver.get(pages.javascriptPage);
@@ -351,7 +349,6 @@ public void testSubmittingFormFromFormInputSubmitElementShouldFireOnSubmitForTha
  }
 
  @JavascriptEnabled
- @Ignore({MARIONETTE})
  @Test
  public void testSubmittingFormFromFormInputTextElementShouldFireOnSubmitForThatFormAndNotClickOnThatInput() {
  driver.get(pages.javascriptPage);
@@ -404,7 +401,7 @@ public void testShouldReportTheXAndYCoordinatesWhenClicking() {
  }
 
  @JavascriptEnabled
- @Ignore(value = {MARIONETTE}, reason = "Not tested")
+ @Ignore(value = {MARIONETTE})
  @Test
  public void testClickEventsShouldBubble() {
  driver.get(pages.clicksPage);

java/client/test/org/openqa/selenium/FrameSwitchingTest.java

@@ -305,7 +305,6 @@ public void testShouldBeAbleToSwitchToParentFromAnIframe() {
  // ----------------------------------------------------------------------------------------------
 
  @Test
- @Ignore(MARIONETTE)
  public void testShouldContinueToReferToTheSameFrameOnceItHasBeenSelected() {
  driver.get(pages.framesetPage);