Changeset 222708 in webkit

Timestamp:

Oct 2, 2017 7:32:52 AM (7 years ago)

Author:

Carlos Garcia Campos

Message:

[GTK][WPE] Enable interactive forms validation by default
​https://bugs.webkit.org/show_bug.cgi?id=177737

Reviewed by Michael Catanzaro.

It's currently disabled for no reason.

• Shared/WebPreferencesDefinitions.h:

Location:

trunk/Source/WebKit

Files:

• ChangeLog (modified) (1 diff)
• Shared/WebPreferencesDefinitions.h (modified) (2 diffs)

trunk/Source/WebKit/ChangeLog

@@ +1 @@
+2017-10-02  Carlos Garcia Campos  <cgarcia@igalia.com>
+
+        [GTK][WPE] Enable interactive forms validation by default
+        https://bugs.webkit.org/show_bug.cgi?id=177737
+
+        Reviewed by Michael Catanzaro.
+
+        It's currently disabled for no reason.
+
+        * Shared/WebPreferencesDefinitions.h:
+
 2017-10-02  Michael Catanzaro  <mcatanzaro@igalia.com>
 

trunk/Source/WebKit/Shared/WebPreferencesDefinitions.h

@@ -56 +56 @@
 #else
 #define DEFAULT_PDFPLUGIN_ENABLED false
-#endif
-
-#if PLATFORM(COCOA)
-#define DEFAULT_HTML_INTERACTIVE_FORM_VALIDATION_ENABLED true
-#else
-#define DEFAULT_HTML_INTERACTIVE_FORM_VALIDATION_ENABLED false
 #endif
 
@@ -295 +289 @@
     macro(MediaPreloadingEnabled, mediaPreloadingEnabled, Bool, bool, false, "", "") \
     macro(IntersectionObserverEnabled, intersectionObserverEnabled, Bool, bool, false, "Intersection Observer", "Enable Intersection Observer support") \
-    macro(InteractiveFormValidationEnabled, interactiveFormValidationEnabled, Bool, bool, DEFAULT_HTML_INTERACTIVE_FORM_VALIDATION_ENABLED, "HTML Interactive Form Validation", "HTML interactive form validation") \
+    macro(InteractiveFormValidationEnabled, interactiveFormValidationEnabled, Bool, bool, true, "HTML Interactive Form Validation", "HTML interactive form validation") \
     macro(ShouldSuppressKeyboardInputDuringProvisionalNavigation, shouldSuppressKeyboardInputDuringProvisionalNavigation, Bool, bool, false, "", "") \
     macro(CSSGridLayoutEnabled, cssGridLayoutEnabled, Bool, bool, true, "CSS Grid", "CSS Grid Layout Module support") \