Bug 596636 - [SeaMonkey 2.1] TEST-UNEXPECTED-FAIL | /tests/content/html/content/test/test_bug589696.html | submit event should not be sent

r=mounir.lamouri a=testing

content/html/content/test/test_bug589696.html

@@ -78,84 +78,94 @@ function checkFormNoValidateAttribute(aElementName)
 checkFormNoValidateAttribute('input');
 checkFormNoValidateAttribute('button');
 
+netscape.security.PrivilegeManager.enablePrivilege('UniversalXPConnect');
+var os = Components.classes['@mozilla.org/observer-service;1']
+                   .getService(Components.interfaces.nsIObserverService);
+var observers = os.enumerateObservers("invalidformsubmit");
+
 /**
  * formnovalidate should prevent form validation if set on the submit control
  * used to submit the form.
  *
- * NOTE: if there is no invalidformsubmit observer, the form submission will
- * never be blocked and this test might be a false-positive but that should not
- * be a problem.
+ * The following test should not be done if there is no observer for
+ * "invalidformsubmit" because the form submission will not be canceled in that
+ * case.
  */
-document.getElementById('av').addEventListener("invalid", function(aEvent) {
+
+if (observers.hasMoreElements()) {
+
+  document.getElementById('av').addEventListener("invalid", function(aEvent) {
     aEvent.target.removeAttribute("invalid", arguments.callee, false);
     ok(true, "formnovalidate should not apply on if not set on the submit " +
              "control used for the submission");
     document.getElementById('b').click();
-}, false);
+  }, false);
 
-document.getElementById('bv').addEventListener("invalid", function(aEvent) {
+  document.getElementById('bv').addEventListener("invalid", function(aEvent) {
     aEvent.target.removeAttribute("invalid", arguments.callee, false);
     ok(true, "formnovalidate should not apply on if not set on the submit " +
              "control used for the submission");
     var c = document.getElementById('c');
     c.focus(); synthesizeKey("VK_RETURN", {type: "keypress"});
-}, false);
+  }, false);
 
-document.getElementById('c').addEventListener("invalid", function(aEvent) {
+  document.getElementById('c').addEventListener("invalid", function(aEvent) {
     aEvent.target.removeAttribute("invalid", arguments.callee, false);
     ok(true, "formnovalidate should only apply on submit controls");
     document.getElementById('d').click();
-}, false);
+  }, false);
 
-document.forms[3].addEventListener("submit", function(aEvent) {
+  document.forms[3].addEventListener("submit", function(aEvent) {
     aEvent.target.removeAttribute("submit", arguments.callee, false);
     ok(true, "formnovalidate applies if set on the submit control used for the submission");
     document.getElementById('e').click();
-}, false);
+  }, false);
 
-document.forms[4].addEventListener("submit", function(aEvent) {
+  document.forms[4].addEventListener("submit", function(aEvent) {
     aEvent.target.removeAttribute("submit", arguments.callee, false);
     ok(true, "formnovalidate applies if set on the submit control used for the submission");
     SimpleTest.executeSoon(SimpleTest.finish);
-}, false);
+  }, false);
 
-/**
+  /**
    * We have to be sure invalid events behave as expected.
    * They should be sent before the submit event so we can just create a test
- * failure if we got one when unexpected. All of them should be catched if sent.
+   * failure if we got one when unexpected. All of them should be caught if
+   * sent.
    * At worst, we got random green which isn't harmful.
    * If expected, they will be part of the chain reaction.
    */
-function unexpectedInvalid(aEvent)
-{
+  function unexpectedInvalid(aEvent)
+  {
     aEvent.target.removeAttribute("invalid", unexpectedInvalid, false);
     ok(false, "invalid event should not be sent");
-}
+  }
 
-document.getElementById('dv').addEventListener("invalid", unexpectedInvalid, false);
-document.getElementById('ev').addEventListener("invalid", unexpectedInvalid, false);
+  document.getElementById('dv').addEventListener("invalid", unexpectedInvalid, false);
+  document.getElementById('ev').addEventListener("invalid", unexpectedInvalid, false);
 
-/**
+  /**
    * Some submission have to be canceled. In that case, the submit events should
    * not be sent.
    * Same behavior as unexpected invalid events.
    */
-function unexpectedSubmit(aEvent)
-{
+  function unexpectedSubmit(aEvent)
+  {
     aEvent.target.removeAttribute("submit", unexpectedSubmit, false);
     ok(false, "submit event should not be sent");
-}
+  }
 
-document.forms[0].addEventListener("submit", unexpectedSubmit, false);
-document.forms[1].addEventListener("submit", unexpectedSubmit, false);
-document.forms[2].addEventListener("submit", unexpectedSubmit, false);
+  document.forms[0].addEventListener("submit", unexpectedSubmit, false);
+  document.forms[1].addEventListener("submit", unexpectedSubmit, false);
+  document.forms[2].addEventListener("submit", unexpectedSubmit, false);
 
-SimpleTest.waitForExplicitFinish();
+  SimpleTest.waitForExplicitFinish();
 
-// This is going to call all the tests (with a chain reaction).
-SimpleTest.waitForFocus(function() {
+  // This is going to call all the tests (with a chain reaction).
+  SimpleTest.waitForFocus(function() {
     document.getElementById('a').click();
-});
+  });
+}
 
 </script>
 </pre>