Title: [241277] trunk/Tools
Revision
241277
Author
[email protected]
Date
2019-02-11 13:47:56 -0800 (Mon, 11 Feb 2019)

Log Message

Unreviewed, rolling out r241269.

Caused 5 layout tests crashes on Mac WK1

Reverted changeset:

"DumpRenderTree should report unknown options"
https://bugs.webkit.org/show_bug.cgi?id=191303
https://trac.webkit.org/changeset/241269

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (241276 => 241277)


--- trunk/Tools/ChangeLog	2019-02-11 20:43:03 UTC (rev 241276)
+++ trunk/Tools/ChangeLog	2019-02-11 21:47:56 UTC (rev 241277)
@@ -1,3 +1,15 @@
+2019-02-11  Truitt Savell  <[email protected]>
+
+        Unreviewed, rolling out r241269.
+
+        Caused 5 layout tests crashes on Mac WK1
+
+        Reverted changeset:
+
+        "DumpRenderTree should report unknown options"
+        https://bugs.webkit.org/show_bug.cgi?id=191303
+        https://trac.webkit.org/changeset/241269
+
 2019-02-11  Wenson Hsieh  <[email protected]>
 
         fast/forms/ios/force-gregorian-calendar-for-credit-card-expiry.html does not work on iPad

Modified: trunk/Tools/DumpRenderTree/TestOptions.cpp (241276 => 241277)


--- trunk/Tools/DumpRenderTree/TestOptions.cpp	2019-02-11 20:43:03 UTC (rev 241276)
+++ trunk/Tools/DumpRenderTree/TestOptions.cpp	2019-02-11 21:47:56 UTC (rev 241277)
@@ -75,7 +75,7 @@
         auto value = pairString.substr(equalsLocation + 1, pairEnd - (equalsLocation + 1));
         if (key == "enableAttachmentElement")
             enableAttachmentElement = parseBooleanTestHeaderValue(value);
-        else if (key == "useAcceleratedDrawing")
+        if (key == "useAcceleratedDrawing")
             useAcceleratedDrawing = parseBooleanTestHeaderValue(value);
         else if (key == "enableIntersectionObserver")
             enableIntersectionObserver = parseBooleanTestHeaderValue(value);
@@ -111,52 +111,6 @@
             enableCSSLogical = parseBooleanTestHeaderValue(value);
         else if (key == "internal:AdClickAttributionEnabled")
             adClickAttributionEnabled = parseBooleanTestHeaderValue(value);
-        else if (key == "applicationManifest")
-            applicationManifest = value;
-        else if (key == "experimental:CSSCustomPropertiesAndValuesEnabled")
-            enableCSSCustomPropertiesAndValues = parseBooleanTestHeaderValue(value);
-        else if (key == "useFlexibleViewport")
-            useFlexibleViewport = parseBooleanTestHeaderValue(value);
-        else if (key == "useThreadedScrolling")
-            useThreadedScrolling  = parseBooleanTestHeaderValue(value);
-        else if (key == "experimental:DarkModeCSSEnabled")
-            enableDarkModeCSS = parseBooleanTestHeaderValue(value);
-        else if (key == "experimental:CSSTypedOMEnabled")
-            enableCSSTypedOM = parseBooleanTestHeaderValue(value);
-        else if (key == "punchOutWhiteBackgrounds")
-            punchOutWhiteBackgrounds = parseBooleanTestHeaderValue(value);
-        else if (key == "enableEditableImages")
-            enableEditableImages = parseBooleanTestHeaderValue(value);
-        else if (key == "useCharacterSelectionGranularity")
-            useCharacterSelectionGranularity = parseBooleanTestHeaderValue(value);
-        else if (key == "spellCheckingDots")
-            spellCheckingDots = parseBooleanTestHeaderValue(value);
-        else if (key == "experimental:CSSPaintingAPIEnabled")
-            enableCSSPaintingAPI = parseBooleanTestHeaderValue(value);
-        else if (key == "experimental:PointerEventsEnabled")
-            enablePointerEvents = parseBooleanTestHeaderValue(value);
-        else if (key =="useMockScrollbars")
-            useMockScrollbars = parseBooleanTestHeaderValue(value);
-        else if (key =="ignoresViewportScaleLimits")
-            ignoresViewportScaleLimits = parseBooleanTestHeaderValue(value);
-        else if (key =="shouldIgnoreMetaViewport")
-            shouldIgnoreMetaViewport = parseBooleanTestHeaderValue(value);
-        else if (key =="enableProcessSwapOnNavigation")
-            enableProcessSwapOnNavigation = parseBooleanTestHeaderValue(value);
-        else if (key =="runSingly")
-            runSingly = parseBooleanTestHeaderValue(value);
-        else if (key =="internal:WebAPIStatisticsEnabled")
-            enableWebAPIStatistics = parseBooleanTestHeaderValue(value);
-        else if (key =="internal:SourceBufferChangeTypeEnabled")
-            enableSourceBufferChangeType = parseBooleanTestHeaderValue(value);
-        else if (key =="needsSiteSpecificQuirks")
-            needsSiteSpecificQuirks = parseBooleanTestHeaderValue(value);
-        else if (key =="modernMediaControls")
-            modernMediaControls = parseBooleanTestHeaderValue(value);
-        else if (key == "experimental:WebGL2Enabled")
-            enableWebGL2 = parseBooleanTestHeaderValue(value);
-        else
-            RELEASE_ASSERT_NOT_REACHED();
         pairStart = pairEnd + 1;
     }
 }

Modified: trunk/Tools/DumpRenderTree/TestOptions.h (241276 => 241277)


--- trunk/Tools/DumpRenderTree/TestOptions.h	2019-02-11 20:43:03 UTC (rev 241276)
+++ trunk/Tools/DumpRenderTree/TestOptions.h	2019-02-11 21:47:56 UTC (rev 241277)
@@ -47,30 +47,6 @@
     bool enableCSSLogical { false };
     bool adClickAttributionEnabled { false };
     std::string jscOptions;
-    // FIXME: Implement the following test options,
-    // https://bugs.webkit.org/show_bug.cgi?id=194396
-    bool enableCSSCustomPropertiesAndValues { false };
-    bool useFlexibleViewport { false };
-    bool useThreadedScrolling { true };
-    bool enableDarkModeCSS { true };
-    bool enableCSSTypedOM { false };
-    bool punchOutWhiteBackgrounds { false };
-    bool enableEditableImages { false};
-    bool useCharacterSelectionGranularity { false };
-    bool spellCheckingDots { false };
-    bool enableCSSPaintingAPI { false };
-    bool enablePointerEvents { true };
-    bool useMockScrollbars { false };
-    bool ignoresViewportScaleLimits { true };
-    bool shouldIgnoreMetaViewport { false };
-    bool enableProcessSwapOnNavigation { false };
-    bool runSingly { false };
-    bool enableWebAPIStatistics { false };
-    bool enableSourceBufferChangeType { true };
-    bool needsSiteSpecificQuirks { false };
-    bool modernMediaControls { true };
-    bool enableWebGL2 { false };
-    std::string applicationManifest;
 
     TestOptions(const std::string& pathOrURL, const std::string& absolutePath);
     bool webViewIsCompatibleWithOptions(const TestOptions&) const;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to