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);