Diff
Modified: trunk/Source/WebCore/ChangeLog (208088 => 208089)
--- trunk/Source/WebCore/ChangeLog 2016-10-29 00:28:15 UTC (rev 208088)
+++ trunk/Source/WebCore/ChangeLog 2016-10-29 00:31:42 UTC (rev 208089)
@@ -1,3 +1,17 @@
+2016-10-28 Commit Queue <commit-qu...@webkit.org>
+
+ Unreviewed, rolling out r207700.
+ https://bugs.webkit.org/show_bug.cgi?id=164168
+
+ We are ready after r208086 (Requested by alexchristensen on
+ #webkit).
+
+ Reverted changeset:
+
+ "Unreviewed, rolling out r207582."
+ https://bugs.webkit.org/show_bug.cgi?id=163819
+ http://trac.webkit.org/changeset/207700
+
2016-10-28 Dave Hyatt <hy...@apple.com>
[CSS Parser] Allow unknown properties in will-change
Modified: trunk/Source/WebCore/platform/URLParser.cpp (208088 => 208089)
--- trunk/Source/WebCore/platform/URLParser.cpp 2016-10-29 00:28:15 UTC (rev 208088)
+++ trunk/Source/WebCore/platform/URLParser.cpp 2016-10-29 00:31:42 UTC (rev 208089)
@@ -2827,31 +2827,16 @@
// It should be able to be deduced from m_isValid and m_string.length() to save memory.
}
-enum class URLParserEnabled {
- Undetermined,
- Yes,
- No
-};
+static bool urlParserEnabled = true;
-static URLParserEnabled urlParserEnabled = URLParserEnabled::Undetermined;
-
void URLParser::setEnabled(bool enabled)
{
- urlParserEnabled = enabled ? URLParserEnabled::Yes : URLParserEnabled::No;
+ urlParserEnabled = enabled;
}
bool URLParser::enabled()
{
- if (urlParserEnabled == URLParserEnabled::Undetermined) {
-#if PLATFORM(MAC)
- urlParserEnabled = MacApplication::isSafari() ? URLParserEnabled::Yes : URLParserEnabled::No;
-#elif PLATFORM(IOS)
- urlParserEnabled = IOSApplication::isMobileSafari() ? URLParserEnabled::Yes : URLParserEnabled::No;
-#else
- urlParserEnabled = URLParserEnabled::Yes;
-#endif
- }
- return urlParserEnabled == URLParserEnabled::Yes;
+ return urlParserEnabled;
}
} // namespace WebCore
Modified: trunk/Source/WebCore/testing/js/WebCoreTestSupport.cpp (208088 => 208089)
--- trunk/Source/WebCore/testing/js/WebCoreTestSupport.cpp 2016-10-29 00:28:15 UTC (rev 208088)
+++ trunk/Source/WebCore/testing/js/WebCoreTestSupport.cpp 2016-10-29 00:31:42 UTC (rev 208089)
@@ -121,11 +121,6 @@
InternalSettings::setAllowsAnySSLCertificate(allowAnySSLCertificate);
}
-void setURLParserEnabled(bool enabled)
-{
- URLParser::setEnabled(enabled);
-}
-
void installMockGamepadProvider()
{
#if ENABLE(GAMEPAD)
Modified: trunk/Source/WebCore/testing/js/WebCoreTestSupport.h (208088 => 208089)
--- trunk/Source/WebCore/testing/js/WebCoreTestSupport.h 2016-10-29 00:28:15 UTC (rev 208088)
+++ trunk/Source/WebCore/testing/js/WebCoreTestSupport.h 2016-10-29 00:31:42 UTC (rev 208089)
@@ -56,7 +56,6 @@
void setLogChannelToAccumulate(const WTF::String& name) TEST_SUPPORT_EXPORT;
void initializeLogChannelsIfNecessary() TEST_SUPPORT_EXPORT;
void setAllowsAnySSLCertificate(bool) TEST_SUPPORT_EXPORT;
-void setURLParserEnabled(bool) TEST_SUPPORT_EXPORT;
void installMockGamepadProvider() TEST_SUPPORT_EXPORT;
void connectMockGamepad(unsigned index) TEST_SUPPORT_EXPORT;
Modified: trunk/Tools/ChangeLog (208088 => 208089)
--- trunk/Tools/ChangeLog 2016-10-29 00:28:15 UTC (rev 208088)
+++ trunk/Tools/ChangeLog 2016-10-29 00:31:42 UTC (rev 208089)
@@ -1,3 +1,17 @@
+2016-10-28 Commit Queue <commit-qu...@webkit.org>
+
+ Unreviewed, rolling out r207700.
+ https://bugs.webkit.org/show_bug.cgi?id=164168
+
+ We are ready after r208086 (Requested by alexchristensen on
+ #webkit).
+
+ Reverted changeset:
+
+ "Unreviewed, rolling out r207582."
+ https://bugs.webkit.org/show_bug.cgi?id=163819
+ http://trac.webkit.org/changeset/207700
+
2016-10-28 Alex Christensen <achristen...@webkit.org>
Partially revert 207805 after resolution in URL spec issue 87
Modified: trunk/Tools/DumpRenderTree/mac/DumpRenderTree.mm (208088 => 208089)
--- trunk/Tools/DumpRenderTree/mac/DumpRenderTree.mm 2016-10-29 00:28:15 UTC (rev 208088)
+++ trunk/Tools/DumpRenderTree/mac/DumpRenderTree.mm 2016-10-29 00:31:42 UTC (rev 208089)
@@ -1411,7 +1411,6 @@
int DumpRenderTreeMain(int argc, const char *argv[])
{
- WebCoreTestSupport::setURLParserEnabled(true);
atexit(atexitFunction);
#if PLATFORM(IOS)
Modified: trunk/Tools/TestWebKitAPI/Tests/WebKit2Cocoa/LoadInvalidURLRequest.mm (208088 => 208089)
--- trunk/Tools/TestWebKitAPI/Tests/WebKit2Cocoa/LoadInvalidURLRequest.mm 2016-10-29 00:28:15 UTC (rev 208088)
+++ trunk/Tools/TestWebKitAPI/Tests/WebKit2Cocoa/LoadInvalidURLRequest.mm 2016-10-29 00:31:42 UTC (rev 208089)
@@ -54,8 +54,8 @@
- (void)webView:(WKWebView *)webView didFailProvisionalNavigation:(WKNavigation *)navigation withError:(NSError *)error
{
- EXPECT_WK_STREQ(error.domain, @"WebKitErrorDomain");
- EXPECT_EQ(error.code, WebKitErrorCannotShowURL);
+ EXPECT_WK_STREQ(error.domain, @"NSURLErrorDomain");
+ EXPECT_EQ(error.code, -1003);
EXPECT_TRUE([error.userInfo[@"NSErrorFailingURLKey"] isEqual:literalURL(literal)]);
didFailProvisionalLoad = true;
Modified: trunk/Tools/TestWebKitAPI/Tests/mac/LoadInvalidURLRequest.mm (208088 => 208089)
--- trunk/Tools/TestWebKitAPI/Tests/mac/LoadInvalidURLRequest.mm 2016-10-29 00:28:15 UTC (rev 208088)
+++ trunk/Tools/TestWebKitAPI/Tests/mac/LoadInvalidURLRequest.mm 2016-10-29 00:31:42 UTC (rev 208089)
@@ -77,7 +77,7 @@
didFailProvisionalLoad = false;
Util::run(&didFinishTest);
- EXPECT_TRUE(didFailProvisionalLoad);
+ EXPECT_FALSE(didFailProvisionalLoad);
}
}
Modified: trunk/Tools/WebKitTestRunner/TestController.cpp (208088 => 208089)
--- trunk/Tools/WebKitTestRunner/TestController.cpp 2016-10-29 00:28:15 UTC (rev 208088)
+++ trunk/Tools/WebKitTestRunner/TestController.cpp 2016-10-29 00:31:42 UTC (rev 208089)
@@ -115,7 +115,6 @@
TestController::TestController(int argc, const char* argv[])
{
- WebCoreTestSupport::setURLParserEnabled(true);
initialize(argc, argv);
controller = this;
run();