Unreviewed, rolling out r209415.
authorcdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 15 Dec 2016 00:47:54 +0000 (00:47 +0000)
committercdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 15 Dec 2016 00:47:54 +0000 (00:47 +0000)
This workaround is no longer needed now that
<rdar://problem/29518605> has been fixed

Reverted changeset:

"Add HTML interactive form validation blacklist for some
WebKit-specific sites"
https://bugs.webkit.org/show_bug.cgi?id=165470
http://trac.webkit.org/changeset/209415

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@209844 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/html/HTMLFormElement.cpp

index 87b0345..5cb1426 100644 (file)
@@ -1,3 +1,17 @@
+2016-12-14  Chris Dumez  <cdumez@apple.com>
+
+        Unreviewed, rolling out r209415.
+
+        This workaround is no longer needed now that
+        <rdar://problem/29518605> has been fixed
+
+        Reverted changeset:
+
+        "Add HTML interactive form validation blacklist for some
+        WebKit-specific sites"
+        https://bugs.webkit.org/show_bug.cgi?id=165470
+        http://trac.webkit.org/changeset/209415
+
 2016-12-14  Dean Jackson  <dino@apple.com>
 
         Update prefers-reduced-motion syntax
index 01e933e..416a2d1 100644 (file)
@@ -254,14 +254,6 @@ bool HTMLFormElement::validateInteractively()
     return false;
 }
 
-static bool isURLBlacklistedForInteractiveFormValidation(const URL& url)
-{
-    // FIXME: Remove this workaround once <rdar://problem/29518605> is fixed.
-    if (url.host() == "idmsa.apple.com")
-        return true;
-    return false;
-}
-
 void HTMLFormElement::prepareForSubmission(Event& event)
 {
     Frame* frame = document().frame();
@@ -271,7 +263,7 @@ void HTMLFormElement::prepareForSubmission(Event& event)
     m_isSubmittingOrPreparingForSubmission = true;
     m_shouldSubmit = false;
 
-    bool shouldValidate = document().page() && document().page()->settings().interactiveFormValidationEnabled() && !noValidate() && !isURLBlacklistedForInteractiveFormValidation(document().url());
+    bool shouldValidate = document().page() && document().page()->settings().interactiveFormValidationEnabled() && !noValidate();
 
     HTMLFormControlElement* submitElement = submitElementFromEvent(event);
     if (submitElement && submitElement->formNoValidate())