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