[webkit-changes] [266102] trunk/Tools

2020-08-24 Thread achristensen
Title: [266102] trunk/Tools








Revision 266102
Author achristen...@apple.com
Date 2020-08-24 22:04:44 -0700 (Mon, 24 Aug 2020)


Log Message
Fix TLSVersion.DefaultBehavior on Mojave
https://bugs.webkit.org/show_bug.cgi?id=215791

* TestWebKitAPI/Tests/WebKitCocoa/TLSDeprecation.mm:
HTTPServer::Protocol::HttpsWithLegacyTLS uses tls_protocol_version_t, so this test times out on Mojave.
Enable it for newer OSes.

Modified Paths

trunk/Tools/ChangeLog
trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/TLSDeprecation.mm




Diff

Modified: trunk/Tools/ChangeLog (266101 => 266102)

--- trunk/Tools/ChangeLog	2020-08-25 04:32:13 UTC (rev 266101)
+++ trunk/Tools/ChangeLog	2020-08-25 05:04:44 UTC (rev 266102)
@@ -1,5 +1,14 @@
 2020-08-24  Alex Christensen  
 
+Fix TLSVersion.DefaultBehavior on Mojave
+https://bugs.webkit.org/show_bug.cgi?id=215791
+
+* TestWebKitAPI/Tests/WebKitCocoa/TLSDeprecation.mm:
+HTTPServer::Protocol::HttpsWithLegacyTLS uses tls_protocol_version_t, so this test times out on Mojave.
+Enable it for newer OSes.
+
+2020-08-24  Alex Christensen  
+
 Make TLSVersion.DefaultBehavior more robust
 https://bugs.webkit.org/show_bug.cgi?id=215791
 


Modified: trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/TLSDeprecation.mm (266101 => 266102)

--- trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/TLSDeprecation.mm	2020-08-25 04:32:13 UTC (rev 266101)
+++ trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/TLSDeprecation.mm	2020-08-25 05:04:44 UTC (rev 266102)
@@ -144,7 +144,7 @@
 const uint16_t tls1_1 = 0x0302;
 static NSString *defaultsKey = @"WebKitEnableLegacyTLS";
 
-#if HAVE(NETWORK_FRAMEWORK)
+#if HAVE(NETWORK_FRAMEWORK) && HAVE(TLS_PROTOCOL_VERSION_T)
 
 TEST(TLSVersion, DefaultBehavior)
 {
@@ -160,7 +160,7 @@
 [delegate waitForDidFinishNavigation];
 }
 
-#endif // HAVE(NETWORK_FRAMEWORK)
+#endif // HAVE(NETWORK_FRAMEWORK) && HAVE(TLS_PROTOCOL_VERSION_T)
 
 #if HAVE(TLS_VERSION_DURING_CHALLENGE)
 






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266101] trunk/Source/JavaScriptCore

2020-08-24 Thread keith_miller
Title: [266101] trunk/Source/_javascript_Core








Revision 266101
Author keith_mil...@apple.com
Date 2020-08-24 21:32:13 -0700 (Mon, 24 Aug 2020)


Log Message
DFG should always run CFG Simplification after Constant Folding.
https://bugs.webkit.org/show_bug.cgi?id=215286

Reviewed by Robin Morisset.

We didn't do this originally because LICM, many years ago, was
unsound if the CFG didn't have exactly the right shape around
loops. This is no longer true so we don't have to worry about
changing the CFG anymore. While, this doesn't appear to be a
speedup on JetStream 2 CFG, probably because we'd eventually
simplify the graph in B3, CFG Simplification is very cheap and
make other DFG optimizations easier in the future.

Also, remove unecessary validation rule that no exitOKs can come
before any Phi nodes in DFG. This isn't required and fails after
merging two basic blocks where the latter block has a Phi.

* dfg/DFGCFGSimplificationPhase.cpp:
(JSC::DFG::CFGSimplificationPhase::run):
* dfg/DFGPlan.cpp:
(JSC::DFG::Plan::compileInThreadImpl):
* dfg/DFGValidate.cpp:

Modified Paths

trunk/Source/_javascript_Core/ChangeLog
trunk/Source/_javascript_Core/dfg/DFGCFGSimplificationPhase.cpp
trunk/Source/_javascript_Core/dfg/DFGPlan.cpp
trunk/Source/_javascript_Core/dfg/DFGValidate.cpp




Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (266100 => 266101)

--- trunk/Source/_javascript_Core/ChangeLog	2020-08-25 03:36:45 UTC (rev 266100)
+++ trunk/Source/_javascript_Core/ChangeLog	2020-08-25 04:32:13 UTC (rev 266101)
@@ -1,5 +1,30 @@
 2020-08-24  Keith Miller  
 
+DFG should always run CFG Simplification after Constant Folding.
+https://bugs.webkit.org/show_bug.cgi?id=215286
+
+Reviewed by Robin Morisset.
+
+We didn't do this originally because LICM, many years ago, was
+unsound if the CFG didn't have exactly the right shape around
+loops. This is no longer true so we don't have to worry about
+changing the CFG anymore. While, this doesn't appear to be a
+speedup on JetStream 2 CFG, probably because we'd eventually
+simplify the graph in B3, CFG Simplification is very cheap and
+make other DFG optimizations easier in the future.
+
+Also, remove unecessary validation rule that no exitOKs can come
+before any Phi nodes in DFG. This isn't required and fails after
+merging two basic blocks where the latter block has a Phi.
+
+* dfg/DFGCFGSimplificationPhase.cpp:
+(JSC::DFG::CFGSimplificationPhase::run):
+* dfg/DFGPlan.cpp:
+(JSC::DFG::Plan::compileInThreadImpl):
+* dfg/DFGValidate.cpp:
+
+2020-08-24  Keith Miller  
+
 Remove MovHintRemoval phase
 https://bugs.webkit.org/show_bug.cgi?id=215785
 


Modified: trunk/Source/_javascript_Core/dfg/DFGCFGSimplificationPhase.cpp (266100 => 266101)

--- trunk/Source/_javascript_Core/dfg/DFGCFGSimplificationPhase.cpp	2020-08-25 03:36:45 UTC (rev 266100)
+++ trunk/Source/_javascript_Core/dfg/DFGCFGSimplificationPhase.cpp	2020-08-25 04:32:13 UTC (rev 266101)
@@ -49,9 +49,6 @@
 
 bool run()
 {
-// FIXME: We should make this work in SSA. https://bugs.webkit.org/show_bug.cgi?id=148260
-DFG_ASSERT(m_graph, nullptr, m_graph.m_form != SSA);
-
 const bool extremeLogging = false;
 
 bool outerChanged = false;


Modified: trunk/Source/_javascript_Core/dfg/DFGPlan.cpp (266100 => 266101)

--- trunk/Source/_javascript_Core/dfg/DFGPlan.cpp	2020-08-25 03:36:45 UTC (rev 266100)
+++ trunk/Source/_javascript_Core/dfg/DFGPlan.cpp	2020-08-25 04:32:13 UTC (rev 266101)
@@ -364,6 +364,7 @@
 if (changed) {
 RUN_PHASE(performCFA);
 RUN_PHASE(performConstantFolding);
+RUN_PHASE(performCFGSimplification);
 }
 
 // If we're doing validation, then run some analyses, to give them an opportunity
@@ -429,6 +430,7 @@
 RUN_PHASE(performLivenessAnalysis);
 RUN_PHASE(performCFA);
 RUN_PHASE(performConstantFolding);
+RUN_PHASE(performCFGSimplification);
 RUN_PHASE(performCleanUp); // Reduce the graph size a lot.
 changed = false;
 RUN_PHASE(performStrengthReduction);
@@ -444,6 +446,7 @@
 RUN_PHASE(performLivenessAnalysis);
 RUN_PHASE(performCFA);
 RUN_PHASE(performConstantFolding);
+RUN_PHASE(performCFGSimplification);
 }
 
 // Currently, this relies on pre-headers still being valid. That precludes running CFG


Modified: trunk/Source/_javascript_Core/dfg/DFGValidate.cpp (266100 => 266101)

--- trunk/Source/_javascript_Core/dfg/DFGValidate.cpp	2020-08-25 03:36:45 UTC (rev 266100)
+++ trunk/Source/_javascript_Core/dfg/DFGValidate.cpp	2020-08-25 04:32:13 UTC (rev 266101)
@@ -830,22 +830,16 @@
 for (BasicBlock* block : m_graph.blocksInNaturalOrder()) {
 VALIDATE((block), block->phis.isEmpty());
 
-bool 

[webkit-changes] [266100] trunk/Tools

2020-08-24 Thread commit-queue
Title: [266100] trunk/Tools








Revision 266100
Author commit-qu...@webkit.org
Date 2020-08-24 20:36:45 -0700 (Mon, 24 Aug 2020)


Log Message
Make TLSVersion.DefaultBehavior more robust
https://bugs.webkit.org/show_bug.cgi?id=215791

Patch by Alex Christensen  on 2020-08-24
Reviewed by Darin Adler.

After r265573 sometimes it would assert, which is not a problem because it was failing the first connection
then succeeding the second connection as intended and as happens in the real internet.
Use HTTPServer which accepts a variable number of connections to keep the test coverage.

* TestWebKitAPI/Tests/WebKitCocoa/TLSDeprecation.mm:
(TestWebKitAPI::TEST):
* TestWebKitAPI/cocoa/HTTPServer.h:
* TestWebKitAPI/cocoa/HTTPServer.mm:
(TestWebKitAPI::HTTPServer::respondWithChallengeThenOK):
(TestWebKitAPI::HTTPServer::respondWithOK):

Modified Paths

trunk/Tools/ChangeLog
trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/TLSDeprecation.mm
trunk/Tools/TestWebKitAPI/cocoa/HTTPServer.h
trunk/Tools/TestWebKitAPI/cocoa/HTTPServer.mm




Diff

Modified: trunk/Tools/ChangeLog (266099 => 266100)

--- trunk/Tools/ChangeLog	2020-08-25 02:44:39 UTC (rev 266099)
+++ trunk/Tools/ChangeLog	2020-08-25 03:36:45 UTC (rev 266100)
@@ -1,3 +1,21 @@
+2020-08-24  Alex Christensen  
+
+Make TLSVersion.DefaultBehavior more robust
+https://bugs.webkit.org/show_bug.cgi?id=215791
+
+Reviewed by Darin Adler.
+
+After r265573 sometimes it would assert, which is not a problem because it was failing the first connection
+then succeeding the second connection as intended and as happens in the real internet.
+Use HTTPServer which accepts a variable number of connections to keep the test coverage.
+
+* TestWebKitAPI/Tests/WebKitCocoa/TLSDeprecation.mm:
+(TestWebKitAPI::TEST):
+* TestWebKitAPI/cocoa/HTTPServer.h:
+* TestWebKitAPI/cocoa/HTTPServer.mm:
+(TestWebKitAPI::HTTPServer::respondWithChallengeThenOK):
+(TestWebKitAPI::HTTPServer::respondWithOK):
+
 2020-08-24  Wenson Hsieh  
 
 Unreviewed, fix the internal iOS 13.4 build


Modified: trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/TLSDeprecation.mm (266099 => 266100)

--- trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/TLSDeprecation.mm	2020-08-25 02:44:39 UTC (rev 266099)
+++ trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/TLSDeprecation.mm	2020-08-25 03:36:45 UTC (rev 266100)
@@ -144,9 +144,11 @@
 const uint16_t tls1_1 = 0x0302;
 static NSString *defaultsKey = @"WebKitEnableLegacyTLS";
 
+#if HAVE(NETWORK_FRAMEWORK)
+
 TEST(TLSVersion, DefaultBehavior)
 {
-TCPServer server(TCPServer::Protocol::HTTPS, TCPServer::respondWithOK, tls1_1);
+HTTPServer server(HTTPServer::respondWithOK, HTTPServer::Protocol::HttpsWithLegacyTLS);
 auto delegate = adoptNS([TestNavigationDelegate new]);
 auto webView = adoptNS([WKWebView new]);
 [webView setNavigationDelegate:delegate.get()];
@@ -158,6 +160,8 @@
 [delegate waitForDidFinishNavigation];
 }
 
+#endif // HAVE(NETWORK_FRAMEWORK)
+
 #if HAVE(TLS_VERSION_DURING_CHALLENGE)
 
 TEST(TLSVersion, NetworkSession)


Modified: trunk/Tools/TestWebKitAPI/cocoa/HTTPServer.h (266099 => 266100)

--- trunk/Tools/TestWebKitAPI/cocoa/HTTPServer.h	2020-08-25 02:44:39 UTC (rev 266099)
+++ trunk/Tools/TestWebKitAPI/cocoa/HTTPServer.h	2020-08-25 03:36:45 UTC (rev 266100)
@@ -54,8 +54,9 @@
 size_t totalRequests() const;
 void cancel();
 
+static void respondWithOK(Connection);
 static void respondWithChallengeThenOK(Connection);
-
+
 private:
 static RetainPtr listenerParameters(Protocol, CertificateVerifier&&, RetainPtr&&, Optional port);
 static void respondToRequests(Connection, Ref);


Modified: trunk/Tools/TestWebKitAPI/cocoa/HTTPServer.mm (266099 => 266100)

--- trunk/Tools/TestWebKitAPI/cocoa/HTTPServer.mm	2020-08-25 02:44:39 UTC (rev 266099)
+++ trunk/Tools/TestWebKitAPI/cocoa/HTTPServer.mm	2020-08-25 03:36:45 UTC (rev 266100)
@@ -150,17 +150,22 @@
 "Content-Length: 0\r\n"
 "WWW-Authenticate: Basic realm=\"testrealm\"\r\n\r\n";
 connection.send(challengeHeader, [connection] {
-connection.receiveHTTPRequest([connection] (Vector&&) {
-connection.send(
-"HTTP/1.1 200 OK\r\n"
-"Content-Length: 13\r\n\r\n"
-"Hello, World!"
-);
-});
+respondWithOK(connection);
 });
 });
 }
 
+void HTTPServer::respondWithOK(Connection connection)
+{
+connection.receiveHTTPRequest([connection] (Vector&&) {
+connection.send(
+"HTTP/1.1 200 OK\r\n"
+"Content-Length: 13\r\n\r\n"
+"Hello, World!"
+);
+});
+}
+
 size_t HTTPServer::totalRequests() const
 {
 return m_requestData->requestCount;






___
webkit-changes mailing list
webkit-changes@lists.webkit.org

[webkit-changes] [266099] trunk/Source/WebKit

2020-08-24 Thread beidson
Title: [266099] trunk/Source/WebKit








Revision 266099
Author beid...@apple.com
Date 2020-08-24 19:44:39 -0700 (Mon, 24 Aug 2020)


Log Message
CrashTracer: com.apple.WebKit.WebContent at com.apple.WebKit: WebKit::PDFPlugin::createScrollbar
 and https://bugs.webkit.org/show_bug.cgi?id=215787

Reviewed by Tim Horton.

To quote Tim from r264945:
No new tests; timing is such that I can't reproduce without inserting
intentional delays into the main thread hops, which is further than
I'm willing to go for a test.

This is a speculative fix due to the aforementioned reproducibility issue.

* WebProcess/Plugins/PDF/PDFPlugin.mm:
(WebKit::PDFPlugin::receivedNonLinearizedPDFSentinel): Check on the main thread whenever this is called,
  instead of when deciding to dispatch to the main thread.
(WebKit::PDFPlugin::threadEntry): We can't do this check on the background thread when considering the dispatch
  to the main thread, as the flag might've changed by then. Let's *just* check it on the main thread.
(WebKit::PDFPlugin::adoptBackgroundThreadDocument): We can't do the check on the background thread when
(WebKit::PDFPlugin::updateScrollbars): This is where the crash itself is. All of the Obj-C code in here is
  safe to do after destroy(), up until the very end when we get into pluginView() derefencing.
  So it seems prudent to add another check here.
(WebKit::PDFPlugin::documentDataDidFinishLoading): In addition to receivedNonLinearizedPDFSentinel and
  adoptBackgroundThreadDocument, this is the final of the (3) calls that end up calling installPDFDocument,
  so for added coverage it seems like a prudent place to add the check.

Modified Paths

trunk/Source/WebKit/ChangeLog
trunk/Source/WebKit/WebProcess/Plugins/PDF/PDFPlugin.mm




Diff

Modified: trunk/Source/WebKit/ChangeLog (266098 => 266099)

--- trunk/Source/WebKit/ChangeLog	2020-08-25 02:03:51 UTC (rev 266098)
+++ trunk/Source/WebKit/ChangeLog	2020-08-25 02:44:39 UTC (rev 266099)
@@ -1,3 +1,30 @@
+2020-08-24  Brady Eidson  
+
+CrashTracer: com.apple.WebKit.WebContent at com.apple.WebKit: WebKit::PDFPlugin::createScrollbar
+ and https://bugs.webkit.org/show_bug.cgi?id=215787
+
+Reviewed by Tim Horton.
+
+To quote Tim from r264945:
+No new tests; timing is such that I can't reproduce without inserting
+intentional delays into the main thread hops, which is further than
+I'm willing to go for a test.
+
+This is a speculative fix due to the aforementioned reproducibility issue.
+
+* WebProcess/Plugins/PDF/PDFPlugin.mm:
+(WebKit::PDFPlugin::receivedNonLinearizedPDFSentinel): Check on the main thread whenever this is called,
+  instead of when deciding to dispatch to the main thread.
+(WebKit::PDFPlugin::threadEntry): We can't do this check on the background thread when considering the dispatch
+  to the main thread, as the flag might've changed by then. Let's *just* check it on the main thread.
+(WebKit::PDFPlugin::adoptBackgroundThreadDocument): We can't do the check on the background thread when 
+(WebKit::PDFPlugin::updateScrollbars): This is where the crash itself is. All of the Obj-C code in here is
+  safe to do after destroy(), up until the very end when we get into pluginView() derefencing.
+  So it seems prudent to add another check here.
+(WebKit::PDFPlugin::documentDataDidFinishLoading): In addition to receivedNonLinearizedPDFSentinel and
+  adoptBackgroundThreadDocument, this is the final of the (3) calls that end up calling installPDFDocument,
+  so for added coverage it seems like a prudent place to add the check.
+
 2020-08-24  Wenson Hsieh  
 
 Unreviewed, fix the internal iOS 13.4 build


Modified: trunk/Source/WebKit/WebProcess/Plugins/PDF/PDFPlugin.mm (266098 => 266099)

--- trunk/Source/WebKit/WebProcess/Plugins/PDF/PDFPlugin.mm	2020-08-25 02:03:51 UTC (rev 266098)
+++ trunk/Source/WebKit/WebProcess/Plugins/PDF/PDFPlugin.mm	2020-08-25 02:44:39 UTC (rev 266099)
@@ -705,13 +705,14 @@
 {
 m_incrementalPDFLoadingEnabled = false;
 
+if (m_hasBeenDestroyed)
+return;
+
 if (!isMainThread()) {
 #if !LOG_DISABLED
 pdfLog("Disabling incremental PDF loading on background thread");
 #endif
 callOnMainThread([this, protectedThis = makeRef(*this)] {
-if (m_hasBeenDestroyed)
-return;
 receivedNonLinearizedPDFSentinel();
 });
 return;
@@ -890,8 +891,6 @@
 [m_backgroundThreadDocument preloadDataOfPagesInRange:NSMakeRange(0, 1) onQueue:firstPageQueue->dispatchQueue() completion:[, this] (NSIndexSet *) mutable {
 if (m_incrementalPDFLoadingEnabled) {
 callOnMainThread([this] {
-if (m_hasBeenDestroyed)
-return;
 adoptBackgroundThreadDocument();
 });
 } else
@@ -986,6 +985,9 @@
 
 void 

[webkit-changes] [266098] trunk/LayoutTests/imported/w3c

2020-08-24 Thread achristensen
Title: [266098] trunk/LayoutTests/imported/w3c








Revision 266098
Author achristen...@apple.com
Date 2020-08-24 19:03:51 -0700 (Mon, 24 Aug 2020)


Log Message
Update iOS layout test results after r266087
https://bugs.webkit.org/show_bug.cgi?id=215671

* web-platform-tests/fetch/content-encoding/bad-gzip-body.any.worker-expected.txt:
Apparently there's a newly passing test here, too.
I wonder why macOS isn't affected by this change.

Modified Paths

trunk/LayoutTests/imported/w3c/ChangeLog
trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/content-encoding/bad-gzip-body.any.worker-expected.txt




Diff

Modified: trunk/LayoutTests/imported/w3c/ChangeLog (266097 => 266098)

--- trunk/LayoutTests/imported/w3c/ChangeLog	2020-08-25 02:00:07 UTC (rev 266097)
+++ trunk/LayoutTests/imported/w3c/ChangeLog	2020-08-25 02:03:51 UTC (rev 266098)
@@ -1,5 +1,14 @@
 2020-08-24  Alex Christensen  
 
+Update iOS layout test results after r266087
+https://bugs.webkit.org/show_bug.cgi?id=215671
+
+* web-platform-tests/fetch/content-encoding/bad-gzip-body.any.worker-expected.txt:
+Apparently there's a newly passing test here, too.
+I wonder why macOS isn't affected by this change.
+
+2020-08-24  Alex Christensen  
+
 Implement Request/Response consuming as FormData
 https://bugs.webkit.org/show_bug.cgi?id=215671
 


Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/content-encoding/bad-gzip-body.any.worker-expected.txt (266097 => 266098)

--- trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/content-encoding/bad-gzip-body.any.worker-expected.txt	2020-08-25 02:00:07 UTC (rev 266097)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/content-encoding/bad-gzip-body.any.worker-expected.txt	2020-08-25 02:03:51 UTC (rev 266098)
@@ -2,9 +2,7 @@
 PASS Fetching a resource with bad gzip content should still resolve 
 PASS Consuming the body of a resource with bad gzip content with arrayBuffer() should reject 
 PASS Consuming the body of a resource with bad gzip content with blob() should reject 
-FAIL Consuming the body of a resource with bad gzip content with formData() should reject promise_rejects_js: function "function () { throw e }" threw object "NotSupportedError: The operation is not supported." ("NotSupportedError") expected instance of function "function TypeError() {
-[native code]
-}" ("TypeError")
+PASS Consuming the body of a resource with bad gzip content with formData() should reject 
 PASS Consuming the body of a resource with bad gzip content with json() should reject 
 PASS Consuming the body of a resource with bad gzip content with text() should reject 
 






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266097] trunk/LayoutTests

2020-08-24 Thread mmaxfield
Title: [266097] trunk/LayoutTests








Revision 266097
Author mmaxfi...@apple.com
Date 2020-08-24 19:00:07 -0700 (Mon, 24 Aug 2020)


Log Message
[macOS Big Sur] svg/W3C-I18N/tspan-direction-rtl.svg is failing


Unreviewed test gardening.

platform/mac-bigsur/svg/W3C-I18N/tspan-direction-rtl-expected.txt: Removed property svn:keywords.

Modified Paths

trunk/LayoutTests/ChangeLog


Property Changed

trunk/LayoutTests/platform/mac-bigsur/svg/W3C-I18N/tspan-direction-rtl-expected.txt




Diff

Modified: trunk/LayoutTests/ChangeLog (266096 => 266097)

--- trunk/LayoutTests/ChangeLog	2020-08-25 01:53:51 UTC (rev 266096)
+++ trunk/LayoutTests/ChangeLog	2020-08-25 02:00:07 UTC (rev 266097)
@@ -7,6 +7,15 @@
 
 * platform/mac-bigsur/svg/W3C-I18N/tspan-direction-rtl-expected.txt: Removed property svn:keywords.
 
+2020-08-24  Myles C. Maxfield  
+
+[macOS Big Sur] svg/W3C-I18N/tspan-direction-rtl.svg is failing
+
+
+Unreviewed test gardening.
+
+* platform/mac-bigsur/svg/W3C-I18N/tspan-direction-rtl-expected.txt: Removed property svn:keywords.
+
 2020-08-24  Hector Lopez  
 
 [ macOS ] Tests expectations changed as test passing but expected to fail
Index: trunk/LayoutTests/platform/mac-bigsur/svg/W3C-I18N/tspan-direction-rtl-expected.txt
===
--- trunk/LayoutTests/platform/mac-bigsur/svg/W3C-I18N/tspan-direction-rtl-expected.txt	2020-08-25 01:53:51 UTC (rev 266096)
+++ trunk/LayoutTests/platform/mac-bigsur/svg/W3C-I18N/tspan-direction-rtl-expected.txt	2020-08-25 02:00:07 UTC (rev 266097)


Property changes: trunk/LayoutTests/platform/mac-bigsur/svg/W3C-I18N/tspan-direction-rtl-expected.txt



Deleted: svn:keywords
-Author Date Id Rev URL
\ No newline at end of property




___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266096] trunk/LayoutTests/ChangeLog

2020-08-24 Thread mmaxfield
Title: [266096] trunk/LayoutTests/ChangeLog








Revision 266096
Author mmaxfi...@apple.com
Date 2020-08-24 18:53:51 -0700 (Mon, 24 Aug 2020)


Log Message
[macOS Big Sur] svg/W3C-I18N/tspan-direction-rtl.svg is failing


Unreviewed test gardening.

* platform/mac-bigsur/svg/W3C-I18N/tspan-direction-rtl-expected.txt: Removed property svn:keywords.

Modified Paths

trunk/LayoutTests/ChangeLog




Diff

Modified: trunk/LayoutTests/ChangeLog (266095 => 266096)

--- trunk/LayoutTests/ChangeLog	2020-08-25 01:51:15 UTC (rev 266095)
+++ trunk/LayoutTests/ChangeLog	2020-08-25 01:53:51 UTC (rev 266096)
@@ -1,3 +1,12 @@
+2020-08-24  Myles C. Maxfield  
+
+[macOS Big Sur] svg/W3C-I18N/tspan-direction-rtl.svg is failing
+
+
+Unreviewed test gardening.
+
+* platform/mac-bigsur/svg/W3C-I18N/tspan-direction-rtl-expected.txt: Removed property svn:keywords.
+
 2020-08-24  Hector Lopez  
 
 [ macOS ] Tests expectations changed as test passing but expected to fail






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266095] trunk/Source/JavaScriptCore

2020-08-24 Thread keith_miller
Title: [266095] trunk/Source/_javascript_Core








Revision 266095
Author keith_mil...@apple.com
Date 2020-08-24 18:51:15 -0700 (Mon, 24 Aug 2020)


Log Message
Remove MovHintRemoval phase
https://bugs.webkit.org/show_bug.cgi?id=215785

Reviewed by Saam Barati.

The MovHintRemoval phase doesn't play nicely with our OSR
Availability. Specifically, it needs to do a tricky dance where it
marks all the live ranges of the ZombieHints as not
exitOK. There's also an issue because we treated unused locals as
kill in this block, which is wrong for SSA when a MovHint is
used in another block. Since removing MovHintRemoval isn't a
performance regression, we are removing it rather than fixing bugs
related to it. Relatedly, since the only place we produce
ZombieHints is MovHintRemoval this patch also removes that node
type.

* Sources.txt:
* dfg/DFGAbstractInterpreterInlines.h:
(JSC::DFG::AbstractInterpreter::executeEffects):
* dfg/DFGClobberize.h:
(JSC::DFG::clobberize):
* dfg/DFGClobbersExitState.cpp:
(JSC::DFG::clobbersExitState):
* dfg/DFGDoesGC.cpp:
(JSC::DFG::doesGC):
* dfg/DFGFixupPhase.cpp:
(JSC::DFG::FixupPhase::fixupNode):
* dfg/DFGMayExit.cpp:
* dfg/DFGMovHintRemovalPhase.cpp: Removed.
* dfg/DFGMovHintRemovalPhase.h: Removed.
* dfg/DFGNode.h:
(JSC::DFG::Node::containsMovHint):
(JSC::DFG::Node::hasUnlinkedOperand):
* dfg/DFGNodeType.h:
* dfg/DFGOSRAvailabilityAnalysisPhase.cpp:
(JSC::DFG::LocalOSRAvailabilityCalculator::executeNode):
* dfg/DFGPhantomInsertionPhase.cpp:
* dfg/DFGPlan.cpp:
(JSC::DFG::Plan::compileInThreadImpl):
* dfg/DFGPredictionPropagationPhase.cpp:
* dfg/DFGSafeToExecute.h:
(JSC::DFG::safeToExecute):
* dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::compileMovHint):
* dfg/DFGSpeculativeJIT32_64.cpp:
(JSC::DFG::SpeculativeJIT::compile):
* dfg/DFGSpeculativeJIT64.cpp:
(JSC::DFG::SpeculativeJIT::compile):
* dfg/DFGVarargsForwardingPhase.cpp:
* ftl/FTLCapabilities.cpp:
(JSC::FTL::canCompile):
* ftl/FTLLowerDFGToB3.cpp:
(JSC::FTL::DFG::LowerDFGToB3::validateAIState):
(JSC::FTL::DFG::LowerDFGToB3::compileNode):
* runtime/OptionsList.h:

Modified Paths

trunk/Source/_javascript_Core/ChangeLog
trunk/Source/_javascript_Core/Sources.txt
trunk/Source/_javascript_Core/dfg/DFGAbstractInterpreterInlines.h
trunk/Source/_javascript_Core/dfg/DFGClobberize.h
trunk/Source/_javascript_Core/dfg/DFGClobbersExitState.cpp
trunk/Source/_javascript_Core/dfg/DFGDoesGC.cpp
trunk/Source/_javascript_Core/dfg/DFGFixupPhase.cpp
trunk/Source/_javascript_Core/dfg/DFGMayExit.cpp
trunk/Source/_javascript_Core/dfg/DFGNode.h
trunk/Source/_javascript_Core/dfg/DFGNodeType.h
trunk/Source/_javascript_Core/dfg/DFGOSRAvailabilityAnalysisPhase.cpp
trunk/Source/_javascript_Core/dfg/DFGPhantomInsertionPhase.cpp
trunk/Source/_javascript_Core/dfg/DFGPlan.cpp
trunk/Source/_javascript_Core/dfg/DFGPredictionPropagationPhase.cpp
trunk/Source/_javascript_Core/dfg/DFGSafeToExecute.h
trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp
trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT32_64.cpp
trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT64.cpp
trunk/Source/_javascript_Core/dfg/DFGVarargsForwardingPhase.cpp
trunk/Source/_javascript_Core/ftl/FTLCapabilities.cpp
trunk/Source/_javascript_Core/ftl/FTLLowerDFGToB3.cpp
trunk/Source/_javascript_Core/runtime/OptionsList.h


Removed Paths

trunk/Source/_javascript_Core/dfg/DFGMovHintRemovalPhase.cpp
trunk/Source/_javascript_Core/dfg/DFGMovHintRemovalPhase.h




Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (266094 => 266095)

--- trunk/Source/_javascript_Core/ChangeLog	2020-08-25 00:36:10 UTC (rev 266094)
+++ trunk/Source/_javascript_Core/ChangeLog	2020-08-25 01:51:15 UTC (rev 266095)
@@ -1,3 +1,61 @@
+2020-08-24  Keith Miller  
+
+Remove MovHintRemoval phase
+https://bugs.webkit.org/show_bug.cgi?id=215785
+
+Reviewed by Saam Barati.
+
+The MovHintRemoval phase doesn't play nicely with our OSR
+Availability. Specifically, it needs to do a tricky dance where it
+marks all the live ranges of the ZombieHints as not
+exitOK. There's also an issue because we treated unused locals as
+kill in this block, which is wrong for SSA when a MovHint is
+used in another block. Since removing MovHintRemoval isn't a
+performance regression, we are removing it rather than fixing bugs
+related to it. Relatedly, since the only place we produce
+ZombieHints is MovHintRemoval this patch also removes that node
+type.
+
+* Sources.txt:
+* dfg/DFGAbstractInterpreterInlines.h:
+(JSC::DFG::AbstractInterpreter::executeEffects):
+* dfg/DFGClobberize.h:
+(JSC::DFG::clobberize):
+* dfg/DFGClobbersExitState.cpp:
+(JSC::DFG::clobbersExitState):
+* dfg/DFGDoesGC.cpp:
+(JSC::DFG::doesGC):
+* dfg/DFGFixupPhase.cpp:
+(JSC::DFG::FixupPhase::fixupNode):
+* dfg/DFGMayExit.cpp:
+* 

[webkit-changes] [266094] trunk/LayoutTests

2020-08-24 Thread hector_i_lopez
Title: [266094] trunk/LayoutTests








Revision 266094
Author hector_i_lo...@apple.com
Date 2020-08-24 17:36:10 -0700 (Mon, 24 Aug 2020)


Log Message
[ macOS ] Tests expectations changed as test passing but expected to fail
https://bugs.webkit.org/show_bug.cgi?id=215786

Unreviewed test gardening.

* TestExpectations:
* platform/ios-wk2/TestExpectations:
* platform/mac-wk2/TestExpectations:
* platform/mac/TestExpectations:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/TestExpectations
trunk/LayoutTests/platform/ios-wk2/TestExpectations
trunk/LayoutTests/platform/mac/TestExpectations
trunk/LayoutTests/platform/mac-wk2/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (266093 => 266094)

--- trunk/LayoutTests/ChangeLog	2020-08-25 00:26:09 UTC (rev 266093)
+++ trunk/LayoutTests/ChangeLog	2020-08-25 00:36:10 UTC (rev 266094)
@@ -1,3 +1,15 @@
+2020-08-24  Hector Lopez  
+
+[ macOS ] Tests expectations changed as test passing but expected to fail
+https://bugs.webkit.org/show_bug.cgi?id=215786
+
+Unreviewed test gardening.
+
+* TestExpectations:
+* platform/ios-wk2/TestExpectations:
+* platform/mac-wk2/TestExpectations:
+* platform/mac/TestExpectations:
+
 2020-08-24  Karl Rackler  
 
 rdar://67706887 (REGRESSION (r264950): [ iOS 13 WK2 ] imported/w3c/web-platform-tests/css/css-cascade/all-prop-initial-xml.html is a constant failure)


Modified: trunk/LayoutTests/TestExpectations (266093 => 266094)

--- trunk/LayoutTests/TestExpectations	2020-08-25 00:26:09 UTC (rev 266093)
+++ trunk/LayoutTests/TestExpectations	2020-08-25 00:36:10 UTC (rev 266094)
@@ -1192,8 +1192,6 @@
 imported/w3c/web-platform-tests/css/css-grid/abspos/orthogonal-positioned-grid-descendants-010.html [ Pass Failure ]
 imported/w3c/web-platform-tests/css/css-grid/abspos/orthogonal-positioned-grid-descendants-013.html [ Pass Failure ]
 imported/w3c/web-platform-tests/css/css-grid/abspos/orthogonal-positioned-grid-descendants-015.html [ Pass Failure ]
-webkit.org/b/191367 imported/w3c/web-platform-tests/css/css-grid/grid-model/grid-container-ignores-first-letter-002.html [ ImageOnlyFailure ]
-webkit.org/b/209461 imported/w3c/web-platform-tests/css/css-grid/grid-items/grid-item-dynamic-min-contribution-001.html [ Failure ]
 webkit.org/b/209461 imported/w3c/web-platform-tests/css/css-grid/grid-items/grid-items-percentage-margins-003.html [ ImageOnlyFailure ]
 webkit.org/b/209461 imported/w3c/web-platform-tests/css/css-grid/grid-items/grid-items-percentage-margins-004.html [ ImageOnlyFailure ]
 webkit.org/b/209461 imported/w3c/web-platform-tests/css/css-grid/grid-items/grid-items-percentage-margins-005.html [ ImageOnlyFailure ]
@@ -2020,7 +2018,6 @@
 
 imported/w3c/web-platform-tests/css/css-content/element-replacement-on-replaced-element.tentative.html [ ImageOnlyFailure ]
 imported/w3c/web-platform-tests/css/css-content/quotes-001.html [ ImageOnlyFailure ]
-imported/w3c/web-platform-tests/css/css-content/quotes-005.html [ ImageOnlyFailure ]
 imported/w3c/web-platform-tests/css/css-content/quotes-006.html [ ImageOnlyFailure ]
 imported/w3c/web-platform-tests/css/css-content/quotes-007.html [ ImageOnlyFailure ]
 imported/w3c/web-platform-tests/css/css-content/quotes-012.html [ ImageOnlyFailure ]
@@ -2027,14 +2024,11 @@
 imported/w3c/web-platform-tests/css/css-content/quotes-013.html [ ImageOnlyFailure ]
 imported/w3c/web-platform-tests/css/css-content/quotes-014.html [ ImageOnlyFailure ]
 imported/w3c/web-platform-tests/css/css-content/quotes-017.html [ ImageOnlyFailure ]
-imported/w3c/web-platform-tests/css/css-content/quotes-018.html [ ImageOnlyFailure ]
 imported/w3c/web-platform-tests/css/css-content/quotes-019.html [ ImageOnlyFailure ]
 imported/w3c/web-platform-tests/css/css-content/quotes-020.html [ ImageOnlyFailure ]
 imported/w3c/web-platform-tests/css/css-content/quotes-021.html [ ImageOnlyFailure ]
 imported/w3c/web-platform-tests/css/css-content/quotes-022.html [ ImageOnlyFailure ]
 imported/w3c/web-platform-tests/css/css-content/quotes-023.html [ ImageOnlyFailure ]
-imported/w3c/web-platform-tests/css/css-content/quotes-025.html [ ImageOnlyFailure ]
-imported/w3c/web-platform-tests/css/css-content/quotes-027.html [ ImageOnlyFailure ]
 imported/w3c/web-platform-tests/css/css-content/quotes-033.html [ ImageOnlyFailure ]
 
 # Need to re-import canvas tests
@@ -3900,7 +3894,6 @@
 
 webkit.org/b/136754 imported/w3c/web-platform-tests/css/css-flexbox/flexbox_stf-table-singleline-2.html [ ImageOnlyFailure ]
 webkit.org/b/145176 imported/w3c/web-platform-tests/css/css-flexbox/flexbox_align-items-stretch-3.html [ ImageOnlyFailure ]
-webkit.org/b/169007 imported/w3c/web-platform-tests/css/css-flexbox/flex-item-contains-strict.html [ Failure ]
 webkit.org/b/206767 imported/w3c/web-platform-tests/css/css-flexbox/flexbox-gap-position-absolute.html [ ImageOnlyFailure ]
 webkit.org/b/206767 

[webkit-changes] [266093] branches/safari-610.1.28.0-branch/Source

2020-08-24 Thread alancoon
Title: [266093] branches/safari-610.1.28.0-branch/Source








Revision 266093
Author alanc...@apple.com
Date 2020-08-24 17:26:09 -0700 (Mon, 24 Aug 2020)


Log Message
Versioning.

WebKit-7610.1.28.0.3

Modified Paths

branches/safari-610.1.28.0-branch/Source/_javascript_Core/Configurations/Version.xcconfig
branches/safari-610.1.28.0-branch/Source/ThirdParty/ANGLE/Configurations/Version.xcconfig
branches/safari-610.1.28.0-branch/Source/ThirdParty/libwebrtc/Configurations/Version.xcconfig
branches/safari-610.1.28.0-branch/Source/WebCore/Configurations/Version.xcconfig
branches/safari-610.1.28.0-branch/Source/WebCore/PAL/Configurations/Version.xcconfig
branches/safari-610.1.28.0-branch/Source/WebInspectorUI/Configurations/Version.xcconfig
branches/safari-610.1.28.0-branch/Source/WebKit/Configurations/Version.xcconfig
branches/safari-610.1.28.0-branch/Source/WebKitLegacy/mac/Configurations/Version.xcconfig




Diff

Modified: branches/safari-610.1.28.0-branch/Source/_javascript_Core/Configurations/Version.xcconfig (266092 => 266093)

--- branches/safari-610.1.28.0-branch/Source/_javascript_Core/Configurations/Version.xcconfig	2020-08-25 00:14:28 UTC (rev 266092)
+++ branches/safari-610.1.28.0-branch/Source/_javascript_Core/Configurations/Version.xcconfig	2020-08-25 00:26:09 UTC (rev 266093)
@@ -25,7 +25,7 @@
 MINOR_VERSION = 1;
 TINY_VERSION = 28;
 MICRO_VERSION = 0;
-NANO_VERSION = 2;
+NANO_VERSION = 3;
 FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION).$(MICRO_VERSION).$(NANO_VERSION);
 
 // The bundle version and short version string are set based on the current build configuration, see below.


Modified: branches/safari-610.1.28.0-branch/Source/ThirdParty/ANGLE/Configurations/Version.xcconfig (266092 => 266093)

--- branches/safari-610.1.28.0-branch/Source/ThirdParty/ANGLE/Configurations/Version.xcconfig	2020-08-25 00:14:28 UTC (rev 266092)
+++ branches/safari-610.1.28.0-branch/Source/ThirdParty/ANGLE/Configurations/Version.xcconfig	2020-08-25 00:26:09 UTC (rev 266093)
@@ -25,7 +25,7 @@
 MINOR_VERSION = 1;
 TINY_VERSION = 28;
 MICRO_VERSION = 0;
-NANO_VERSION = 2;
+NANO_VERSION = 3;
 FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION).$(MICRO_VERSION).$(NANO_VERSION);
 
 // The bundle version and short version string are set based on the current build configuration, see below.


Modified: branches/safari-610.1.28.0-branch/Source/ThirdParty/libwebrtc/Configurations/Version.xcconfig (266092 => 266093)

--- branches/safari-610.1.28.0-branch/Source/ThirdParty/libwebrtc/Configurations/Version.xcconfig	2020-08-25 00:14:28 UTC (rev 266092)
+++ branches/safari-610.1.28.0-branch/Source/ThirdParty/libwebrtc/Configurations/Version.xcconfig	2020-08-25 00:26:09 UTC (rev 266093)
@@ -25,7 +25,7 @@
 MINOR_VERSION = 1;
 TINY_VERSION = 28;
 MICRO_VERSION = 0;
-NANO_VERSION = 2;
+NANO_VERSION = 3;
 FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION).$(MICRO_VERSION).$(NANO_VERSION);
 
 // The bundle version and short version string are set based on the current build configuration, see below.


Modified: branches/safari-610.1.28.0-branch/Source/WebCore/Configurations/Version.xcconfig (266092 => 266093)

--- branches/safari-610.1.28.0-branch/Source/WebCore/Configurations/Version.xcconfig	2020-08-25 00:14:28 UTC (rev 266092)
+++ branches/safari-610.1.28.0-branch/Source/WebCore/Configurations/Version.xcconfig	2020-08-25 00:26:09 UTC (rev 266093)
@@ -25,7 +25,7 @@
 MINOR_VERSION = 1;
 TINY_VERSION = 28;
 MICRO_VERSION = 0;
-NANO_VERSION = 2;
+NANO_VERSION = 3;
 FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION).$(MICRO_VERSION).$(NANO_VERSION);
 
 // The bundle version and short version string are set based on the current build configuration, see below.


Modified: branches/safari-610.1.28.0-branch/Source/WebCore/PAL/Configurations/Version.xcconfig (266092 => 266093)

--- branches/safari-610.1.28.0-branch/Source/WebCore/PAL/Configurations/Version.xcconfig	2020-08-25 00:14:28 UTC (rev 266092)
+++ branches/safari-610.1.28.0-branch/Source/WebCore/PAL/Configurations/Version.xcconfig	2020-08-25 00:26:09 UTC (rev 266093)
@@ -25,7 +25,7 @@
 MINOR_VERSION = 1;
 TINY_VERSION = 28;
 MICRO_VERSION = 0;
-NANO_VERSION = 2;
+NANO_VERSION = 3;
 FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION).$(MICRO_VERSION).$(NANO_VERSION);
 
 // The bundle version and short version string are set based on the current build configuration, see below.


Modified: branches/safari-610.1.28.0-branch/Source/WebInspectorUI/Configurations/Version.xcconfig (266092 => 266093)

--- branches/safari-610.1.28.0-branch/Source/WebInspectorUI/Configurations/Version.xcconfig	2020-08-25 00:14:28 UTC (rev 266092)
+++ branches/safari-610.1.28.0-branch/Source/WebInspectorUI/Configurations/Version.xcconfig	2020-08-25 00:26:09 UTC (rev 266093)
@@ -2,7 +2,7 @@
 MINOR_VERSION = 1;
 TINY_VERSION = 28;
 MICRO_VERSION = 0;
-NANO_VERSION = 2;
+NANO_VERSION = 3;
 FULL_VERSION = 

[webkit-changes] [266092] trunk/LayoutTests

2020-08-24 Thread rackler
Title: [266092] trunk/LayoutTests








Revision 266092
Author rack...@apple.com
Date 2020-08-24 17:14:28 -0700 (Mon, 24 Aug 2020)


Log Message
rdar://67706887 (REGRESSION (r264950): [ iOS 13 WK2 ] imported/w3c/web-platform-tests/css/css-cascade/all-prop-initial-xml.html is a constant failure)

Unreviewed test gardening.

* platform/ios-13/TestExpectations:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/ios-13/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (266091 => 266092)

--- trunk/LayoutTests/ChangeLog	2020-08-25 00:05:58 UTC (rev 266091)
+++ trunk/LayoutTests/ChangeLog	2020-08-25 00:14:28 UTC (rev 266092)
@@ -1,3 +1,11 @@
+2020-08-24  Karl Rackler  
+
+rdar://67706887 (REGRESSION (r264950): [ iOS 13 WK2 ] imported/w3c/web-platform-tests/css/css-cascade/all-prop-initial-xml.html is a constant failure)
+
+Unreviewed test gardening.
+
+* platform/ios-13/TestExpectations:
+
 2020-08-24  Hector Lopez  
 
 [ iOS wk2 Debug ] imported/w3c/web-platform-tests/html/semantics/scripting-1/the-script-element/moving-between-documents/before-prepare-iframe-fetch-error-external-module.html is a flaky failure


Modified: trunk/LayoutTests/platform/ios-13/TestExpectations (266091 => 266092)

--- trunk/LayoutTests/platform/ios-13/TestExpectations	2020-08-25 00:05:58 UTC (rev 266091)
+++ trunk/LayoutTests/platform/ios-13/TestExpectations	2020-08-25 00:14:28 UTC (rev 266092)
@@ -10,3 +10,6 @@
 http/tests/resourceLoadStatistics/cname-cloaking-top-no-cname-sub-1p-cname.html [ Skip ]
 http/tests/resourceLoadStatistics/cname-cloaking-top-no-cname-sub-3p-cname.html [ Skip ]
 http/tests/resourceLoadStatistics/cname-cloaking-top-no-cname-sub-no-cname.html [ Skip ]
+
+# rdar://67706887 (REGRESSION (r264950): [ iOS 13 WK2 ] imported/w3c/web-platform-tests/css/css-cascade/all-prop-initial-xml.html is a constant failure)
+imported/w3c/web-platform-tests/css/css-cascade/all-prop-initial-xml.html [ Failure ]






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266091] trunk/JSTests

2020-08-24 Thread keith_miller
Title: [266091] trunk/JSTests








Revision 266091
Author keith_mil...@apple.com
Date 2020-08-24 17:05:58 -0700 (Mon, 24 Aug 2020)


Log Message
Update test262 (mid Aug 2020 edition)
https://bugs.webkit.org/show_bug.cgi?id=215784

Rubber-stamped by Saam Barati and Yusuke Suzuki.

* test262/expectations.yaml:
* test262/latest-changes-summary.txt:
* test262/test/built-ins/String/prototype/split/limit-touint32-error.js: Added.
(ExpectedError):
(ExpectedError.prototype.toString):
(nonStringableSeparator.Symbol.toPrimitive):
(nonStringableSeparator.toString):
(nonStringableSeparator.valueOf):
(nonNumberableLimit.Symbol.toPrimitive):
* test262/test/built-ins/String/prototype/split/separator-tostring-error.js: Added.
(ExpectedError):
(ExpectedError.prototype.toString):
(nonStringableSeparator.toString):
* test262/test/built-ins/String/prototype/split/separator-undef-limit-zero.js:
* test262/test/built-ins/String/prototype/split/this-value-tostring-error.js: Added.
(ExpectedError):
(ExpectedError.prototype.toString):
(nonStringableReceiver.toString):
(splitter.Symbol.split):
(catch):
(nonStringableSeparator.Symbol.toPrimitive):
(nonStringableSeparator.toString):
(nonStringableSeparator.valueOf):
* test262/test/language/module-code/export-default-asyncfunction-declaration-binding-exists.js: Added.
(A):
(export.default.async A):
* test262/test/language/module-code/export-default-asyncfunction-declaration-binding.js: Added.
(export.default.async A):
* test262/test/language/module-code/export-default-asyncgenerator-declaration-binding-exists.js: Added.
(AG):
(export.default.async AG):
* test262/test/language/module-code/export-default-asyncgenerator-declaration-binding.js: Added.
(export.default.async AG):
* test262/test/language/module-code/export-default-function-declaration-binding-exists.js: Added.
(F):
(export.default.F):
* test262/test/language/module-code/export-default-function-declaration-binding.js: Added.
(export.default.F):
* test262/test/language/module-code/export-default-generator-declaration-binding-exists.js: Added.
(G):
(export.default.G):
* test262/test/language/module-code/export-default-generator-declaration-binding.js: Added.
(export.default.G):
* test262/test262-Revision.txt:

Modified Paths

trunk/JSTests/ChangeLog
trunk/JSTests/test262/expectations.yaml
trunk/JSTests/test262/latest-changes-summary.txt
trunk/JSTests/test262/test/built-ins/String/prototype/split/separator-undef-limit-zero.js
trunk/JSTests/test262/test262-Revision.txt


Added Paths

trunk/JSTests/test262/test/built-ins/String/prototype/split/limit-touint32-error.js
trunk/JSTests/test262/test/built-ins/String/prototype/split/separator-tostring-error.js
trunk/JSTests/test262/test/built-ins/String/prototype/split/this-value-tostring-error.js
trunk/JSTests/test262/test/language/module-code/export-default-asyncfunction-declaration-binding-exists.js
trunk/JSTests/test262/test/language/module-code/export-default-asyncfunction-declaration-binding.js
trunk/JSTests/test262/test/language/module-code/export-default-asyncgenerator-declaration-binding-exists.js
trunk/JSTests/test262/test/language/module-code/export-default-asyncgenerator-declaration-binding.js
trunk/JSTests/test262/test/language/module-code/export-default-function-declaration-binding-exists.js
trunk/JSTests/test262/test/language/module-code/export-default-function-declaration-binding.js
trunk/JSTests/test262/test/language/module-code/export-default-generator-declaration-binding-exists.js
trunk/JSTests/test262/test/language/module-code/export-default-generator-declaration-binding.js




Diff

Modified: trunk/JSTests/ChangeLog (266090 => 266091)

--- trunk/JSTests/ChangeLog	2020-08-24 22:48:19 UTC (rev 266090)
+++ trunk/JSTests/ChangeLog	2020-08-25 00:05:58 UTC (rev 266091)
@@ -1,3 +1,55 @@
+2020-08-24  Keith Miller  
+
+Update test262 (mid Aug 2020 edition)
+https://bugs.webkit.org/show_bug.cgi?id=215784
+
+Rubber-stamped by Saam Barati and Yusuke Suzuki.
+
+* test262/expectations.yaml:
+* test262/latest-changes-summary.txt:
+* test262/test/built-ins/String/prototype/split/limit-touint32-error.js: Added.
+(ExpectedError):
+(ExpectedError.prototype.toString):
+(nonStringableSeparator.Symbol.toPrimitive):
+(nonStringableSeparator.toString):
+(nonStringableSeparator.valueOf):
+(nonNumberableLimit.Symbol.toPrimitive):
+* test262/test/built-ins/String/prototype/split/separator-tostring-error.js: Added.
+(ExpectedError):
+(ExpectedError.prototype.toString):
+(nonStringableSeparator.toString):
+* test262/test/built-ins/String/prototype/split/separator-undef-limit-zero.js:
+* test262/test/built-ins/String/prototype/split/this-value-tostring-error.js: Added.
+(ExpectedError):
+(ExpectedError.prototype.toString):
+(nonStringableReceiver.toString):
+(splitter.Symbol.split):
+(catch):
+

[webkit-changes] [266090] trunk

2020-08-24 Thread wenson_hsieh
Title: [266090] trunk








Revision 266090
Author wenson_hs...@apple.com
Date 2020-08-24 15:48:19 -0700 (Mon, 24 Aug 2020)


Log Message
Unreviewed, fix the internal iOS 13.4 build

Source/WebKit:

The -_selectionClipRect SPI method isn't declared in any private headers on iOS 13.4 and prior. This isn't a
problem when building with a non-internal SDK since the SPI is declared in the `!USE(APPLE_INTERNAL_SDK)`
section in `UIKitSPI.h`, but on internal builds of iOS 13.4, this declaration is absent from both the internal
SDK and `UIKitSPI.h`.

To fix the build, simply declare it in the IPI section — once we stop supporting iOS 13.4 as a build target,
this can be moved back into the non-internal section only.

* Platform/spi/ios/UIKitSPI.h:

Tools:

See Source/WebKit/ChangeLog.

* WebKitTestRunner/ios/UIScriptControllerIOS.mm:
(WTR::clipSelectionViewRectToContentView):

Modified Paths

trunk/Source/WebKit/ChangeLog
trunk/Source/WebKit/Platform/spi/ios/UIKitSPI.h
trunk/Tools/ChangeLog
trunk/Tools/WebKitTestRunner/ios/UIScriptControllerIOS.mm




Diff

Modified: trunk/Source/WebKit/ChangeLog (266089 => 266090)

--- trunk/Source/WebKit/ChangeLog	2020-08-24 22:46:50 UTC (rev 266089)
+++ trunk/Source/WebKit/ChangeLog	2020-08-24 22:48:19 UTC (rev 266090)
@@ -1,3 +1,17 @@
+2020-08-24  Wenson Hsieh  
+
+Unreviewed, fix the internal iOS 13.4 build
+
+The -_selectionClipRect SPI method isn't declared in any private headers on iOS 13.4 and prior. This isn't a
+problem when building with a non-internal SDK since the SPI is declared in the `!USE(APPLE_INTERNAL_SDK)`
+section in `UIKitSPI.h`, but on internal builds of iOS 13.4, this declaration is absent from both the internal
+SDK and `UIKitSPI.h`.
+
+To fix the build, simply declare it in the IPI section — once we stop supporting iOS 13.4 as a build target,
+this can be moved back into the non-internal section only.
+
+* Platform/spi/ios/UIKitSPI.h:
+
 2020-08-24  Alex Christensen  
 
 Implement Request/Response consuming as FormData


Modified: trunk/Source/WebKit/Platform/spi/ios/UIKitSPI.h (266089 => 266090)

--- trunk/Source/WebKit/Platform/spi/ios/UIKitSPI.h	2020-08-24 22:46:50 UTC (rev 266089)
+++ trunk/Source/WebKit/Platform/spi/ios/UIKitSPI.h	2020-08-24 22:48:19 UTC (rev 266090)
@@ -487,7 +487,6 @@
 - (void)modifierFlagsDidChangeFrom:(UIKeyModifierFlags)oldFlags to:(UIKeyModifierFlags)newFlags;
 #endif
 @property (nonatomic) UITextGranularity selectionGranularity;
-@property (nonatomic, readonly) CGRect _selectionClipRect;
 @required
 - (BOOL)hasContent;
 - (BOOL)hasSelection;
@@ -1374,6 +1373,11 @@
 @end
 #endif
 
+@protocol UITextInputInternal 
+@optional
+@property (nonatomic, readonly) CGRect _selectionClipRect;
+@end
+
 @interface UIDevice ()
 @property (nonatomic, setter=_setBacklightLevel:) float _backlightLevel;
 @end


Modified: trunk/Tools/ChangeLog (266089 => 266090)

--- trunk/Tools/ChangeLog	2020-08-24 22:46:50 UTC (rev 266089)
+++ trunk/Tools/ChangeLog	2020-08-24 22:48:19 UTC (rev 266090)
@@ -1,3 +1,12 @@
+2020-08-24  Wenson Hsieh  
+
+Unreviewed, fix the internal iOS 13.4 build
+
+See Source/WebKit/ChangeLog.
+
+* WebKitTestRunner/ios/UIScriptControllerIOS.mm:
+(WTR::clipSelectionViewRectToContentView):
+
 2020-08-24  Aakash Jain  
 
 [ews] enable email notifications to patch authors for build or layout test failures on their patch


Modified: trunk/Tools/WebKitTestRunner/ios/UIScriptControllerIOS.mm (266089 => 266090)

--- trunk/Tools/WebKitTestRunner/ios/UIScriptControllerIOS.mm	2020-08-24 22:46:50 UTC (rev 266089)
+++ trunk/Tools/WebKitTestRunner/ios/UIScriptControllerIOS.mm	2020-08-24 22:48:19 UTC (rev 266090)
@@ -793,9 +793,9 @@
 static void clipSelectionViewRectToContentView(CGRect& rect, UIView *contentView)
 {
 rect = CGRectIntersection(contentView.bounds, rect);
-// The content view (a WKContentView in WebKit) is expected to implement the optional UITextInputPrivate method -_selectionClipRect.
+// The content view (a WKContentView in WebKit) is expected to implement the optional text input method -_selectionClipRect.
 ASSERT([contentView respondsToSelector:@selector(_selectionClipRect)]);
-auto selectionClipRect = [(UIView  *)contentView _selectionClipRect];
+auto selectionClipRect = [(UIView  *)contentView _selectionClipRect];
 if (!CGRectIsNull(selectionClipRect))
 rect = CGRectIntersection(selectionClipRect, rect);
 }






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266089] trunk/Tools

2020-08-24 Thread aakash_jain
Title: [266089] trunk/Tools








Revision 266089
Author aakash_j...@apple.com
Date 2020-08-24 15:46:50 -0700 (Mon, 24 Aug 2020)


Log Message
[ews] enable email notifications to patch authors for build or layout test failures on their patch
https://bugs.webkit.org/show_bug.cgi?id=215776

Reviewed by Jonathan Bedard.

* BuildSlaveSupport/ews-build/steps.py:
(AnalyzeCompileWebKitResults.analyzeResults): Send email notification to patch authors for build failure due to their patch.
(AnalyzeLayoutTestsResults.report_failure): Send email notification to patch authors for layout test failure.

Modified Paths

trunk/Tools/BuildSlaveSupport/ews-build/steps.py
trunk/Tools/ChangeLog




Diff

Modified: trunk/Tools/BuildSlaveSupport/ews-build/steps.py (266088 => 266089)

--- trunk/Tools/BuildSlaveSupport/ews-build/steps.py	2020-08-24 22:45:09 UTC (rev 266088)
+++ trunk/Tools/BuildSlaveSupport/ews-build/steps.py	2020-08-24 22:46:50 UTC (rev 266089)
@@ -1419,6 +1419,8 @@
 self.build.results = FAILURE
 patch_id = self.getProperty('patch_id', '')
 message = 'Patch {} does not build'.format(patch_id)
+self.send_email_for_new_build_failure()
+
 self.descriptionDone = message
 if self.getProperty('buildername', '').lower() == 'commit-queue':
 self.setProperty('bugzilla_comment_text', message)
@@ -2071,6 +2073,7 @@
 if len(new_failures) > self.NUM_FAILURES_TO_DISPLAY:
 message += ' ...'
 self.descriptionDone = message
+self.send_email_for_new_test_failures(new_failures)
 
 if self.getProperty('buildername', '').lower() == 'commit-queue':
 self.setProperty('bugzilla_comment_text', message)


Modified: trunk/Tools/ChangeLog (266088 => 266089)

--- trunk/Tools/ChangeLog	2020-08-24 22:45:09 UTC (rev 266088)
+++ trunk/Tools/ChangeLog	2020-08-24 22:46:50 UTC (rev 266089)
@@ -1,5 +1,16 @@
 2020-08-24  Aakash Jain  
 
+[ews] enable email notifications to patch authors for build or layout test failures on their patch
+https://bugs.webkit.org/show_bug.cgi?id=215776
+
+Reviewed by Jonathan Bedard.
+
+* BuildSlaveSupport/ews-build/steps.py:
+(AnalyzeCompileWebKitResults.analyzeResults): Send email notification to patch authors for build failure due to their patch.
+(AnalyzeLayoutTestsResults.report_failure): Send email notification to patch authors for layout test failure.
+
+2020-08-24  Aakash Jain  
+
 [ews] set references header in email so as to group similar emails together
 https://bugs.webkit.org/show_bug.cgi?id=215777
 






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266088] trunk/Tools

2020-08-24 Thread aakash_jain
Title: [266088] trunk/Tools








Revision 266088
Author aakash_j...@apple.com
Date 2020-08-24 15:45:09 -0700 (Mon, 24 Aug 2020)


Log Message
[ews] set references header in email so as to group similar emails together
https://bugs.webkit.org/show_bug.cgi?id=215777

Reviewed by Jonathan Bedard.

* BuildSlaveSupport/ews-build/send_email.py:
(send_email): Add support for setting references header.
(send_email_to_patch_author):
(send_email_to_bot_watchers):
* BuildSlaveSupport/ews-build/steps.py:
(AnalyzeCompileWebKitResults.send_email_for_new_build_failure): Set references header appropriately.
(ReRunWebKitTests.send_email_for_flaky_failure): Ditto.
(AnalyzeLayoutTestsResults.send_email_for_flaky_failure): Ditto.
(AnalyzeLayoutTestsResults.send_email_for_pre_existing_failure): Ditto.
(AnalyzeLayoutTestsResults.send_email_for_new_test_failures): Ditto.

Modified Paths

trunk/Tools/BuildSlaveSupport/ews-build/send_email.py
trunk/Tools/BuildSlaveSupport/ews-build/steps.py
trunk/Tools/ChangeLog




Diff

Modified: trunk/Tools/BuildSlaveSupport/ews-build/send_email.py (266087 => 266088)

--- trunk/Tools/BuildSlaveSupport/ews-build/send_email.py	2020-08-24 22:36:02 UTC (rev 266087)
+++ trunk/Tools/BuildSlaveSupport/ews-build/send_email.py	2020-08-24 22:45:09 UTC (rev 266088)
@@ -42,7 +42,7 @@
 return []
 
 
-def send_email(to_emails, subject, text):
+def send_email(to_emails, subject, text, reference=''):
 if is_test_mode_enabled:
 return
 if not to_emails:
@@ -59,6 +59,9 @@
 msg['From'] = FROM_EMAIL
 msg['To'] = ', '.join(to_emails)
 msg['Subject'] = subject
+msg.add_header('reply-to', 'aakash_j...@apple.com')
+if reference:
+msg.add_header('references', '{}@webkit.org'.format(reference))
 
 server = smtplib.SMTP(SERVER)
 server.sendmail(FROM_EMAIL, to_emails, msg.as_string())
@@ -65,14 +68,14 @@
 server.quit()
 
 
-def send_email_to_patch_author(author_email, subject, text):
+def send_email_to_patch_author(author_email, subject, text, reference=''):
 if not author_email:
 return
 if author_email in get_email_ids('EMAIL_IDS_TO_UNSUBSCRIBE'):
 print('email {} is in unsubscribe list, skipping email'.format(author_email))
 return
-send_email([author_email], subject, text)
+send_email([author_email], subject, text, reference)
 
 
-def send_email_to_bot_watchers(subject, text):
-send_email(get_email_ids('BOT_WATCHERS_EMAILS'), subject, text)
+def send_email_to_bot_watchers(subject, text, reference=''):
+send_email(get_email_ids('BOT_WATCHERS_EMAILS'), subject, text, reference)


Modified: trunk/Tools/BuildSlaveSupport/ews-build/steps.py (266087 => 266088)

--- trunk/Tools/BuildSlaveSupport/ews-build/steps.py	2020-08-24 22:36:02 UTC (rev 266087)
+++ trunk/Tools/BuildSlaveSupport/ews-build/steps.py	2020-08-24 22:45:09 UTC (rev 266088)
@@ -1490,7 +1490,7 @@
 logs = logs.replace('&', '').replace('<', '').replace('>', '')
 email_text += u'\n\nError lines:\n\n{}'.format(logs)
 email_text += '\n\nTo unsubscrible from these notifications or to provide any feedback please email aakash_j...@apple.com'
-send_email_to_patch_author(patch_author, email_subject, email_text)
+send_email_to_patch_author(patch_author, email_subject, email_text, patch_id)
 except Exception as e:
 print('Error in sending email for new build failure: {}'.format(e))
 
@@ -2027,7 +2027,7 @@
 email_subject = u'Flaky test: {}'.format(test_name)
 email_text = 'Test {} flaked in {}\n\nBuilder: {}'.format(test_name, build_url, builder_name)
 email_text = 'Flaky test: {}\n\nBuild: {}\n\nBuilder: {}\n\nWorker: {}\n\nHistory: {}'.format(test_name, build_url, builder_name, worker_name, history_url)
-send_email_to_bot_watchers(email_subject, email_text)
+send_email_to_bot_watchers(email_subject, email_text, 'flaky-{}'.format(test_name))
 except Exception as e:
 # Catching all exceptions here to ensure that failure to send email doesn't impact the build
 print('Error in sending email for flaky failures: {}'.format(e))
@@ -2125,7 +2125,7 @@
 
 email_subject = u'Flaky test: {}'.format(test_name)
 email_text = 'Flaky test: {}\n\nBuild: {}\n\nBuilder: {}\n\nWorker: {}\n\nHistory: {}'.format(test_name, build_url, builder_name, worker_name, history_url)
-send_email_to_bot_watchers(email_subject, email_text)
+send_email_to_bot_watchers(email_subject, email_text, 'flaky-{}'.format(test_name))
 except Exception as e:
 print('Error in sending email for flaky failure: {}'.format(e))
 
@@ -2138,7 +2138,7 @@
 
 email_subject = u'Pre-existing test failure: {}'.format(test_name)
 email_text = 'Test {} failed on clean tree run in {}.\n\nBuilder: {}\n\nWorker: {}\n\nHistory: {}'.format(test_name, 

[webkit-changes] [266087] trunk

2020-08-24 Thread commit-queue
Title: [266087] trunk








Revision 266087
Author commit-qu...@webkit.org
Date 2020-08-24 15:36:02 -0700 (Mon, 24 Aug 2020)


Log Message
Implement Request/Response consuming as FormData
https://bugs.webkit.org/show_bug.cgi?id=215671

Patch by Alex Christensen  on 2020-08-24
Reviewed by Darin Adler.

LayoutTests/imported/w3c:

* web-platform-tests/fetch/api/abort/general.any-expected.txt:
* web-platform-tests/fetch/api/abort/general.any.worker-expected.txt:
* web-platform-tests/fetch/api/request/request-consume-empty-expected.txt:
This remaining failing test now fails similarly in all browsers.
* web-platform-tests/fetch/api/request/request-consume-expected.txt:
* web-platform-tests/fetch/api/request/request-init-002-expected.txt:
* web-platform-tests/fetch/api/response/response-consume-empty-expected.txt:
This remaining failing test now fails similarly in all browsers.
* web-platform-tests/fetch/api/response/response-consume-expected.txt:
* web-platform-tests/fetch/api/response/response-error-from-stream-expected.txt:
This change makes the formData failures in this file look like all the other failures in this file,
which should be fixed together in a separate patch.
* web-platform-tests/fetch/api/response/response-init-002-expected.txt:
* web-platform-tests/url/urlencoded-parser.any-expected.txt:
* web-platform-tests/url/urlencoded-parser.any.worker-expected.txt:
* web-platform-tests/service-workers/service-worker/fetch-event-respond-with-custom-response.https-expected.txt:

Source/WebCore:

Covered by many newly passing WPT tests, for most of which Safari was the only failing browser.

* Modules/fetch/FetchBody.cpp:
(WebCore::FetchBody::formData):
(WebCore::FetchBody::consume):
(WebCore::FetchBody::consumeFormData):
* Modules/fetch/FetchBody.h:
* Modules/fetch/FetchBodyConsumer.cpp:
(WebCore::formDataFromData):
(WebCore::resolveWithTypeAndData):
(WebCore::FetchBodyConsumer::resolve):
* Modules/fetch/FetchBodyConsumer.h:

Source/WebKit:

* WebProcess/Storage/WebServiceWorkerFetchTaskClient.cpp:
(WebKit::WebServiceWorkerFetchTaskClient::didReceiveFormDataAndFinish):
Add a fast path that allows non-blob FormData responses from service workers to not hang.
This part is covered by this layout test:
imported/w3c/web-platform-tests/service-workers/service-worker/fetch-event-respond-with-custom-response.https.html

Source/WTF:

In order to be compatible with other browsers, we need a verson of String::fromUTF8 that
uses U8_NEXT_OR_FFFD instead of U8_NEXT, but changing that across the board will break other things.
Leave everything else as it is, use templates and constexpr to not add any branches, but add
String::fromUTF8ReplacingInvalidSequences to allow me to make our FormData consuming compatible with other browsers.

* wtf/text/WTFString.cpp:
(WTF::fromUTF8Helper):
(WTF::String::fromUTF8):
(WTF::String::fromUTF8ReplacingInvalidSequences):
* wtf/text/WTFString.h:
* wtf/unicode/UTF8Conversion.cpp:
(WTF::Unicode::convertUTF8ToUTF16Impl):
(WTF::Unicode::convertUTF8ToUTF16):
(WTF::Unicode::convertUTF8ToUTF16ReplacingInvalidSequences):
* wtf/unicode/UTF8Conversion.h:

Modified Paths

trunk/LayoutTests/imported/w3c/ChangeLog
trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/abort/general.any-expected.txt
trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/abort/general.any.worker-expected.txt
trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/request/request-consume-empty-expected.txt
trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/request/request-consume-expected.txt
trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/request/request-init-002-expected.txt
trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/response/response-consume-empty-expected.txt
trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/response/response-consume-expected.txt
trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/response/response-error-from-stream-expected.txt
trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/response/response-init-002-expected.txt
trunk/LayoutTests/imported/w3c/web-platform-tests/service-workers/service-worker/fetch-event-respond-with-custom-response.https-expected.txt
trunk/LayoutTests/imported/w3c/web-platform-tests/url/urlencoded-parser.any-expected.txt
trunk/LayoutTests/imported/w3c/web-platform-tests/url/urlencoded-parser.any.worker-expected.txt
trunk/Source/WTF/ChangeLog
trunk/Source/WTF/wtf/PlatformHave.h
trunk/Source/WTF/wtf/StringExtras.h
trunk/Source/WTF/wtf/URLParser.cpp
trunk/Source/WTF/wtf/text/WTFString.cpp
trunk/Source/WTF/wtf/text/WTFString.h
trunk/Source/WTF/wtf/unicode/UTF8Conversion.cpp
trunk/Source/WTF/wtf/unicode/UTF8Conversion.h
trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/Modules/fetch/FetchBody.cpp
trunk/Source/WebCore/Modules/fetch/FetchBody.h
trunk/Source/WebCore/Modules/fetch/FetchBodyConsumer.cpp
trunk/Source/WebCore/Modules/fetch/FetchBodyConsumer.h

[webkit-changes] [266086] trunk/Source

2020-08-24 Thread commit-queue
Title: [266086] trunk/Source








Revision 266086
Author commit-qu...@webkit.org
Date 2020-08-24 14:41:43 -0700 (Mon, 24 Aug 2020)


Log Message
Make _WKWebsiteDataStoreConfiguration SPI for HSTS storage to replace _WKProcessPoolConfiguration.hstsStorageDirectory
https://bugs.webkit.org/show_bug.cgi?id=213048

Patch by Alex Christensen  on 2020-08-24
Reviewed by Youenn Fablet.

Source/WebCore/PAL:

* pal/spi/cf/CFNetworkSPI.h:

Source/WebKit:

This uses CFNetwork SPI introduced in rdar://problem/50109631 to allow HSTS storage per NSURLSession.
To be complete, I also deprecated our UI process HSTS state removal attempt SPIs, WKContextResetHSTSHosts and
WKContextResetHSTSHostsAddedAfterDate, which had their last use removed in rdar://problem/64220838.

I manually verified that this new SPI puts HSTS data in the specified location, and I also verified that HSTS
state querying and removal works with the new CFNetwork SPI as it did with the old one.

* NetworkProcess/NetworkProcess.cpp:
(WebKit::NetworkProcess::fetchWebsiteData):
(WebKit::NetworkProcess::deleteWebsiteData):
(WebKit::NetworkProcess::deleteWebsiteDataForOrigins):
(WebKit::NetworkProcess::deleteAndRestrictWebsiteDataForRegistrableDomains):
(WebKit::NetworkProcess::registrableDomainsWithWebsiteData):
* NetworkProcess/NetworkProcess.h:
* NetworkProcess/NetworkSessionCreationParameters.cpp:
(WebKit::NetworkSessionCreationParameters::encode const):
(WebKit::NetworkSessionCreationParameters::decode):
* NetworkProcess/NetworkSessionCreationParameters.h:
* NetworkProcess/cocoa/NetworkProcessCocoa.mm:
(WebKit::NetworkProcess::hostNamesWithHSTSCache const):
(WebKit::NetworkProcess::deleteHSTSCacheForHostNames):
(WebKit::NetworkProcess::clearHSTSCache):
(WebKit::NetworkProcess::getHostNamesWithHSTSCache): Deleted.
* NetworkProcess/cocoa/NetworkSessionCocoa.h:
* NetworkProcess/cocoa/NetworkSessionCocoa.mm:
(WebKit::NetworkSessionCocoa::hstsStorage const):
(WebKit::NetworkSessionCocoa::NetworkSessionCocoa):
* NetworkProcess/soup/NetworkProcessSoup.cpp:
(WebKit::NetworkProcess::hostNamesWithHSTSCache const):
(WebKit::NetworkProcess::deleteHSTSCacheForHostNames):
(WebKit::NetworkProcess::clearHSTSCache):
(WebKit::NetworkProcess::getHostNamesWithHSTSCache): Deleted.
* UIProcess/API/C/mac/WKContextPrivateMac.h:
* UIProcess/API/C/mac/WKContextPrivateMac.mm:
(WKContextResetHSTSHosts):
(WKContextResetHSTSHostsAddedAfterDate):
* UIProcess/API/Cocoa/_WKProcessPoolConfiguration.h:
* UIProcess/API/Cocoa/_WKWebsiteDataStoreConfiguration.h:
* UIProcess/API/Cocoa/_WKWebsiteDataStoreConfiguration.mm:
(-[_WKWebsiteDataStoreConfiguration hstsStorageDirectory]):
(-[_WKWebsiteDataStoreConfiguration setHSTSStorageDirectory:]):
* UIProcess/Cocoa/WebProcessPoolCocoa.mm:
(WebKit::privateBrowsingSession): Deleted.
(WebKit::WebProcessPool::resetHSTSHosts): Deleted.
(WebKit::WebProcessPool::resetHSTSHostsAddedAfterDate): Deleted.
* UIProcess/WebProcessPool.h:
* UIProcess/WebsiteData/WebsiteDataStore.cpp:
(WebKit::WebsiteDataStore::resolveDirectoriesIfNecessary):
(WebKit::WebsiteDataStore::parameters):
* UIProcess/WebsiteData/WebsiteDataStore.h:
(WebKit::WebsiteDataStore::resolvedHSTSStorageDirectory const):
* UIProcess/WebsiteData/WebsiteDataStoreConfiguration.cpp:
(WebKit::WebsiteDataStoreConfiguration::copy const):
* UIProcess/WebsiteData/WebsiteDataStoreConfiguration.h:

Source/WTF:

* wtf/PlatformHave.h:

Modified Paths

trunk/Source/WTF/ChangeLog
trunk/Source/WTF/wtf/PlatformHave.h
trunk/Source/WebCore/PAL/ChangeLog
trunk/Source/WebCore/PAL/pal/spi/cf/CFNetworkSPI.h
trunk/Source/WebKit/ChangeLog
trunk/Source/WebKit/NetworkProcess/NetworkProcess.cpp
trunk/Source/WebKit/NetworkProcess/NetworkProcess.h
trunk/Source/WebKit/NetworkProcess/NetworkSessionCreationParameters.cpp
trunk/Source/WebKit/NetworkProcess/NetworkSessionCreationParameters.h
trunk/Source/WebKit/NetworkProcess/cocoa/NetworkProcessCocoa.mm
trunk/Source/WebKit/NetworkProcess/cocoa/NetworkSessionCocoa.h
trunk/Source/WebKit/NetworkProcess/cocoa/NetworkSessionCocoa.mm
trunk/Source/WebKit/NetworkProcess/soup/NetworkProcessSoup.cpp
trunk/Source/WebKit/UIProcess/API/C/mac/WKContextPrivateMac.h
trunk/Source/WebKit/UIProcess/API/C/mac/WKContextPrivateMac.mm
trunk/Source/WebKit/UIProcess/API/Cocoa/_WKProcessPoolConfiguration.h
trunk/Source/WebKit/UIProcess/API/Cocoa/_WKWebsiteDataStoreConfiguration.h
trunk/Source/WebKit/UIProcess/API/Cocoa/_WKWebsiteDataStoreConfiguration.mm
trunk/Source/WebKit/UIProcess/Cocoa/WebProcessPoolCocoa.mm
trunk/Source/WebKit/UIProcess/WebProcessPool.h
trunk/Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.cpp
trunk/Source/WebKit/UIProcess/WebsiteData/WebsiteDataStore.h
trunk/Source/WebKit/UIProcess/WebsiteData/WebsiteDataStoreConfiguration.cpp
trunk/Source/WebKit/UIProcess/WebsiteData/WebsiteDataStoreConfiguration.h




Diff

Modified: trunk/Source/WTF/ChangeLog (266085 => 266086)

--- trunk/Source/WTF/ChangeLog	2020-08-24 21:35:54 UTC (rev 266085)
+++ trunk/Source/WTF/ChangeLog	2020-08-24 

[webkit-changes] [266085] trunk/LayoutTests

2020-08-24 Thread hector_i_lopez
Title: [266085] trunk/LayoutTests








Revision 266085
Author hector_i_lo...@apple.com
Date 2020-08-24 14:35:54 -0700 (Mon, 24 Aug 2020)


Log Message
[ iOS wk2 Debug ] imported/w3c/web-platform-tests/html/semantics/scripting-1/the-script-element/moving-between-documents/before-prepare-iframe-fetch-error-external-module.html is a flaky failure
https://bugs.webkit.org/show_bug.cgi?id=215783

Unreviewed test gardening.

* platform/ios-wk2/TestExpectations:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/ios-wk2/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (266084 => 266085)

--- trunk/LayoutTests/ChangeLog	2020-08-24 21:06:20 UTC (rev 266084)
+++ trunk/LayoutTests/ChangeLog	2020-08-24 21:35:54 UTC (rev 266085)
@@ -1,5 +1,14 @@
 2020-08-24  Hector Lopez  
 
+[ iOS wk2 Debug ] imported/w3c/web-platform-tests/html/semantics/scripting-1/the-script-element/moving-between-documents/before-prepare-iframe-fetch-error-external-module.html is a flaky failure
+https://bugs.webkit.org/show_bug.cgi?id=215783
+
+Unreviewed test gardening.
+
+* platform/ios-wk2/TestExpectations:
+
+2020-08-24  Hector Lopez  
+
[ macOS wk1 ] fast/overflow/horizontal-scroll-after-back.html is a flaky timeout
 https://bugs.webkit.org/show_bug.cgi?id=215778
 


Modified: trunk/LayoutTests/platform/ios-wk2/TestExpectations (266084 => 266085)

--- trunk/LayoutTests/platform/ios-wk2/TestExpectations	2020-08-24 21:06:20 UTC (rev 266084)
+++ trunk/LayoutTests/platform/ios-wk2/TestExpectations	2020-08-24 21:35:54 UTC (rev 266085)
@@ -1864,3 +1864,5 @@
 webkit.org/b/215773 http/tests/websocket/tests/hybi/client-close-2.html [ Pass Failure ]
 
 webkit.org/b/215772 platform/ios/ios/fast/coordinates/range-client-rects.html [ Failure ]
+
+webkit.org/b/215783 [ Debug ] imported/w3c/web-platform-tests/html/semantics/scripting-1/the-script-element/moving-between-documents/before-prepare-iframe-fetch-error-external-module.html [ Pass Failure ]






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266084] trunk/Websites/webkit.org

2020-08-24 Thread jond
Title: [266084] trunk/Websites/webkit.org








Revision 266084
Author j...@apple.com
Date 2020-08-24 14:06:20 -0700 (Mon, 24 Aug 2020)


Log Message
Disable unused RPC services
https://bugs.webkit.org/show_bug.cgi?id=215779

Reviewed by Alexey Proskuryakov.

* .htaccess:

Modified Paths

trunk/Websites/webkit.org/.htaccess
trunk/Websites/webkit.org/ChangeLog




Diff

Modified: trunk/Websites/webkit.org/.htaccess (266083 => 266084)

--- trunk/Websites/webkit.org/.htaccess	2020-08-24 19:26:03 UTC (rev 266083)
+++ trunk/Websites/webkit.org/.htaccess	2020-08-24 21:06:20 UTC (rev 266084)
@@ -111,6 +111,12 @@
 # Pixar USD support. Change to model/vnd.usdz+zip once WebKit support is in the wild.
 AddType model/vnd.pixar.usd usdz
 
+# Disable XML-RPC
+
+Order Allow,Deny
+Deny from all
+
+
 # BEGIN WordPress
 
 RewriteEngine On


Modified: trunk/Websites/webkit.org/ChangeLog (266083 => 266084)

--- trunk/Websites/webkit.org/ChangeLog	2020-08-24 19:26:03 UTC (rev 266083)
+++ trunk/Websites/webkit.org/ChangeLog	2020-08-24 21:06:20 UTC (rev 266084)
@@ -1,3 +1,12 @@
+2020-08-24  Jon Davis  
+
+Disable unused RPC services
+https://bugs.webkit.org/show_bug.cgi?id=215779
+
+Reviewed by Alexey Proskuryakov.
+
+* .htaccess:
+
 2020-08-18  Darin Adler  
 
 * languages.md: Fixed "Movaje" typo.






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266083] trunk/LayoutTests

2020-08-24 Thread hector_i_lopez
Title: [266083] trunk/LayoutTests








Revision 266083
Author hector_i_lo...@apple.com
Date 2020-08-24 12:26:03 -0700 (Mon, 24 Aug 2020)


Log Message
   [ macOS wk1 ] fast/overflow/horizontal-scroll-after-back.html is a flaky timeout
https://bugs.webkit.org/show_bug.cgi?id=215778

Unreviewed test gardening.

* platform/mac-wk1/TestExpectations:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/mac-wk1/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (266082 => 266083)

--- trunk/LayoutTests/ChangeLog	2020-08-24 19:17:52 UTC (rev 266082)
+++ trunk/LayoutTests/ChangeLog	2020-08-24 19:26:03 UTC (rev 266083)
@@ -1,3 +1,12 @@
+2020-08-24  Hector Lopez  
+
+   [ macOS wk1 ] fast/overflow/horizontal-scroll-after-back.html is a flaky timeout
+https://bugs.webkit.org/show_bug.cgi?id=215778
+
+Unreviewed test gardening.
+
+* platform/mac-wk1/TestExpectations:
+
 2020-08-24  Emilio Cobos Álvarez  
 
 Support quotes:auto and fix quotes serialization.


Modified: trunk/LayoutTests/platform/mac-wk1/TestExpectations (266082 => 266083)

--- trunk/LayoutTests/platform/mac-wk1/TestExpectations	2020-08-24 19:17:52 UTC (rev 266082)
+++ trunk/LayoutTests/platform/mac-wk1/TestExpectations	2020-08-24 19:26:03 UTC (rev 266083)
@@ -1137,3 +1137,6 @@
 
 webkit.org/b/215767 [ Debug ] inspector/animation/nameChanged.html [ Pass Crash ]
 
+webkit.org/b/215778 fast/overflow/horizontal-scroll-after-back.html [ Pass Timeout ]
+
+






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266082] trunk

2020-08-24 Thread emilio
Title: [266082] trunk








Revision 266082
Author emi...@crisal.io
Date 2020-08-24 12:17:52 -0700 (Mon, 24 Aug 2020)


Log Message
Support quotes:auto and fix quotes serialization.
https://bugs.webkit.org/show_bug.cgi?id=215646

Reviewed by Darin Adler.

LayoutTests/imported/w3c:

* web-platform-tests/css/css-content/inheritance-expected.txt: Annotate progression.
* web-platform-tests/css/cssom/cssstyledeclaration-csstext-expected.txt: Ditto.

Source/WebCore:

Tests: imported/w3c/web-platform-tests/css/css-content/inheritance.html
   imported/w3c/web-platform-tests/css/cssom/cssstyledeclaration-csstext.html

* css/CSSComputedStyleDeclaration.cpp:
(WebCore::RefvalueForQuotes):
(WebCore::ComputedStyleExtractor::valueForPropertyInStyle):
* css/parser/CSSPropertyParser.cpp:
(WebCore::consumeQuotes):
* rendering/style/QuotesData.h:
(WebCore::QuotesData::size const):
* style/StyleBuilderConverter.h:
(WebCore::Style::BuilderConverter::convertQuotes):

LayoutTests:

* platform/gtk/imported/w3c/web-platform-tests/css/cssom/cssstyledeclaration-csstext-expected.txt: Annotate progression.
* platform/ios-wk2/imported/w3c/web-platform-tests/css/cssom/cssstyledeclaration-csstext-expected.txt: Ditto.
* platform/wpe/imported/w3c/web-platform-tests/css/cssom/cssstyledeclaration-csstext-expected.txt: Ditto.

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/imported/w3c/ChangeLog
trunk/LayoutTests/imported/w3c/web-platform-tests/css/css-content/inheritance-expected.txt
trunk/LayoutTests/imported/w3c/web-platform-tests/css/cssom/cssstyledeclaration-csstext-expected.txt
trunk/LayoutTests/platform/gtk/imported/w3c/web-platform-tests/css/cssom/cssstyledeclaration-csstext-expected.txt
trunk/LayoutTests/platform/ios-wk2/imported/w3c/web-platform-tests/css/cssom/cssstyledeclaration-csstext-expected.txt
trunk/LayoutTests/platform/wpe/imported/w3c/web-platform-tests/css/cssom/cssstyledeclaration-csstext-expected.txt
trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/css/CSSComputedStyleDeclaration.cpp
trunk/Source/WebCore/css/parser/CSSPropertyParser.cpp
trunk/Source/WebCore/rendering/style/QuotesData.h
trunk/Source/WebCore/style/StyleBuilderConverter.h




Diff

Modified: trunk/LayoutTests/ChangeLog (266081 => 266082)

--- trunk/LayoutTests/ChangeLog	2020-08-24 18:57:05 UTC (rev 266081)
+++ trunk/LayoutTests/ChangeLog	2020-08-24 19:17:52 UTC (rev 266082)
@@ -1,3 +1,14 @@
+2020-08-24  Emilio Cobos Álvarez  
+
+Support quotes:auto and fix quotes serialization.
+https://bugs.webkit.org/show_bug.cgi?id=215646
+
+Reviewed by Darin Adler.
+
+* platform/gtk/imported/w3c/web-platform-tests/css/cssom/cssstyledeclaration-csstext-expected.txt: Annotate progression.
+* platform/ios-wk2/imported/w3c/web-platform-tests/css/cssom/cssstyledeclaration-csstext-expected.txt: Ditto.
+* platform/wpe/imported/w3c/web-platform-tests/css/cssom/cssstyledeclaration-csstext-expected.txt: Ditto.
+
 2020-08-24  Hector Lopez  
 
[ macOS wk2 Release ] http/tests/workers/service/basic-install-event-waitUntil-reject.html is a flaky timeout


Modified: trunk/LayoutTests/imported/w3c/ChangeLog (266081 => 266082)

--- trunk/LayoutTests/imported/w3c/ChangeLog	2020-08-24 18:57:05 UTC (rev 266081)
+++ trunk/LayoutTests/imported/w3c/ChangeLog	2020-08-24 19:17:52 UTC (rev 266082)
@@ -1,3 +1,13 @@
+2020-08-24  Emilio Cobos Álvarez  
+
+Support quotes:auto and fix quotes serialization.
+https://bugs.webkit.org/show_bug.cgi?id=215646
+
+Reviewed by Darin Adler.
+
+* web-platform-tests/css/css-content/inheritance-expected.txt: Annotate progression.
+* web-platform-tests/css/cssom/cssstyledeclaration-csstext-expected.txt: Ditto.
+
 2020-08-24  Justin Uberti  
 
 RTCRtpSynchronizationSource.rtpTimestamp is not present


Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/css/css-content/inheritance-expected.txt (266081 => 266082)

--- trunk/LayoutTests/imported/w3c/web-platform-tests/css/css-content/inheritance-expected.txt	2020-08-24 18:57:05 UTC (rev 266081)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/css/css-content/inheritance-expected.txt	2020-08-24 19:17:52 UTC (rev 266082)
@@ -1,6 +1,6 @@
 
-FAIL Property quotes has initial value auto assert_equals: expected "auto" but got ""
-FAIL Property quotes inherits assert_equals: expected "none" but got ""
+PASS Property quotes has initial value auto 
+PASS Property quotes inherits 
 FAIL Property bookmark-level has initial value none assert_true: bookmark-level doesn't seem to be supported in the computed style expected true got false
 FAIL Property bookmark-level does not inherit assert_true: expected true got false
 FAIL Property bookmark-state has initial value open assert_true: bookmark-state doesn't seem to be supported in the computed style expected true got false


Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/css/cssom/cssstyledeclaration-csstext-expected.txt (266081 

[webkit-changes] [266081] trunk/Source

2020-08-24 Thread repstein
Title: [266081] trunk/Source








Revision 266081
Author repst...@apple.com
Date 2020-08-24 11:57:05 -0700 (Mon, 24 Aug 2020)


Log Message
Versioning.

WebKit-7611.1.1

Modified Paths

trunk/Source/_javascript_Core/Configurations/Version.xcconfig
trunk/Source/ThirdParty/ANGLE/Configurations/Version.xcconfig
trunk/Source/ThirdParty/libwebrtc/Configurations/Version.xcconfig
trunk/Source/WebCore/Configurations/Version.xcconfig
trunk/Source/WebCore/PAL/Configurations/Version.xcconfig
trunk/Source/WebInspectorUI/Configurations/Version.xcconfig
trunk/Source/WebKit/Configurations/Version.xcconfig
trunk/Source/WebKitLegacy/mac/Configurations/Version.xcconfig




Diff

Modified: trunk/Source/_javascript_Core/Configurations/Version.xcconfig (266080 => 266081)

--- trunk/Source/_javascript_Core/Configurations/Version.xcconfig	2020-08-24 18:53:31 UTC (rev 266080)
+++ trunk/Source/_javascript_Core/Configurations/Version.xcconfig	2020-08-24 18:57:05 UTC (rev 266081)
@@ -21,9 +21,9 @@
 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
 
-MAJOR_VERSION = 610;
-MINOR_VERSION = 2;
-TINY_VERSION = 2;
+MAJOR_VERSION = 611;
+MINOR_VERSION = 1;
+TINY_VERSION = 1;
 MICRO_VERSION = 0;
 NANO_VERSION = 0;
 FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);


Modified: trunk/Source/ThirdParty/ANGLE/Configurations/Version.xcconfig (266080 => 266081)

--- trunk/Source/ThirdParty/ANGLE/Configurations/Version.xcconfig	2020-08-24 18:53:31 UTC (rev 266080)
+++ trunk/Source/ThirdParty/ANGLE/Configurations/Version.xcconfig	2020-08-24 18:57:05 UTC (rev 266081)
@@ -21,9 +21,9 @@
 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
 
-MAJOR_VERSION = 610;
-MINOR_VERSION = 2;
-TINY_VERSION = 2;
+MAJOR_VERSION = 611;
+MINOR_VERSION = 1;
+TINY_VERSION = 1;
 MICRO_VERSION = 0;
 NANO_VERSION = 0;
 FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);


Modified: trunk/Source/ThirdParty/libwebrtc/Configurations/Version.xcconfig (266080 => 266081)

--- trunk/Source/ThirdParty/libwebrtc/Configurations/Version.xcconfig	2020-08-24 18:53:31 UTC (rev 266080)
+++ trunk/Source/ThirdParty/libwebrtc/Configurations/Version.xcconfig	2020-08-24 18:57:05 UTC (rev 266081)
@@ -21,9 +21,9 @@
 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
 
-MAJOR_VERSION = 610;
-MINOR_VERSION = 2;
-TINY_VERSION = 2;
+MAJOR_VERSION = 611;
+MINOR_VERSION = 1;
+TINY_VERSION = 1;
 MICRO_VERSION = 0;
 NANO_VERSION = 0;
 FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);


Modified: trunk/Source/WebCore/Configurations/Version.xcconfig (266080 => 266081)

--- trunk/Source/WebCore/Configurations/Version.xcconfig	2020-08-24 18:53:31 UTC (rev 266080)
+++ trunk/Source/WebCore/Configurations/Version.xcconfig	2020-08-24 18:57:05 UTC (rev 266081)
@@ -21,9 +21,9 @@
 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
 
-MAJOR_VERSION = 610;
-MINOR_VERSION = 2;
-TINY_VERSION = 2;
+MAJOR_VERSION = 611;
+MINOR_VERSION = 1;
+TINY_VERSION = 1;
 MICRO_VERSION = 0;
 NANO_VERSION = 0;
 FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);


Modified: trunk/Source/WebCore/PAL/Configurations/Version.xcconfig (266080 => 266081)

--- trunk/Source/WebCore/PAL/Configurations/Version.xcconfig	2020-08-24 18:53:31 UTC (rev 266080)
+++ trunk/Source/WebCore/PAL/Configurations/Version.xcconfig	2020-08-24 18:57:05 UTC (rev 266081)
@@ -21,9 +21,9 @@
 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
 
-MAJOR_VERSION = 610;
-MINOR_VERSION = 2;
-TINY_VERSION = 2;
+MAJOR_VERSION = 611;
+MINOR_VERSION = 1;
+TINY_VERSION = 1;
 MICRO_VERSION = 0;
 NANO_VERSION = 0;
 FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);


Modified: trunk/Source/WebInspectorUI/Configurations/Version.xcconfig (266080 => 266081)

--- trunk/Source/WebInspectorUI/Configurations/Version.xcconfig	2020-08-24 18:53:31 UTC (rev 266080)
+++ trunk/Source/WebInspectorUI/Configurations/Version.xcconfig	2020-08-24 18:57:05 UTC (rev 266081)
@@ -1,6 +1,6 @@
-MAJOR_VERSION = 610;
-MINOR_VERSION = 2;
-TINY_VERSION = 2;
+MAJOR_VERSION = 611;
+MINOR_VERSION = 1;
+TINY_VERSION = 1;
 MICRO_VERSION = 0;
 NANO_VERSION = 0;
 FULL_VERSION = $(MAJOR_VERSION).$(MINOR_VERSION).$(TINY_VERSION);


Modified: trunk/Source/WebKit/Configurations/Version.xcconfig (266080 => 266081)

--- trunk/Source/WebKit/Configurations/Version.xcconfig	2020-08-24 18:53:31 UTC (rev 266080)
+++ trunk/Source/WebKit/Configurations/Version.xcconfig	2020-08-24 18:57:05 UTC (rev 266081)
@@ -21,9 +21,9 @@
 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN 

[webkit-changes] [266080] trunk/Tools

2020-08-24 Thread jbedard
Title: [266080] trunk/Tools








Revision 266080
Author jbed...@apple.com
Date 2020-08-24 11:53:31 -0700 (Mon, 24 Aug 2020)


Log Message
[resultsdbpy] Fix pip package
https://bugs.webkit.org/show_bug.cgi?id=215721


Reviewed by Dewei Zhu.

* Scripts/libraries/resultsdbpy/MANIFEST.in: Add html and css.
* Scripts/libraries/resultsdbpy/resultsdbpy/__init__.py: Include submodules.
* Scripts/libraries/resultsdbpy/setup.py: Version bump.

Modified Paths

trunk/Tools/ChangeLog
trunk/Tools/Scripts/libraries/resultsdbpy/MANIFEST.in
trunk/Tools/Scripts/libraries/resultsdbpy/resultsdbpy/__init__.py
trunk/Tools/Scripts/libraries/resultsdbpy/setup.py




Diff

Modified: trunk/Tools/ChangeLog (266079 => 266080)

--- trunk/Tools/ChangeLog	2020-08-24 18:44:29 UTC (rev 266079)
+++ trunk/Tools/ChangeLog	2020-08-24 18:53:31 UTC (rev 266080)
@@ -1,3 +1,15 @@
+2020-08-24  Jonathan Bedard  
+
+[resultsdbpy] Fix pip package
+https://bugs.webkit.org/show_bug.cgi?id=215721
+
+
+Reviewed by Dewei Zhu.
+
+* Scripts/libraries/resultsdbpy/MANIFEST.in: Add html and css.
+* Scripts/libraries/resultsdbpy/resultsdbpy/__init__.py: Include submodules.
+* Scripts/libraries/resultsdbpy/setup.py: Version bump.
+
 2020-08-24  Sihui Liu  
 
 Text manipulationshould not manipulate nodes out of paragraph range


Modified: trunk/Tools/Scripts/libraries/resultsdbpy/MANIFEST.in (266079 => 266080)

--- trunk/Tools/Scripts/libraries/resultsdbpy/MANIFEST.in	2020-08-24 18:44:29 UTC (rev 266079)
+++ trunk/Tools/Scripts/libraries/resultsdbpy/MANIFEST.in	2020-08-24 18:53:31 UTC (rev 266080)
@@ -1 +1,3 @@
 include README.md
+recursive-include resultsdbpy/view/static *
+recursive-include resultsdbpy/view/templates *


Modified: trunk/Tools/Scripts/libraries/resultsdbpy/resultsdbpy/__init__.py (266079 => 266080)

--- trunk/Tools/Scripts/libraries/resultsdbpy/resultsdbpy/__init__.py	2020-08-24 18:44:29 UTC (rev 266079)
+++ trunk/Tools/Scripts/libraries/resultsdbpy/resultsdbpy/__init__.py	2020-08-24 18:53:31 UTC (rev 266080)
@@ -43,6 +43,6 @@
 "Please install webkitcorepy with `pip install webkitcorepy --extra-index-url `"
 )
 
-version = Version(1, 0, 0)
+version = Version(1, 0, 1)
 
 name = 'resultsdbpy'


Modified: trunk/Tools/Scripts/libraries/resultsdbpy/setup.py (266079 => 266080)

--- trunk/Tools/Scripts/libraries/resultsdbpy/setup.py	2020-08-24 18:44:29 UTC (rev 266079)
+++ trunk/Tools/Scripts/libraries/resultsdbpy/setup.py	2020-08-24 18:53:31 UTC (rev 266080)
@@ -50,7 +50,14 @@
 author='Jonathan Bedard',
 author_email='jbed...@apple.com',
 license='Modified BSD',
-packages=['resultsdbpy'],
+packages=[
+'resultsdbpy',
+'resultsdbpy.controller',
+'resultsdbpy.example',
+'resultsdbpy.flask_support',
+'resultsdbpy.model',
+'resultsdbpy.view',
+],
 install_requires=[
 'cassandra-driver',
 'fakeredis',






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266079] branches/safari-610-branch/

2020-08-24 Thread repstein
Title: [266079] branches/safari-610-branch/








Revision 266079
Author repst...@apple.com
Date 2020-08-24 11:44:29 -0700 (Mon, 24 Aug 2020)


Log Message
New branch.

Added Paths

branches/safari-610-branch/




Diff




___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266078] trunk/LayoutTests

2020-08-24 Thread hector_i_lopez
Title: [266078] trunk/LayoutTests








Revision 266078
Author hector_i_lo...@apple.com
Date 2020-08-24 11:44:25 -0700 (Mon, 24 Aug 2020)


Log Message
   [ macOS wk2 Release ] http/tests/workers/service/basic-install-event-waitUntil-reject.html is a flaky timeout
https://bugs.webkit.org/show_bug.cgi?id=214997

Unreviewed test gardening.

* platform/mac-wk2/TestExpectations:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/mac-wk2/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (266077 => 266078)

--- trunk/LayoutTests/ChangeLog	2020-08-24 18:40:30 UTC (rev 266077)
+++ trunk/LayoutTests/ChangeLog	2020-08-24 18:44:25 UTC (rev 266078)
@@ -1,3 +1,12 @@
+2020-08-24  Hector Lopez  
+
+   [ macOS wk2 Release ] http/tests/workers/service/basic-install-event-waitUntil-reject.html is a flaky timeout
+https://bugs.webkit.org/show_bug.cgi?id=214997
+
+Unreviewed test gardening.
+
+* platform/mac-wk2/TestExpectations:
+
 2020-08-24  Devin Rousso  
 
 Web Inspector: allow event breakpoints to be configured


Modified: trunk/LayoutTests/platform/mac-wk2/TestExpectations (266077 => 266078)

--- trunk/LayoutTests/platform/mac-wk2/TestExpectations	2020-08-24 18:40:30 UTC (rev 266077)
+++ trunk/LayoutTests/platform/mac-wk2/TestExpectations	2020-08-24 18:44:25 UTC (rev 266078)
@@ -1276,3 +1276,6 @@
 [ BigSur+ ] media/vp9.html [ Pass ]
 
 webkit.org/b/215700 [ Debug ] fast/scrolling/mac/scrollbars/overlay-scrollbar-hovered.html [ Pass Failure Timeout ]
+
+webkit.org/b/214997 [ Release ] http/tests/workers/service/basic-install-event-waitUntil-reject.html [ Pass Timeout ]
+






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266076] trunk/Source/WebInspectorUI

2020-08-24 Thread drousso
Title: [266076] trunk/Source/WebInspectorUI








Revision 266076
Author drou...@apple.com
Date 2020-08-24 11:13:03 -0700 (Mon, 24 Aug 2020)


Log Message
Web Inspector: remove legacy code for replacing the Resources Tab and Debugger Tab with the Sources Tab
https://bugs.webkit.org/show_bug.cgi?id=205826

Reviewed by Joseph Pecoraro.

* UserInterface/Base/Main.js:
(WI.loaded):

Modified Paths

trunk/Source/WebInspectorUI/ChangeLog
trunk/Source/WebInspectorUI/UserInterface/Base/Main.js




Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (266075 => 266076)

--- trunk/Source/WebInspectorUI/ChangeLog	2020-08-24 18:06:01 UTC (rev 266075)
+++ trunk/Source/WebInspectorUI/ChangeLog	2020-08-24 18:13:03 UTC (rev 266076)
@@ -1,5 +1,15 @@
 2020-08-24  Devin Rousso  
 
+Web Inspector: remove legacy code for replacing the Resources Tab and Debugger Tab with the Sources Tab
+https://bugs.webkit.org/show_bug.cgi?id=205826
+
+Reviewed by Joseph Pecoraro.
+
+* UserInterface/Base/Main.js:
+(WI.loaded):
+
+2020-08-24  Devin Rousso  
+
 Web Inspector: allow event breakpoints to be configured
 https://bugs.webkit.org/show_bug.cgi?id=215362
 


Modified: trunk/Source/WebInspectorUI/UserInterface/Base/Main.js (266075 => 266076)

--- trunk/Source/WebInspectorUI/UserInterface/Base/Main.js	2020-08-24 18:06:01 UTC (rev 266075)
+++ trunk/Source/WebInspectorUI/UserInterface/Base/Main.js	2020-08-24 18:13:03 UTC (rev 266076)
@@ -158,26 +158,6 @@
 ]);
 WI._selectedTabIndexSetting = new WI.Setting("selected-tab-index", 0);
 
-// FIXME:  Web Inspector: remove legacy code for replacing the Resources Tab and Debugger Tab with the Sources Tab
-let debuggerIndex = WI._openTabsSetting.value.indexOf("debugger");
-let resourcesIndex = WI._openTabsSetting.value.indexOf("resources");
-if (debuggerIndex >= 0 || resourcesIndex >= 0) {
-WI._openTabsSetting.value.remove("debugger");
-WI._openTabsSetting.value.remove("resources");
-
-if (debuggerIndex === -1)
-debuggerIndex = Infinity;
-if (resourcesIndex === -1)
-resourcesIndex = Infinity;
-
-let sourcesIndex = Math.min(debuggerIndex, resourcesIndex);
-WI._openTabsSetting.value.splice(sourcesIndex, 1, WI.SourcesTabContentView.Type);
-WI._openTabsSetting.save();
-
-if (WI._selectedTabIndexSetting.value === debuggerIndex || WI._selectedTabIndexSetting.value === resourcesIndex)
-WI._selectedTabIndexSetting.value = sourcesIndex;
-}
-
 // State.
 WI.printStylesEnabled = false;
 WI.setZoomFactor(WI.settings.zoomFactor.value);






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266075] trunk

2020-08-24 Thread sihui_liu
Title: [266075] trunk








Revision 266075
Author sihui_...@apple.com
Date 2020-08-24 11:06:01 -0700 (Mon, 24 Aug 2020)


Log Message
Source/WebCore:
Text manipulation should not manipulate nodes out of paragraph range
https://bugs.webkit.org/show_bug.cgi?id=215406

Reviewed by Wenson Hsieh.

TextManipulationController currently does not set correct start path for insertion. Therefore, it does not mark
the nodes on the start path but out of range correctly, and may change position of those nodes by mistake. For
example, in the newly added test, text node "zero" can be moved around even though it is not in range.

API test: TextManipulation.CompleteTextManipulationShouldOnlyChangeNodesInParagraphRange

* editing/TextManipulationController.cpp:
(WebCore::TextManipulationController::replace):

Tools:
Text manipulationshould not manipulate nodes out of paragraph range
https://bugs.webkit.org/show_bug.cgi?id=215406

Reviewed by Wenson Hsieh.

* TestWebKitAPI/Tests/WebKitCocoa/TextManipulation.mm:
(TestWebKitAPI::TEST):

Modified Paths

trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/editing/TextManipulationController.cpp
trunk/Tools/ChangeLog
trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/TextManipulation.mm




Diff

Modified: trunk/Source/WebCore/ChangeLog (266074 => 266075)

--- trunk/Source/WebCore/ChangeLog	2020-08-24 17:34:12 UTC (rev 266074)
+++ trunk/Source/WebCore/ChangeLog	2020-08-24 18:06:01 UTC (rev 266075)
@@ -1,3 +1,19 @@
+2020-08-24  Sihui Liu  
+
+Text manipulation should not manipulate nodes out of paragraph range
+https://bugs.webkit.org/show_bug.cgi?id=215406
+
+Reviewed by Wenson Hsieh.
+
+TextManipulationController currently does not set correct start path for insertion. Therefore, it does not mark 
+the nodes on the start path but out of range correctly, and may change position of those nodes by mistake. For 
+example, in the newly added test, text node "zero" can be moved around even though it is not in range.
+
+API test: TextManipulation.CompleteTextManipulationShouldOnlyChangeNodesInParagraphRange
+
+* editing/TextManipulationController.cpp:
+(WebCore::TextManipulationController::replace):
+
 2020-08-24  Devin Rousso  
 
 Web Inspector: allow event breakpoints to be configured


Modified: trunk/Source/WebCore/editing/TextManipulationController.cpp (266074 => 266075)

--- trunk/Source/WebCore/editing/TextManipulationController.cpp	2020-08-24 17:34:12 UTC (rev 266074)
+++ trunk/Source/WebCore/editing/TextManipulationController.cpp	2020-08-24 18:06:01 UTC (rev 266075)
@@ -829,8 +829,20 @@
 nodesToRemove.remove(*node);
 
 HashSet> reusedOriginalNodes;
-Vector lastTopDownPath;
 Vector insertions;
+auto startTopDownPath = getPath(commonAncestor.get(), firstContentNode.get());
+while (!startTopDownPath.isEmpty()) {
+auto lastNode = startTopDownPath.last();
+ASSERT(is(lastNode.get()));
+if (!downcast(lastNode.get()).hasOneChild())
+break;
+nodesToRemove.add(startTopDownPath.takeLast());
+}
+auto lastTopDownPath = startTopDownPath.map([&](auto node) -> NodeEntry {
+reusedOriginalNodes.add(node.copyRef());
+return { node, node };
+});
+
 for (size_t index = 0; index < replacementTokens.size(); ++index) {
 auto& replacementToken = replacementTokens[index];
 auto it = tokenExchangeMap.find(replacementToken.identifier);


Modified: trunk/Tools/ChangeLog (266074 => 266075)

--- trunk/Tools/ChangeLog	2020-08-24 17:34:12 UTC (rev 266074)
+++ trunk/Tools/ChangeLog	2020-08-24 18:06:01 UTC (rev 266075)
@@ -1,3 +1,13 @@
+2020-08-24  Sihui Liu  
+
+Text manipulationshould not manipulate nodes out of paragraph range
+https://bugs.webkit.org/show_bug.cgi?id=215406
+
+Reviewed by Wenson Hsieh.
+
+* TestWebKitAPI/Tests/WebKitCocoa/TextManipulation.mm:
+(TestWebKitAPI::TEST):
+
 2020-08-24  Aditya Keerthi  
 
 [macOS] Show picker for date and datetime-local input types


Modified: trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/TextManipulation.mm (266074 => 266075)

--- trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/TextManipulation.mm	2020-08-24 17:34:12 UTC (rev 266074)
+++ trunk/Tools/TestWebKitAPI/Tests/WebKitCocoa/TextManipulation.mm	2020-08-24 18:06:01 UTC (rev 266075)
@@ -3131,6 +3131,54 @@
 EXPECT_WK_STREQ("ImagesLink }
 
+TEST(TextManipulation, CompleteTextManipulationShouldOnlyChangeNodesInParagraphRange)
+{
+auto delegate = adoptNS([[TextManipulationDelegate alloc] init]);
+auto webView = adoptNS([[TestWKWebView alloc] initWithFrame:NSMakeRect(0, 0, 400, 400)]);
+[webView _setTextManipulationDelegate:delegate.get()];
+[webView synchronouslyLoadHTMLString:@""
+""
+""
+"span { white-space:pre; }"
+""
+""
+""
+"zero two four"
+"one"
+"three"
+ 

[webkit-changes] [266073] trunk/LayoutTests

2020-08-24 Thread hector_i_lopez
Title: [266073] trunk/LayoutTests








Revision 266073
Author hector_i_lo...@apple.com
Date 2020-08-24 10:32:03 -0700 (Mon, 24 Aug 2020)


Log Message
Regression (r266028): platform/ios/ios/fast/coordinates/range-client-rects.html
https://bugs.webkit.org/show_bug.cgi?id=215772

Unreviewwed test gardening.

* platform/ios-wk2/TestExpectations:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/ios-wk2/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (266072 => 266073)

--- trunk/LayoutTests/ChangeLog	2020-08-24 17:31:56 UTC (rev 266072)
+++ trunk/LayoutTests/ChangeLog	2020-08-24 17:32:03 UTC (rev 266073)
@@ -1,3 +1,12 @@
+2020-08-24  Hector Lopez  
+
+Regression (r266028): platform/ios/ios/fast/coordinates/range-client-rects.html
+https://bugs.webkit.org/show_bug.cgi?id=215772
+
+Unreviewwed test gardening.
+
+* platform/ios-wk2/TestExpectations:
+
 2020-08-24  Youenn Fablet  
 
 Add a test showing the difference of behavior when closing a WebSocket connection between legacy WebSocket and NSURLSession WebSocket code paths


Modified: trunk/LayoutTests/platform/ios-wk2/TestExpectations (266072 => 266073)

--- trunk/LayoutTests/platform/ios-wk2/TestExpectations	2020-08-24 17:31:56 UTC (rev 266072)
+++ trunk/LayoutTests/platform/ios-wk2/TestExpectations	2020-08-24 17:32:03 UTC (rev 266073)
@@ -1862,3 +1862,5 @@
 webkit.org/b/215706 [ Debug ] imported/w3c/web-platform-tests/html/semantics/scripting-1/the-script-element/moving-between-documents/before-prepare-createHTMLDocument-parse-error-external-classic.html [ Pass Failure ]
 
 webkit.org/b/215773 http/tests/websocket/tests/hybi/client-close-2.html [ Pass Failure ]
+
+webkit.org/b/215772 platform/ios/ios/fast/coordinates/range-client-rects.html [ Failure ]






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266072] trunk/Source

2020-08-24 Thread drousso
Title: [266072] trunk/Source








Revision 266072
Author drou...@apple.com
Date 2020-08-24 10:31:56 -0700 (Mon, 24 Aug 2020)


Log Message
Web Inspector: remove "extra domains" concept now that domains can be added based on the debuggable type
https://bugs.webkit.org/show_bug.cgi?id=201150


Reviewed by Brian Burg.

Source/_javascript_Core:

* inspector/scripts/codegen/objc_generator_templates.py:
* inspector/augmentable/AugmentableInspectorController.h:

* inspector/JSGlobalObjectInspectorController.h:
* inspector/JSGlobalObjectInspectorController.cpp:
(Inspector::JSGlobalObjectInspectorController::connectFrontend):
(Inspector::JSGlobalObjectInspectorController::registerAlternateAgent): Added.
(Inspector::JSGlobalObjectInspectorController::appendExtraAgent): Deleted.

* inspector/InspectorAgentRegistry.h:
* inspector/InspectorAgentRegistry.cpp:
(Inspector::AgentRegistry::appendExtraAgent): Deleted.

* inspector/protocol/Inspector.json:
* inspector/agents/InspectorAgent.h:
* inspector/agents/InspectorAgent.cpp:
(Inspector::InspectorAgent::activateExtraDomain): Deleted.
(Inspector::InspectorAgent::activateExtraDomains): Deleted.

* inspector/scripts/tests/expected/command-targetType-matching-domain-debuggableType.json-result:
* inspector/scripts/tests/expected/commands-with-async-attribute.json-result:
* inspector/scripts/tests/expected/commands-with-optional-call-return-parameters.json-result:
* inspector/scripts/tests/expected/definitions-with-mac-platform.json-result:
* inspector/scripts/tests/expected/domain-debuggableTypes.json-result:
* inspector/scripts/tests/expected/domain-targetType-matching-domain-debuggableType.json-result:
* inspector/scripts/tests/expected/domain-targetTypes.json-result:
* inspector/scripts/tests/expected/domains-with-varying-command-sizes.json-result:
* inspector/scripts/tests/expected/enum-values.json-result:
* inspector/scripts/tests/expected/event-targetType-matching-domain-debuggableType.json-result:
* inspector/scripts/tests/expected/generate-domains-with-feature-guards.json-result:
Rebase protocol tests.

Source/WebInspectorUI:

* UserInterface/Base/Object.js:
* UserInterface/Protocol/InspectorObserver.js:
(WI.InspectorObserver.prototype.activateExtraDomains):
* UserInterface/Protocol/Target.js:
(WI.Target.prototype.activateExtraDomain):
* UserInterface/Controllers/AppController.js:
(WI.AppController.prototype.activateExtraDomains):
* UserInterface/Controllers/AnimationManager.js:
(WI.AnimationManager.prototype.activateExtraDomain):
* UserInterface/Controllers/ApplicationCacheManager.js:
(WI.ApplicationCacheManager.prototype.activateExtraDomain):
* UserInterface/Controllers/CanvasManager.js:
(WI.CanvasManager.prototype.activateExtraDomain):
* UserInterface/Controllers/DOMStorageManager.js:
(WI.DOMStorageManager.prototype.activateExtraDomain):
* UserInterface/Controllers/DatabaseManager.js:
(WI.DatabaseManager.prototype.activateExtraDomain):
* UserInterface/Controllers/HeapManager.js:
(WI.HeapManager.prototype.activateExtraDomain):
* UserInterface/Controllers/IndexedDBManager.js:
(WI.IndexedDBManager.prototype.activateExtraDomain):
* UserInterface/Controllers/MemoryManager.js:
(WI.MemoryManager.prototype.activateExtraDomain):
* UserInterface/Controllers/NetworkManager.js:
(WI.NetworkManager.prototype._extraDomainsActivated):
* UserInterface/Controllers/TimelineManager.js:
(WI.TimelineManager.prototype.activateExtraDomain):
* UserInterface/Base/Main.js:
(WI.activateExtraDomains):
* UserInterface/Views/SourcesNavigationSidebarPanel.js:
(WI.SourcesNavigationSidebarPanel):
(WI.SourcesNavigationSidebarPanel.prototype._handleExtraDomainsActivated):
Add compatibility comments.

* UserInterface/Protocol/InspectorBackend.js:
(InspectorBackendClass):
(InspectorBackend.Domain):
Add FIXMEs.

Modified Paths

trunk/Source/_javascript_Core/ChangeLog
trunk/Source/_javascript_Core/inspector/InspectorAgentRegistry.cpp
trunk/Source/_javascript_Core/inspector/InspectorAgentRegistry.h
trunk/Source/_javascript_Core/inspector/JSGlobalObjectInspectorController.cpp
trunk/Source/_javascript_Core/inspector/JSGlobalObjectInspectorController.h
trunk/Source/_javascript_Core/inspector/agents/InspectorAgent.cpp
trunk/Source/_javascript_Core/inspector/agents/InspectorAgent.h
trunk/Source/_javascript_Core/inspector/augmentable/AugmentableInspectorController.h
trunk/Source/_javascript_Core/inspector/protocol/Inspector.json
trunk/Source/_javascript_Core/inspector/scripts/codegen/objc_generator_templates.py
trunk/Source/_javascript_Core/inspector/scripts/tests/expected/command-targetType-matching-domain-debuggableType.json-result
trunk/Source/_javascript_Core/inspector/scripts/tests/expected/commands-with-async-attribute.json-result
trunk/Source/_javascript_Core/inspector/scripts/tests/expected/commands-with-optional-call-return-parameters.json-result
trunk/Source/_javascript_Core/inspector/scripts/tests/expected/definitions-with-mac-platform.json-result

[webkit-changes] [266071] trunk/Source/WebInspectorUI

2020-08-24 Thread drousso
Title: [266071] trunk/Source/WebInspectorUI








Revision 266071
Author drou...@apple.com
Date 2020-08-24 10:30:29 -0700 (Mon, 24 Aug 2020)


Log Message
Web Inspector: remove legacy code for replacing the Canvas Tab with the Graphics Tab
https://bugs.webkit.org/show_bug.cgi?id=205827

Reviewed by Joseph Pecoraro.

* UserInterface/Base/Main.js:
(WI.loaded):

Modified Paths

trunk/Source/WebInspectorUI/ChangeLog
trunk/Source/WebInspectorUI/UserInterface/Base/Main.js




Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (266070 => 266071)

--- trunk/Source/WebInspectorUI/ChangeLog	2020-08-24 17:24:52 UTC (rev 266070)
+++ trunk/Source/WebInspectorUI/ChangeLog	2020-08-24 17:30:29 UTC (rev 266071)
@@ -1,5 +1,15 @@
 2020-08-24  Devin Rousso  
 
+Web Inspector: remove legacy code for replacing the Canvas Tab with the Graphics Tab
+https://bugs.webkit.org/show_bug.cgi?id=205827
+
+Reviewed by Joseph Pecoraro.
+
+* UserInterface/Base/Main.js:
+(WI.loaded):
+
+2020-08-24  Devin Rousso  
+
 Web Inspector: Elements: Styles: don't show swatches for properties that aren't used/apply
 https://bugs.webkit.org/show_bug.cgi?id=215681
 


Modified: trunk/Source/WebInspectorUI/UserInterface/Base/Main.js (266070 => 266071)

--- trunk/Source/WebInspectorUI/UserInterface/Base/Main.js	2020-08-24 17:24:52 UTC (rev 266070)
+++ trunk/Source/WebInspectorUI/UserInterface/Base/Main.js	2020-08-24 17:30:29 UTC (rev 266071)
@@ -178,13 +178,6 @@
 WI._selectedTabIndexSetting.value = sourcesIndex;
 }
 
-// FIXME:  Web Inspector: remove legacy code for replacing the Canvas Tab with the Graphics Tab
-let canvasIndex = WI._openTabsSetting.value.indexOf("canvas");
-if (canvasIndex >= 0) {
-WI._openTabsSetting.value.splice(canvasIndex, 1, WI.GraphicsTabContentView.Type);
-WI._openTabsSetting.save();
-}
-
 // State.
 WI.printStylesEnabled = false;
 WI.setZoomFactor(WI.settings.zoomFactor.value);






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266070] trunk/Source/WebInspectorUI

2020-08-24 Thread drousso
Title: [266070] trunk/Source/WebInspectorUI








Revision 266070
Author drou...@apple.com
Date 2020-08-24 10:24:52 -0700 (Mon, 24 Aug 2020)


Log Message
Web Inspector: Elements: Styles: don't show swatches for properties that aren't used/apply
https://bugs.webkit.org/show_bug.cgi?id=215681

Reviewed by Brian Burg.

* UserInterface/Views/SpreadsheetStyleProperty.js:
(WI.SpreadsheetStyleProperty.prototype._renderValue):

Modified Paths

trunk/Source/WebInspectorUI/ChangeLog
trunk/Source/WebInspectorUI/UserInterface/Views/SpreadsheetStyleProperty.js




Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (266069 => 266070)

--- trunk/Source/WebInspectorUI/ChangeLog	2020-08-24 17:21:07 UTC (rev 266069)
+++ trunk/Source/WebInspectorUI/ChangeLog	2020-08-24 17:24:52 UTC (rev 266070)
@@ -1,5 +1,15 @@
 2020-08-24  Devin Rousso  
 
+Web Inspector: Elements: Styles: don't show swatches for properties that aren't used/apply
+https://bugs.webkit.org/show_bug.cgi?id=215681
+
+Reviewed by Brian Burg.
+
+* UserInterface/Views/SpreadsheetStyleProperty.js:
+(WI.SpreadsheetStyleProperty.prototype._renderValue):
+
+2020-08-24  Devin Rousso  
+
 Web Inspector: Elements: Styles: don't show non-inherited properties
 https://bugs.webkit.org/show_bug.cgi?id=215682
 


Modified: trunk/Source/WebInspectorUI/UserInterface/Views/SpreadsheetStyleProperty.js (266069 => 266070)

--- trunk/Source/WebInspectorUI/UserInterface/Views/SpreadsheetStyleProperty.js	2020-08-24 17:21:07 UTC (rev 266069)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/SpreadsheetStyleProperty.js	2020-08-24 17:24:52 UTC (rev 266070)
@@ -463,7 +463,7 @@
 const maxValueLength = 150;
 let tokens = WI.tokenizeCSSValue(value);
 
-if (this._property.enabled)
+if (this._property.enabled && !this._property.overridden && this._property.valid && !this._property.hasOtherVendorNameOrKeyword())
 tokens = this._replaceSpecialTokens(tokens);
 
 tokens = tokens.map((token) => {






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266069] trunk/Source/WebInspectorUI

2020-08-24 Thread drousso
Title: [266069] trunk/Source/WebInspectorUI








Revision 266069
Author drou...@apple.com
Date 2020-08-24 10:21:07 -0700 (Mon, 24 Aug 2020)


Log Message
Web Inspector: Elements: Styles: don't show non-inherited properties
https://bugs.webkit.org/show_bug.cgi?id=215682

Reviewed by Brian Burg.

* UserInterface/Views/SpreadsheetCSSStyleDeclarationEditor.js:
(WI.SpreadsheetCSSStyleDeclarationEditor.prototype.get propertiesToRender):

Modified Paths

trunk/Source/WebInspectorUI/ChangeLog
trunk/Source/WebInspectorUI/UserInterface/Views/SpreadsheetCSSStyleDeclarationEditor.js




Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (266068 => 266069)

--- trunk/Source/WebInspectorUI/ChangeLog	2020-08-24 17:17:55 UTC (rev 266068)
+++ trunk/Source/WebInspectorUI/ChangeLog	2020-08-24 17:21:07 UTC (rev 266069)
@@ -1,5 +1,15 @@
 2020-08-24  Devin Rousso  
 
+Web Inspector: Elements: Styles: don't show non-inherited properties
+https://bugs.webkit.org/show_bug.cgi?id=215682
+
+Reviewed by Brian Burg.
+
+* UserInterface/Views/SpreadsheetCSSStyleDeclarationEditor.js:
+(WI.SpreadsheetCSSStyleDeclarationEditor.prototype.get propertiesToRender):
+
+2020-08-24  Devin Rousso  
+
 Web Inspector: Elements: Styles: grey out properties that aren't used/apply
 https://bugs.webkit.org/show_bug.cgi?id=215680
 


Modified: trunk/Source/WebInspectorUI/UserInterface/Views/SpreadsheetCSSStyleDeclarationEditor.js (266068 => 266069)

--- trunk/Source/WebInspectorUI/UserInterface/Views/SpreadsheetCSSStyleDeclarationEditor.js	2020-08-24 17:17:55 UTC (rev 266068)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/SpreadsheetCSSStyleDeclarationEditor.js	2020-08-24 17:21:07 UTC (rev 266069)
@@ -230,6 +230,9 @@
 else
 properties = this._style.properties;
 
+if (this._style.inherited)
+properties = properties.filter((property) => property.inherited);
+
 if (this._sortPropertiesByName)
 properties.sort((a, b) => a.name.extendedLocaleCompare(b.name));
 






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266068] trunk/LayoutTests

2020-08-24 Thread youenn
Title: [266068] trunk/LayoutTests








Revision 266068
Author you...@apple.com
Date 2020-08-24 10:17:55 -0700 (Mon, 24 Aug 2020)


Log Message
Add a test showing the difference of behavior when closing a WebSocket connection between legacy WebSocket and NSURLSession WebSocket code paths
https://bugs.webkit.org/show_bug.cgi?id=215766

Reviewed by Alex Christensen.

Add a test showing the difference of behavior at connection close time in python websocket script.
This behavior was previously covered in http/tests/websocket/tests/hybi/close-on-* tests.
The test is written so that Chrome, Firefox and NSURLSession WebSocket code path generate all PASS.
Legacy WebSocket code path generates one FAIL.

* http/tests/websocket/tests/hybi/close-and-exceptions_wsh.py: Added.
* http/tests/websocket/tests/hybi/close-and-server-script-exception-expected.txt: Added.
* http/tests/websocket/tests/hybi/close-and-server-script-exception.html: Added.
* http/tests/websocket/tests/hybi/resources/close-and-server-script-exception-iframe.html: Added.

Modified Paths

trunk/LayoutTests/ChangeLog


Added Paths

trunk/LayoutTests/http/tests/websocket/tests/hybi/close-and-exceptions_wsh.py
trunk/LayoutTests/http/tests/websocket/tests/hybi/close-and-server-script-exception-expected.txt
trunk/LayoutTests/http/tests/websocket/tests/hybi/close-and-server-script-exception.html
trunk/LayoutTests/http/tests/websocket/tests/hybi/resources/close-and-server-script-exception-iframe.html




Diff

Modified: trunk/LayoutTests/ChangeLog (266067 => 266068)

--- trunk/LayoutTests/ChangeLog	2020-08-24 17:13:04 UTC (rev 266067)
+++ trunk/LayoutTests/ChangeLog	2020-08-24 17:17:55 UTC (rev 266068)
@@ -1,3 +1,20 @@
+2020-08-24  Youenn Fablet  
+
+Add a test showing the difference of behavior when closing a WebSocket connection between legacy WebSocket and NSURLSession WebSocket code paths
+https://bugs.webkit.org/show_bug.cgi?id=215766
+
+Reviewed by Alex Christensen.
+
+Add a test showing the difference of behavior at connection close time in python websocket script.
+This behavior was previously covered in http/tests/websocket/tests/hybi/close-on-* tests.
+The test is written so that Chrome, Firefox and NSURLSession WebSocket code path generate all PASS.
+Legacy WebSocket code path generates one FAIL.
+
+* http/tests/websocket/tests/hybi/close-and-exceptions_wsh.py: Added.
+* http/tests/websocket/tests/hybi/close-and-server-script-exception-expected.txt: Added.
+* http/tests/websocket/tests/hybi/close-and-server-script-exception.html: Added.
+* http/tests/websocket/tests/hybi/resources/close-and-server-script-exception-iframe.html: Added.
+
 2020-08-24  Hector Lopez  
 
 [ iOS wk2 ] http/tests/websocket/tests/hybi/client-close-2.html is a flaky failure


Added: trunk/LayoutTests/http/tests/websocket/tests/hybi/close-and-exceptions_wsh.py (0 => 266068)

--- trunk/LayoutTests/http/tests/websocket/tests/hybi/close-and-exceptions_wsh.py	(rev 0)
+++ trunk/LayoutTests/http/tests/websocket/tests/hybi/close-and-exceptions_wsh.py	2020-08-24 17:17:55 UTC (rev 266068)
@@ -0,0 +1,55 @@
+# Copyright 2009, Google Inc.
+# All rights reserved.
+#
+# Redistribution and use in source and binary forms, with or without
+# modification, are permitted provided that the following conditions are
+# met:
+#
+# * Redistributions of source code must retain the above copyright
+# notice, this list of conditions and the following disclaimer.
+# * Redistributions in binary form must reproduce the above
+# copyright notice, this list of conditions and the following disclaimer
+# in the documentation and/or other materials provided with the
+# distribution.
+# * Neither the name of Google Inc. nor the names of its
+# contributors may be used to endorse or promote products derived from
+# this software without specific prior written permission.
+#
+# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+
+
+from mod_pywebsocket import msgutil
+
+
+connections = {}
+
+
+def web_socket_do_extra_handshake(request):
+pass  # Always accept.
+
+
+def web_socket_transfer_data(request):
+global connections
+connections[request] = 

[webkit-changes] [266067] trunk/LayoutTests

2020-08-24 Thread hector_i_lopez
Title: [266067] trunk/LayoutTests








Revision 266067
Author hector_i_lo...@apple.com
Date 2020-08-24 10:13:04 -0700 (Mon, 24 Aug 2020)


Log Message
[ iOS wk2 ] http/tests/websocket/tests/hybi/client-close-2.html is a flaky failure
https://bugs.webkit.org/show_bug.cgi?id=215773

Unreviewed test gardening.

* platform/ios-wk2/TestExpectations:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/ios-wk2/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (266066 => 266067)

--- trunk/LayoutTests/ChangeLog	2020-08-24 17:11:11 UTC (rev 266066)
+++ trunk/LayoutTests/ChangeLog	2020-08-24 17:13:04 UTC (rev 266067)
@@ -1,5 +1,14 @@
 2020-08-24  Hector Lopez  
 
+[ iOS wk2 ] http/tests/websocket/tests/hybi/client-close-2.html is a flaky failure
+https://bugs.webkit.org/show_bug.cgi?id=215773
+
+Unreviewed test gardening.
+
+* platform/ios-wk2/TestExpectations:
+
+2020-08-24  Hector Lopez  
+
 webkit-test-runner: Add support for the reftest-wait class name
 https://bugs.webkit.org/show_bug.cgi?id=186045
 


Modified: trunk/LayoutTests/platform/ios-wk2/TestExpectations (266066 => 266067)

--- trunk/LayoutTests/platform/ios-wk2/TestExpectations	2020-08-24 17:11:11 UTC (rev 266066)
+++ trunk/LayoutTests/platform/ios-wk2/TestExpectations	2020-08-24 17:13:04 UTC (rev 266067)
@@ -1859,4 +1859,6 @@
 
 webkit.org/b/215583 media/track/track-in-band-style.html [ Pass Timeout ]
 
-webkit.org/b/215706 [ Debug ] imported/w3c/web-platform-tests/html/semantics/scripting-1/the-script-element/moving-between-documents/before-prepare-createHTMLDocument-parse-error-external-classic.html [ Pass Failure ]
\ No newline at end of file
+webkit.org/b/215706 [ Debug ] imported/w3c/web-platform-tests/html/semantics/scripting-1/the-script-element/moving-between-documents/before-prepare-createHTMLDocument-parse-error-external-classic.html [ Pass Failure ]
+
+webkit.org/b/215773 http/tests/websocket/tests/hybi/client-close-2.html [ Pass Failure ]






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266066] trunk/Source/WebInspectorUI

2020-08-24 Thread drousso
Title: [266066] trunk/Source/WebInspectorUI








Revision 266066
Author drou...@apple.com
Date 2020-08-24 10:11:11 -0700 (Mon, 24 Aug 2020)


Log Message
Web Inspector: Elements: Styles: grey out properties that aren't used/apply
https://bugs.webkit.org/show_bug.cgi?id=215680

Reviewed by Brian Burg.

* UserInterface/Views/SpreadsheetCSSStyleDeclarationEditor.css:
(body:not(.meta-key-pressed) .spreadsheet-style-declaration-editor > .property:not(.disabled, .invalid-name, .invalid-value, .other-vendor, .overridden) > .content .name:not(.editing), body.meta-key-pressed .spreadsheet-style-declaration-editor > .property:not(.disabled, .invalid-name, .invalid-value, .other-vendor, .overridden) > .content .name:not(:hover, .editing)): Added.
(body:not(.meta-key-pressed) .spreadsheet-style-declaration-editor > .property:not(.disabled, .invalid-name, .invalid-value, .other-vendor, .overridden) > .content .value:not(.editing), body.meta-key-pressed .spreadsheet-style-declaration-editor > .property:not(.disabled, .invalid-name, .invalid-value, .other-vendor, .overridden) > .content .value:not(:hover, .editing)): Added.
(body:not(.meta-key-pressed) .spreadsheet-style-declaration-editor > .property:not(.disabled):is(.invalid-name, .invalid-value, .other-vendor, .overridden) > .content > .name:not(.editing), body:not(.meta-key-pressed) .spreadsheet-style-declaration-editor > .property:not(.disabled):is(.invalid-name, .invalid-value, .other-vendor, .overridden) > .content > .value-container > .value:not(.editing), body.meta-key-pressed .spreadsheet-style-declaration-editor > .property:not(.disabled):is(.invalid-name, .invalid-value, .other-vendor, .overridden) > .content > .name:not(:hover, .editing), body.meta-key-pressed .spreadsheet-style-declaration-editor > .property:not(.disabled):is(.invalid-name, .invalid-value, .other-vendor, .overridden) > .content > .value-container > .value:not(:hover, .editing)): Added.
(body:not(.meta-key-pressed) .spreadsheet-style-declaration-editor > .property:not(.disabled, .invalid-name, .invalid-value, .other-vendor, .overridden) > .content .value:not(.editing) .token-link, body.meta-key-pressed .spreadsheet-style-declaration-editor > .property:not(.disabled, .invalid-name, .invalid-value, .other-vendor, .overridden) > .content .value:not(:hover, .editing) .token-link): Added.
(body:not(.meta-key-pressed) .spreadsheet-style-declaration-editor > .property:not(.disabled, .invalid-name, .invalid-value, .other-vendor, .overridden) > .content .value:not(.editing) .token-string, body.meta-key-pressed .spreadsheet-style-declaration-editor > .property:not(.disabled, .invalid-name, .invalid-value, .other-vendor, .overridden) > .content .value:not(:hover, .editing) .token-string): Added.
(body:not(.meta-key-pressed) .spreadsheet-style-declaration-editor > .property:not(.disabled, .invalid-name, .invalid-value, .other-vendor, .overridden) > .content .value:not(.editing) .token-comment, body.meta-key-pressed .spreadsheet-style-declaration-editor > .property:not(.disabled, .invalid-name, .invalid-value, .other-vendor, .overridden) > .content .value:not(:hover, .editing) .token-comment): Added.
(body:not(.meta-key-pressed) .spreadsheet-style-declaration-editor > .property:not(.disabled) > .content .name:not(.editing), body.meta-key-pressed .spreadsheet-style-declaration-editor > .property:not(.disabled) > .content .name:not(:hover, .editing)): Deleted.
(body:not(.meta-key-pressed) .spreadsheet-style-declaration-editor > .property:not(.disabled) > .content .value:not(.editing), body.meta-key-pressed .spreadsheet-style-declaration-editor > .property:not(.disabled) > .content .value:not(:hover, .editing)): Deleted.
(body:not(.meta-key-pressed) .spreadsheet-style-declaration-editor > .property:not(.disabled) > .content .value:not(.editing) .token-link, body.meta-key-pressed .spreadsheet-style-declaration-editor > .property:not(.disabled) > .content .value:not(:hover, .editing) .token-link): Deleted.
(body:not(.meta-key-pressed) .spreadsheet-style-declaration-editor > .property:not(.disabled) > .content .value:not(.editing) .token-string, body.meta-key-pressed .spreadsheet-style-declaration-editor > .property:not(.disabled) > .content .value:not(:hover, .editing) .token-string): Deleted.
(body:not(.meta-key-pressed) .spreadsheet-style-declaration-editor > .property:not(.disabled) > .content .value:not(.editing) .token-comment, body.meta-key-pressed .spreadsheet-style-declaration-editor > .property:not(.disabled) > .content .value:not(:hover, .editing) .token-comment): Deleted.
Add additional `:not`s to existing rules to ensure that they don't conflict with a new rule
that changes the color of the `.name`/`.value` to grey when the property is not applied/used.

Modified Paths

trunk/Source/WebInspectorUI/ChangeLog
trunk/Source/WebInspectorUI/UserInterface/Views/SpreadsheetCSSStyleDeclarationEditor.css




Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (266065 => 266066)

--- 

[webkit-changes] [266065] trunk/LayoutTests

2020-08-24 Thread hector_i_lopez
Title: [266065] trunk/LayoutTests








Revision 266065
Author hector_i_lo...@apple.com
Date 2020-08-24 10:01:03 -0700 (Mon, 24 Aug 2020)


Log Message
webkit-test-runner: Add support for the reftest-wait class name
https://bugs.webkit.org/show_bug.cgi?id=186045

Unreviewed test gardening.

* platform/ios-wk2/TestExpectations:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/ios-wk2/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (266064 => 266065)

--- trunk/LayoutTests/ChangeLog	2020-08-24 16:51:20 UTC (rev 266064)
+++ trunk/LayoutTests/ChangeLog	2020-08-24 17:01:03 UTC (rev 266065)
@@ -1,3 +1,12 @@
+2020-08-24  Hector Lopez  
+
+webkit-test-runner: Add support for the reftest-wait class name
+https://bugs.webkit.org/show_bug.cgi?id=186045
+
+Unreviewed test gardening.
+
+* platform/ios-wk2/TestExpectations:
+
 2020-08-24  Aditya Keerthi  
 
 [macOS] Show picker for date and datetime-local input types


Modified: trunk/LayoutTests/platform/ios-wk2/TestExpectations (266064 => 266065)

--- trunk/LayoutTests/platform/ios-wk2/TestExpectations	2020-08-24 16:51:20 UTC (rev 266064)
+++ trunk/LayoutTests/platform/ios-wk2/TestExpectations	2020-08-24 17:01:03 UTC (rev 266065)
@@ -1764,7 +1764,7 @@
 
 webkit.org/b/212219 media/track/track-cue-missing.html [ Pass Failure ]
 
-webkit.org/b/186045 [ Release ] imported/w3c/web-platform-tests/css/css-display/display-none-inline-img.html [ Pass Failure ]
+webkit.org/b/186045 [ Release ] imported/w3c/web-platform-tests/css/css-display/display-none-inline-img.html [ Pass ImageOnlyFailure ]
 
 webkit.org/b/212532 http/wpt/service-workers/service-worker-crashing-while-fetching.https.html [ Slow Pass Failure ]
 webkit.org/b/212532 http/wpt/service-workers/service-worker-different-process.https.html [ Slow Pass Failure ]






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266064] trunk/Source/ThirdParty/libwebrtc

2020-08-24 Thread youenn
Title: [266064] trunk/Source/ThirdParty/libwebrtc








Revision 266064
Author you...@apple.com
Date 2020-08-24 09:51:20 -0700 (Mon, 24 Aug 2020)


Log Message
Enable VP9D_SET_LOOP_FILTER_OPT for libvpx vp9 decoder
https://bugs.webkit.org/show_bug.cgi?id=215765


Reviewed by Eric Carlson.

Following https://webrtc-review.googlesource.com/c/src/+/177335 upstream, let's enable VP9D_SET_LOOP_FILTER_OPT for improved performances.

* Source/webrtc/modules/video_coding/codecs/vp9/vp9_impl.cc:

Modified Paths

trunk/Source/ThirdParty/libwebrtc/ChangeLog
trunk/Source/ThirdParty/libwebrtc/Source/webrtc/modules/video_coding/codecs/vp9/vp9_impl.cc




Diff

Modified: trunk/Source/ThirdParty/libwebrtc/ChangeLog (266063 => 266064)

--- trunk/Source/ThirdParty/libwebrtc/ChangeLog	2020-08-24 16:25:10 UTC (rev 266063)
+++ trunk/Source/ThirdParty/libwebrtc/ChangeLog	2020-08-24 16:51:20 UTC (rev 266064)
@@ -1,3 +1,15 @@
+2020-08-24  Youenn Fablet  
+
+Enable VP9D_SET_LOOP_FILTER_OPT for libvpx vp9 decoder
+https://bugs.webkit.org/show_bug.cgi?id=215765
+
+
+Reviewed by Eric Carlson.
+
+Following https://webrtc-review.googlesource.com/c/src/+/177335 upstream, let's enable VP9D_SET_LOOP_FILTER_OPT for improved performances.
+
+* Source/webrtc/modules/video_coding/codecs/vp9/vp9_impl.cc:
+
 2020-08-12  Youenn Fablet  
 
 Enable H264 low latency code path by default for MacOS


Modified: trunk/Source/ThirdParty/libwebrtc/Source/webrtc/modules/video_coding/codecs/vp9/vp9_impl.cc (266063 => 266064)

--- trunk/Source/ThirdParty/libwebrtc/Source/webrtc/modules/video_coding/codecs/vp9/vp9_impl.cc	2020-08-24 16:25:10 UTC (rev 266063)
+++ trunk/Source/ThirdParty/libwebrtc/Source/webrtc/modules/video_coding/codecs/vp9/vp9_impl.cc	2020-08-24 16:51:20 UTC (rev 266064)
@@ -1670,6 +1670,15 @@
   return WEBRTC_VIDEO_CODEC_UNINITIALIZED;
 }
   }
+
+  vpx_codec_err_t status =
+vpx_codec_control(decoder_, VP9D_SET_LOOP_FILTER_OPT, 1);
+  if (status != VPX_CODEC_OK) {
+RTC_LOG(LS_ERROR) << "Failed to enable VP9D_SET_LOOP_FILTER_OPT. "
+  << vpx_codec_error(decoder_);
+return WEBRTC_VIDEO_CODEC_UNINITIALIZED;
+  }
+
   return WEBRTC_VIDEO_CODEC_OK;
 }
 






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266063] trunk

2020-08-24 Thread akeerthi
Title: [266063] trunk








Revision 266063
Author akeer...@apple.com
Date 2020-08-24 09:25:10 -0700 (Mon, 24 Aug 2020)


Log Message
[macOS] Show picker for date and datetime-local input types
https://bugs.webkit.org/show_bug.cgi?id=214946

Reviewed by Darin Adler and Wenson Hsieh.

Source/WebCore:

Date and datetime-local input types require a calendar picker to be presented when activated.
Consequently, BaseChooserOnlyDateAndTimeInputType::handleDOMActivateEvent was modified to
create a DateTimeChooser and display a calendar upon activation. This object is destroyed
when the element is blurred, hiding the calendar.

There is currently no picker UI for month, week, and time input types. As a result,
handleDOMActivateEvent is a no-op on those input types.

Wrote an encoder and decoder for DateTimeChooserParameters, so that the picker can be
created with the correct values.

Tests: fast/forms/date/date-show-hide-picker.html
   fast/forms/datetimelocal/datetimelocal-show-hide-picker.html

* WebCore.xcodeproj/project.pbxproj:
* html/BaseChooserOnlyDateAndTimeInputType.cpp:
(WebCore::BaseChooserOnlyDateAndTimeInputType::handleDOMActivateEvent):
(WebCore::BaseChooserOnlyDateAndTimeInputType::elementDidBlur):
(WebCore::BaseChooserOnlyDateAndTimeInputType::isPresentingAttachedView const):
(WebCore::BaseChooserOnlyDateAndTimeInputType::didChooseValue):
* html/BaseChooserOnlyDateAndTimeInputType.h:
* html/HTMLInputElement.cpp:
* html/MonthInputType.cpp:
(WebCore::MonthInputType::handleDOMActivateEvent):
* html/MonthInputType.h:
* html/TimeInputType.cpp:
(WebCore::TimeInputType::handleDOMActivateEvent):
* html/TimeInputType.h:
* html/WeekInputType.cpp:
(WebCore::WeekInputType::handleDOMActivateEvent):
* html/WeekInputType.h:
* loader/EmptyClients.cpp:
(WebCore::EmptyChromeClient::createDateTimeChooser):
* loader/EmptyClients.h:
* page/Chrome.cpp:
(WebCore::Chrome::createDateTimeChooser):
* page/Chrome.h:
* page/ChromeClient.h:
* platform/DateTimeChooser.h:
* platform/DateTimeChooserClient.h:
* platform/DateTimeChooserParameters.h: Added.
(WebCore::DateTimeChooserParameters::encode const):
(WebCore::DateTimeChooserParameters::decode):

Source/WebKit:

Created WKDateTimePicker as a wrapper around NSDatePicker. The picker is
displayed in its own NSWindow, ensuring the view is always above the page.
WebPageProxy and WKDateTimePicker communicate through WebDateTimePickerMac,
in order for the picker to be initialized with the correct initial, minimum,
and maximum date, and so that the chosen date can be sent back to the
WebProcess.

Added IPC messages to enable communication between the UIProcess and the
WebProcess necessary for showing and hiding the picker.

* Sources.txt:
* SourcesCocoa.txt:
* UIProcess/PageClient.h:
* UIProcess/WebDateTimePicker.cpp: Added.
(WebKit::WebDateTimePicker::WebDateTimePicker):
(WebKit::WebDateTimePicker::~WebDateTimePicker):
(WebKit::WebDateTimePicker::endPicker):
* UIProcess/WebDateTimePicker.h: Added.
* UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::showDateTimePicker):
(WebKit::WebPageProxy::endDateTimePicker):
(WebKit::WebPageProxy::didChooseDate):
(WebKit::WebPageProxy::didEndDateTimePicker):
(WebKit::WebPageProxy::closeOverlayedViews):
* UIProcess/WebPageProxy.h:
* UIProcess/WebPageProxy.messages.in:
* UIProcess/ios/PageClientImplIOS.h:
* UIProcess/ios/PageClientImplIOS.mm:
(WebKit::PageClientImpl::createDateTimePicker):
* UIProcess/mac/PageClientImplMac.h:
* UIProcess/mac/PageClientImplMac.mm:
(WebKit::PageClientImpl::createDateTimePicker):
* UIProcess/mac/WebDateTimePickerMac.h: Added.
* UIProcess/mac/WebDateTimePickerMac.mm: Added.
(WebKit::WebDateTimePickerMac::create):
(WebKit::WebDateTimePickerMac::~WebDateTimePickerMac):
(WebKit::WebDateTimePickerMac::WebDateTimePickerMac):
(WebKit::WebDateTimePickerMac::endPicker):
(WebKit::WebDateTimePickerMac::showDateTimePicker):
(WebKit::WebDateTimePickerMac::didChooseDate):
(-[WKDateTimePickerWindow initWithContentRect:styleMask:backing:defer:]):
(-[WKDateTimePickerWindow canBecomeKeyWindow]):
(-[WKDateTimePickerWindow hasKeyAppearance]):
(-[WKDateTimePickerWindow shadowOptions]):
(-[WKDateTimePicker initWithParams:inView:]):
(-[WKDateTimePicker showPicker:]):
(-[WKDateTimePicker invalidate]):
(-[WKDateTimePicker didChooseDate:]):
(-[WKDateTimePicker dateFormatStringForType:]):
* WebKit.xcodeproj/project.pbxproj:
* WebProcess/WebCoreSupport/WebChromeClient.cpp:
(WebKit::WebChromeClient::createDateTimeChooser):
* WebProcess/WebCoreSupport/WebChromeClient.h:
* WebProcess/WebCoreSupport/WebDateTimeChooser.cpp: Added.
(WebKit::WebDateTimeChooser::WebDateTimeChooser):
(WebKit::WebDateTimeChooser::didChooseDate):
(WebKit::WebDateTimeChooser::didEndChooser):
(WebKit::WebDateTimeChooser::endChooser):
(WebKit::WebDateTimeChooser::showChooser):
* WebProcess/WebCoreSupport/WebDateTimeChooser.h: Added.
* WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::setActiveDateTimeChooser):
(WebKit::WebPage::didChooseDate):

[webkit-changes] [266062] trunk/Source/WebCore

2020-08-24 Thread zalan
Title: [266062] trunk/Source/WebCore








Revision 266062
Author za...@apple.com
Date 2020-08-24 09:22:06 -0700 (Mon, 24 Aug 2020)


Log Message
[LFC][IFC] Rename LineBox::baseline to alignment baseline
https://bugs.webkit.org/show_bug.cgi?id=214784

Reviewed by Antti Koivisto.

See https://www.w3.org/TR/css-inline-3/#alignment-baseline
Use the spec term. This helps to tell the dominant and the alignment baselines apart during line building.

* layout/inlineformatting/InlineFormattingContext.cpp:
(WebCore::Layout::InlineFormattingContext::setDisplayBoxesForLine):
* layout/inlineformatting/InlineLineBox.h:
(WebCore::Layout::LineBox::LineBox):
(WebCore::Layout::LineBox::setAlignmentBaselineIfGreater):
(WebCore::Layout::LineBox::setAscentIfGreater):
(WebCore::Layout::LineBox::alignmentBaseline const):
(WebCore::Layout::LineBox::resetAlignmentBaseline):
(WebCore::Layout::LineBox::setBaselineIfGreater): Deleted.
(WebCore::Layout::LineBox::baseline const): Deleted.
(WebCore::Layout::LineBox::resetBaseline): Deleted.
* layout/inlineformatting/InlineLineBuilder.cpp:
(WebCore::Layout::LineBuilder::close):
(WebCore::Layout::LineBuilder::adjustBaselineAndLineHeight):
* layout/inlineformatting/InlineLineBuilder.h:
(WebCore::Layout::LineBuilder::baseline const):

Modified Paths

trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/layout/inlineformatting/InlineFormattingContext.cpp
trunk/Source/WebCore/layout/inlineformatting/InlineLineBox.h
trunk/Source/WebCore/layout/inlineformatting/InlineLineBuilder.cpp
trunk/Source/WebCore/layout/inlineformatting/InlineLineBuilder.h




Diff

Modified: trunk/Source/WebCore/ChangeLog (266061 => 266062)

--- trunk/Source/WebCore/ChangeLog	2020-08-24 15:50:22 UTC (rev 266061)
+++ trunk/Source/WebCore/ChangeLog	2020-08-24 16:22:06 UTC (rev 266062)
@@ -1,3 +1,30 @@
+2020-08-24  Zalan Bujtas  
+
+[LFC][IFC] Rename LineBox::baseline to alignment baseline
+https://bugs.webkit.org/show_bug.cgi?id=214784
+
+Reviewed by Antti Koivisto.
+
+See https://www.w3.org/TR/css-inline-3/#alignment-baseline
+Use the spec term. This helps to tell the dominant and the alignment baselines apart during line building. 
+
+* layout/inlineformatting/InlineFormattingContext.cpp:
+(WebCore::Layout::InlineFormattingContext::setDisplayBoxesForLine):
+* layout/inlineformatting/InlineLineBox.h:
+(WebCore::Layout::LineBox::LineBox):
+(WebCore::Layout::LineBox::setAlignmentBaselineIfGreater):
+(WebCore::Layout::LineBox::setAscentIfGreater):
+(WebCore::Layout::LineBox::alignmentBaseline const):
+(WebCore::Layout::LineBox::resetAlignmentBaseline):
+(WebCore::Layout::LineBox::setBaselineIfGreater): Deleted.
+(WebCore::Layout::LineBox::baseline const): Deleted.
+(WebCore::Layout::LineBox::resetBaseline): Deleted.
+* layout/inlineformatting/InlineLineBuilder.cpp:
+(WebCore::Layout::LineBuilder::close):
+(WebCore::Layout::LineBuilder::adjustBaselineAndLineHeight):
+* layout/inlineformatting/InlineLineBuilder.h:
+(WebCore::Layout::LineBuilder::baseline const):
+
 2020-08-24  Adrian Perez de Castro  
 
 Non-unified build fixes, late August 2020 edition


Modified: trunk/Source/WebCore/layout/inlineformatting/InlineFormattingContext.cpp (266061 => 266062)

--- trunk/Source/WebCore/layout/inlineformatting/InlineFormattingContext.cpp	2020-08-24 15:50:22 UTC (rev 266061)
+++ trunk/Source/WebCore/layout/inlineformatting/InlineFormattingContext.cpp	2020-08-24 16:22:06 UTC (rev 266062)
@@ -569,7 +569,7 @@
 ASSERT_NOT_REACHED();
 }
 // FIXME: This is where the logical to physical translate should happen.
-inlineContent.lineBoxes.append({ lineBox.logicalRect(), lineBox.scrollableOverflow(), lineInkOverflow, lineBox.baseline() });
+inlineContent.lineBoxes.append({ lineBox.logicalRect(), lineBox.scrollableOverflow(), lineInkOverflow, lineBox.alignmentBaseline() });
 }
 
 void InlineFormattingContext::invalidateFormattingState(const InvalidationState&)


Modified: trunk/Source/WebCore/layout/inlineformatting/InlineLineBox.h (266061 => 266062)

--- trunk/Source/WebCore/layout/inlineformatting/InlineLineBox.h	2020-08-24 15:50:22 UTC (rev 266061)
+++ trunk/Source/WebCore/layout/inlineformatting/InlineLineBox.h	2020-08-24 16:22:06 UTC (rev 266062)
@@ -50,7 +50,7 @@
 AscentAndDescent ascentAndDescent;
 };
 
-LineBox(const Display::InlineRect&, const AscentAndDescent&, InlineLayoutUnit baseline);
+LineBox(const Display::InlineRect&, const AscentAndDescent&, InlineLayoutUnit alignmentBaseline);
 LineBox() = default;
 
 const Display::InlineRect& logicalRect() const { return m_rect; }
@@ -64,12 +64,12 @@
 InlineLayoutUnit logicalWidth() const { return m_rect.width(); }
 InlineLayoutUnit logicalHeight() const { return m_rect.height(); }
 
-// Baseline from line logical top. Note that the baseline does not 

[webkit-changes] [266061] trunk/Source

2020-08-24 Thread aperez
Title: [266061] trunk/Source








Revision 266061
Author ape...@igalia.com
Date 2020-08-24 08:50:22 -0700 (Mon, 24 Aug 2020)


Log Message
Non-unified build fixes, late August 2020 edition
https://bugs.webkit.org/show_bug.cgi?id=215768

Unreviewed build fix.

Source/WebCore:

No new tests needed.


* Modules/webaudio/StereoPannerNode.cpp: Add missing headers AudioNodeInput.h,
AudioNodeOutput.h, and wtf/IsoMallocInlines.h.
* bindings/js/JSHTMLCanvasElementCustom.cpp:
(WebCore::JSHTMLCanvasElement::visitAdditionalChildren):
Add JSC:: namespace to SlotVisitor parameter type.
* dom/SimpleRange.cpp: Add missing ShadowRoot.h header.
* history/CachedPage.cpp: Add missing FrameLoaderClient.h header.

Source/WebKit:


* UIProcess/WebURLSchemeTask.cpp: Add missing SharedBufferDataReference.h header.

Modified Paths

trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/Modules/webaudio/StereoPannerNode.cpp
trunk/Source/WebCore/bindings/js/JSHTMLCanvasElementCustom.cpp
trunk/Source/WebCore/dom/SimpleRange.cpp
trunk/Source/WebCore/history/CachedPage.cpp
trunk/Source/WebKit/ChangeLog
trunk/Source/WebKit/UIProcess/WebURLSchemeTask.cpp




Diff

Modified: trunk/Source/WebCore/ChangeLog (266060 => 266061)

--- trunk/Source/WebCore/ChangeLog	2020-08-24 15:42:26 UTC (rev 266060)
+++ trunk/Source/WebCore/ChangeLog	2020-08-24 15:50:22 UTC (rev 266061)
@@ -1,3 +1,20 @@
+2020-08-24  Adrian Perez de Castro  
+
+Non-unified build fixes, late August 2020 edition
+https://bugs.webkit.org/show_bug.cgi?id=215768
+
+Unreviewed build fix.
+
+No new tests needed.
+
+* Modules/webaudio/StereoPannerNode.cpp: Add missing headers AudioNodeInput.h,
+AudioNodeOutput.h, and wtf/IsoMallocInlines.h.
+* bindings/js/JSHTMLCanvasElementCustom.cpp:
+(WebCore::JSHTMLCanvasElement::visitAdditionalChildren):
+Add JSC:: namespace to SlotVisitor parameter type.
+* dom/SimpleRange.cpp: Add missing ShadowRoot.h header.
+* history/CachedPage.cpp: Add missing FrameLoaderClient.h header.
+
 2020-08-24  Zalan Bujtas  
 
 [LFC][IFC] Use the term 'baseline' to indicate alignment baseline


Modified: trunk/Source/WebCore/Modules/webaudio/StereoPannerNode.cpp (266060 => 266061)

--- trunk/Source/WebCore/Modules/webaudio/StereoPannerNode.cpp	2020-08-24 15:42:26 UTC (rev 266060)
+++ trunk/Source/WebCore/Modules/webaudio/StereoPannerNode.cpp	2020-08-24 15:50:22 UTC (rev 266061)
@@ -30,6 +30,9 @@
 #if ENABLE(WEB_AUDIO)
 
 #include "AudioBus.h"
+#include "AudioNodeInput.h"
+#include "AudioNodeOutput.h"
+#include 
 
 namespace WebCore {
 


Modified: trunk/Source/WebCore/bindings/js/JSHTMLCanvasElementCustom.cpp (266060 => 266061)

--- trunk/Source/WebCore/bindings/js/JSHTMLCanvasElementCustom.cpp	2020-08-24 15:42:26 UTC (rev 266060)
+++ trunk/Source/WebCore/bindings/js/JSHTMLCanvasElementCustom.cpp	2020-08-24 15:50:22 UTC (rev 266061)
@@ -31,7 +31,7 @@
 
 namespace WebCore {
 
-void JSHTMLCanvasElement::visitAdditionalChildren(SlotVisitor& visitor)
+void JSHTMLCanvasElement::visitAdditionalChildren(JSC::SlotVisitor& visitor)
 {
 visitor.addOpaqueRoot(static_cast(()));
 }


Modified: trunk/Source/WebCore/dom/SimpleRange.cpp (266060 => 266061)

--- trunk/Source/WebCore/dom/SimpleRange.cpp	2020-08-24 15:42:26 UTC (rev 266060)
+++ trunk/Source/WebCore/dom/SimpleRange.cpp	2020-08-24 15:50:22 UTC (rev 266061)
@@ -28,6 +28,7 @@
 
 #include "CharacterData.h"
 #include "NodeTraversal.h"
+#include "ShadowRoot.h"
 
 namespace WebCore {
 


Modified: trunk/Source/WebCore/history/CachedPage.cpp (266060 => 266061)

--- trunk/Source/WebCore/history/CachedPage.cpp	2020-08-24 15:42:26 UTC (rev 266060)
+++ trunk/Source/WebCore/history/CachedPage.cpp	2020-08-24 15:50:22 UTC (rev 266061)
@@ -31,6 +31,7 @@
 #include "FocusController.h"
 #include "Frame.h"
 #include "FrameLoader.h"
+#include "FrameLoaderClient.h"
 #include "FrameView.h"
 #include "HistoryController.h"
 #include "HistoryItem.h"


Modified: trunk/Source/WebKit/ChangeLog (266060 => 266061)

--- trunk/Source/WebKit/ChangeLog	2020-08-24 15:42:26 UTC (rev 266060)
+++ trunk/Source/WebKit/ChangeLog	2020-08-24 15:50:22 UTC (rev 266061)
@@ -1,3 +1,12 @@
+2020-08-24  Adrian Perez de Castro  
+
+Non-unified build fixes, late August 2020 edition
+https://bugs.webkit.org/show_bug.cgi?id=215768
+
+Unreviewed build fix.
+
+* UIProcess/WebURLSchemeTask.cpp: Add missing SharedBufferDataReference.h header.
+
 2020-08-24  Youenn Fablet  
 
 Cocoa WebSocketTask should expose WebSocket server extensions


Modified: trunk/Source/WebKit/UIProcess/WebURLSchemeTask.cpp (266060 => 266061)

--- trunk/Source/WebKit/UIProcess/WebURLSchemeTask.cpp	2020-08-24 15:42:26 UTC (rev 266060)
+++ trunk/Source/WebKit/UIProcess/WebURLSchemeTask.cpp	2020-08-24 15:50:22 UTC (rev 266061)
@@ -27,6 +27,7 @@
 #include "WebURLSchemeTask.h"
 
 #include "APIFrameInfo.h"
+#include "SharedBufferDataReference.h"
 #include 

[webkit-changes] [266060] trunk/LayoutTests

2020-08-24 Thread hector_i_lopez
Title: [266060] trunk/LayoutTests








Revision 266060
Author hector_i_lo...@apple.com
Date 2020-08-24 08:42:26 -0700 (Mon, 24 Aug 2020)


Log Message
WebGL conformance: Failures and Timeouts in suite 2.0.0/conformance2
https://bugs.webkit.org/show_bug.cgi?id=189672

Unreviewed test gardening.

* platform/mac/TestExpectations:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/mac/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (266059 => 266060)

--- trunk/LayoutTests/ChangeLog	2020-08-24 15:02:59 UTC (rev 266059)
+++ trunk/LayoutTests/ChangeLog	2020-08-24 15:42:26 UTC (rev 266060)
@@ -1,3 +1,12 @@
+2020-08-24  Hector Lopez  
+
+WebGL conformance: Failures and Timeouts in suite 2.0.0/conformance2
+https://bugs.webkit.org/show_bug.cgi?id=189672
+
+Unreviewed test gardening.
+
+* platform/mac/TestExpectations:
+
 2020-08-24  Youenn Fablet  
 
 http/tests/websocket/tests/hybi/close-on-* tests are not interoperable


Modified: trunk/LayoutTests/platform/mac/TestExpectations (266059 => 266060)

--- trunk/LayoutTests/platform/mac/TestExpectations	2020-08-24 15:02:59 UTC (rev 266059)
+++ trunk/LayoutTests/platform/mac/TestExpectations	2020-08-24 15:42:26 UTC (rev 266060)
@@ -2209,3 +2209,6 @@
 webkit.org/b/215649 accessibility/mac/select-element-selection-with-optgroups.html [ Pass Failure ]
 
 webkit.org/b/215723 fast/repaint/list-item-equal-style-change-no-repaint.html [ Pass Failure ]
+
+webkit.org/b/189672 [ Release ] webgl/2.0.0/conformance2/textures/canvas/tex-2d-rg32f-rg-float.html [ Pass Failure ]
+






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266059] trunk/LayoutTests

2020-08-24 Thread youenn
Title: [266059] trunk/LayoutTests








Revision 266059
Author you...@apple.com
Date 2020-08-24 08:02:59 -0700 (Mon, 24 Aug 2020)


Log Message
http/tests/websocket/tests/hybi/close-on-* tests are not interoperable
https://bugs.webkit.org/show_bug.cgi?id=215692

Reviewed by Darin Adler.

Before the patch, tests were not passing in Chrome, Firefox and NSURLSession WebSocket code path.
This was due to the fact the web socket script is using try/finally and the closure of the connection is not raising an exception
while it is for Chrome, Firefox and NSURLSession WebSocket code path.
Change the WebSocket server script to more reliably send the socket message at closing time of the connection.

* http/tests/websocket/tests/hybi/close-on-navigate-new-location-expected.txt:
* http/tests/websocket/tests/hybi/close-on-navigate-new-location.html:
* http/tests/websocket/tests/hybi/close-on-unload-and-force-gc-expected.txt:
* http/tests/websocket/tests/hybi/close-on-unload-and-force-gc.html:
* http/tests/websocket/tests/hybi/close-on-unload-expected.txt:
* http/tests/websocket/tests/hybi/close-on-unload-reference-in-parent-expected.txt:
* http/tests/websocket/tests/hybi/close-on-unload-reference-in-parent.html:
* http/tests/websocket/tests/hybi/close-on-unload.html:
* http/tests/websocket/tests/hybi/close-on-unload_wsh.py:
(web_socket_transfer_data):
* http/tests/websocket/tests/hybi/send-after-close-on-unload-expected.txt:
* http/tests/websocket/tests/hybi/send-after-close-on-unload.html:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/http/tests/websocket/tests/hybi/close-on-navigate-new-location-expected.txt
trunk/LayoutTests/http/tests/websocket/tests/hybi/close-on-navigate-new-location.html
trunk/LayoutTests/http/tests/websocket/tests/hybi/close-on-unload-and-force-gc-expected.txt
trunk/LayoutTests/http/tests/websocket/tests/hybi/close-on-unload-and-force-gc.html
trunk/LayoutTests/http/tests/websocket/tests/hybi/close-on-unload-expected.txt
trunk/LayoutTests/http/tests/websocket/tests/hybi/close-on-unload-reference-in-parent-expected.txt
trunk/LayoutTests/http/tests/websocket/tests/hybi/close-on-unload-reference-in-parent.html
trunk/LayoutTests/http/tests/websocket/tests/hybi/close-on-unload.html
trunk/LayoutTests/http/tests/websocket/tests/hybi/close-on-unload_wsh.py
trunk/LayoutTests/http/tests/websocket/tests/hybi/send-after-close-on-unload-expected.txt
trunk/LayoutTests/http/tests/websocket/tests/hybi/send-after-close-on-unload.html




Diff

Modified: trunk/LayoutTests/ChangeLog (266058 => 266059)

--- trunk/LayoutTests/ChangeLog	2020-08-24 15:02:47 UTC (rev 266058)
+++ trunk/LayoutTests/ChangeLog	2020-08-24 15:02:59 UTC (rev 266059)
@@ -1,3 +1,28 @@
+2020-08-24  Youenn Fablet  
+
+http/tests/websocket/tests/hybi/close-on-* tests are not interoperable
+https://bugs.webkit.org/show_bug.cgi?id=215692
+
+Reviewed by Darin Adler.
+
+Before the patch, tests were not passing in Chrome, Firefox and NSURLSession WebSocket code path.
+This was due to the fact the web socket script is using try/finally and the closure of the connection is not raising an exception
+while it is for Chrome, Firefox and NSURLSession WebSocket code path.
+Change the WebSocket server script to more reliably send the socket message at closing time of the connection.
+
+* http/tests/websocket/tests/hybi/close-on-navigate-new-location-expected.txt:
+* http/tests/websocket/tests/hybi/close-on-navigate-new-location.html:
+* http/tests/websocket/tests/hybi/close-on-unload-and-force-gc-expected.txt:
+* http/tests/websocket/tests/hybi/close-on-unload-and-force-gc.html:
+* http/tests/websocket/tests/hybi/close-on-unload-expected.txt:
+* http/tests/websocket/tests/hybi/close-on-unload-reference-in-parent-expected.txt:
+* http/tests/websocket/tests/hybi/close-on-unload-reference-in-parent.html:
+* http/tests/websocket/tests/hybi/close-on-unload.html:
+* http/tests/websocket/tests/hybi/close-on-unload_wsh.py:
+(web_socket_transfer_data):
+* http/tests/websocket/tests/hybi/send-after-close-on-unload-expected.txt:
+* http/tests/websocket/tests/hybi/send-after-close-on-unload.html:
+
 2020-08-24  Hector Lopez  
 [ macOS wk1 Debug ] inspector/animation/nameChanged.html is a flaky crash
 https://bugs.webkit.org/show_bug.cgi?id=215767


Modified: trunk/LayoutTests/http/tests/websocket/tests/hybi/close-on-navigate-new-location-expected.txt (266058 => 266059)

--- trunk/LayoutTests/http/tests/websocket/tests/hybi/close-on-navigate-new-location-expected.txt	2020-08-24 15:02:47 UTC (rev 266058)
+++ trunk/LayoutTests/http/tests/websocket/tests/hybi/close-on-navigate-new-location-expected.txt	2020-08-24 15:02:59 UTC (rev 266059)
@@ -5,7 +5,7 @@
 PASS ws on master document is ready.
 PASS insert a iframe, where open ws called 'socket1'
 PASS 'socket1' is sent to the server. navigate 

[webkit-changes] [266058] trunk/LayoutTests

2020-08-24 Thread hector_i_lopez
Title: [266058] trunk/LayoutTests








Revision 266058
Author hector_i_lo...@apple.com
Date 2020-08-24 08:02:47 -0700 (Mon, 24 Aug 2020)


Log Message
[ macOS wk1 Debug ] inspector/animation/nameChanged.html is a flaky crash
https://bugs.webkit.org/show_bug.cgi?id=215767

Unreviewed test gardening.

* platform/mac-wk1/TestExpectations:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/platform/mac-wk1/TestExpectations




Diff

Modified: trunk/LayoutTests/ChangeLog (266057 => 266058)

--- trunk/LayoutTests/ChangeLog	2020-08-24 14:57:58 UTC (rev 266057)
+++ trunk/LayoutTests/ChangeLog	2020-08-24 15:02:47 UTC (rev 266058)
@@ -1,3 +1,11 @@
+2020-08-24  Hector Lopez  
+[ macOS wk1 Debug ] inspector/animation/nameChanged.html is a flaky crash
+https://bugs.webkit.org/show_bug.cgi?id=215767
+
+Unreviewed test gardening.
+
+* platform/mac-wk1/TestExpectations:
+
 2020-08-24  Youenn Fablet  
 
 Cocoa WebSocketTask should expose WebSocket server extensions


Modified: trunk/LayoutTests/platform/mac-wk1/TestExpectations (266057 => 266058)

--- trunk/LayoutTests/platform/mac-wk1/TestExpectations	2020-08-24 14:57:58 UTC (rev 266057)
+++ trunk/LayoutTests/platform/mac-wk1/TestExpectations	2020-08-24 15:02:47 UTC (rev 266058)
@@ -1135,3 +1135,5 @@
 
 webkit.org/b/122021 media/track/track-cue-rendering-mode-changed.html [ Pass Failure ]
 
+webkit.org/b/215767 [ Debug ] inspector/animation/nameChanged.html [ Pass Crash ]
+






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266057] trunk

2020-08-24 Thread youenn
Title: [266057] trunk








Revision 266057
Author you...@apple.com
Date 2020-08-24 07:57:58 -0700 (Mon, 24 Aug 2020)


Log Message
Cocoa WebSocketTask should expose WebSocket server extensions
https://bugs.webkit.org/show_bug.cgi?id=215696

Reviewed by Darin Adler.

Source/WebKit:

Send back to WebProcess the value of server WebSocket extensions.

* NetworkProcess/cocoa/WebSocketTaskCocoa.mm:
(WebKit::WebSocketTask::didConnect):

LayoutTests:

Make the tests agnostic of using the legacy or standard deflate extension.
If using the standard deflate extensions, do not expect subparameters (this matches Chrome, Firefox and NSURLSession code path).

* http/tests/websocket/tests/hybi/deflate-frame-parameter-expected.txt:
* http/tests/websocket/tests/hybi/deflate-frame-parameter.html:
* http/tests/websocket/tests/hybi/extensions-expected.txt:
* http/tests/websocket/tests/hybi/extensions.html:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/http/tests/websocket/tests/hybi/deflate-frame-parameter-expected.txt
trunk/LayoutTests/http/tests/websocket/tests/hybi/deflate-frame-parameter.html
trunk/LayoutTests/http/tests/websocket/tests/hybi/extensions-expected.txt
trunk/LayoutTests/http/tests/websocket/tests/hybi/extensions.html
trunk/Source/WebKit/ChangeLog
trunk/Source/WebKit/NetworkProcess/cocoa/WebSocketTaskCocoa.mm




Diff

Modified: trunk/LayoutTests/ChangeLog (266056 => 266057)

--- trunk/LayoutTests/ChangeLog	2020-08-24 14:50:31 UTC (rev 266056)
+++ trunk/LayoutTests/ChangeLog	2020-08-24 14:57:58 UTC (rev 266057)
@@ -1,3 +1,18 @@
+2020-08-24  Youenn Fablet  
+
+Cocoa WebSocketTask should expose WebSocket server extensions
+https://bugs.webkit.org/show_bug.cgi?id=215696
+
+Reviewed by Darin Adler.
+
+Make the tests agnostic of using the legacy or standard deflate extension.
+If using the standard deflate extensions, do not expect subparameters (this matches Chrome, Firefox and NSURLSession code path).
+
+* http/tests/websocket/tests/hybi/deflate-frame-parameter-expected.txt:
+* http/tests/websocket/tests/hybi/deflate-frame-parameter.html:
+* http/tests/websocket/tests/hybi/extensions-expected.txt:
+* http/tests/websocket/tests/hybi/extensions.html:
+
 2020-08-24  Rob Buis  
 
 Make window.find not default the search string to undefined


Modified: trunk/LayoutTests/http/tests/websocket/tests/hybi/deflate-frame-parameter-expected.txt (266056 => 266057)

--- trunk/LayoutTests/http/tests/websocket/tests/hybi/deflate-frame-parameter-expected.txt	2020-08-24 14:50:31 UTC (rev 266056)
+++ trunk/LayoutTests/http/tests/websocket/tests/hybi/deflate-frame-parameter-expected.txt	2020-08-24 14:57:58 UTC (rev 266057)
@@ -3,23 +3,19 @@
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
 
 Testing query: "max_window_bits=8"
-PASS ws.extensions.search('x-webkit-deflate-frame') != -1 is true
-PASS ws.extensions.search('max_window_bits=8') != -1 is true
+PASS testResult is true
 PASS event.data is firstMessage
 PASS event.data is secondMessage
 onclose() was called.
 PASS closeEvent.wasClean is true
 Testing query: "no_context_takeover"
-PASS ws.extensions.search('x-webkit-deflate-frame') != -1 is true
-PASS ws.extensions.search('no_context_takeover') != -1 is true
+PASS testResult is true
 PASS event.data is firstMessage
 PASS event.data is secondMessage
 onclose() was called.
 PASS closeEvent.wasClean is true
 Testing query: "max_window_bits=8_context_takeover"
-PASS ws.extensions.search('x-webkit-deflate-frame') != -1 is true
-PASS ws.extensions.search('max_window_bits=8') != -1 is true
-PASS ws.extensions.search('no_context_takeover') != -1 is true
+PASS testResult is true
 PASS event.data is firstMessage
 PASS event.data is secondMessage
 onclose() was called.


Modified: trunk/LayoutTests/http/tests/websocket/tests/hybi/deflate-frame-parameter.html (266056 => 266057)

--- trunk/LayoutTests/http/tests/websocket/tests/hybi/deflate-frame-parameter.html	2020-08-24 14:50:31 UTC (rev 266056)
+++ trunk/LayoutTests/http/tests/websocket/tests/hybi/deflate-frame-parameter.html	2020-08-24 14:57:58 UTC (rev 266057)
@@ -38,6 +38,7 @@
 secondMessage += 'a';
 }
 
+let testResult;
 function doTest(queryIndex)
 {
 var query = queries[queryIndex];
@@ -49,10 +50,15 @@
 
 ws._onopen_ = function(event)
 {
-shouldBeTrue("ws.extensions.search('x-webkit-deflate-frame') != -1");
-parameters = query.split('&');
-for (var i = 0; i < parameters.length; ++i)
-shouldBeTrue("ws.extensions.search('" + parameters[i] + "') != -1");
+const isUsingLegacyExtension = ws.extensions.search('x-webkit-deflate-frame') != -1;
+if (isUsingLegacyExtension) {
+testResult = true;
+parameters = query.split('&');
+for (var i = 0; i < parameters.length; ++i)
+testResult = testResult && (ws.extensions.search(parameters[i]) != 

[webkit-changes] [266056] trunk/Source/WebCore

2020-08-24 Thread zalan
Title: [266056] trunk/Source/WebCore








Revision 266056
Author za...@apple.com
Date 2020-08-24 07:50:31 -0700 (Mon, 24 Aug 2020)


Log Message
[LFC][IFC] Use the term 'baseline' to indicate alignment baseline
https://bugs.webkit.org/show_bug.cgi?id=214779

Reviewed by Antti Koivisto.

This patch is in preparation for decoupling the line box and the root inline box geometry.
Inline boxes have ascent/descent pairs, while line boxes have inline boxes and a baseline.

* layout/displaytree/DisplayPainter.cpp:
(WebCore::Display::paintInlineContent):
* layout/inlineformatting/InlineFormattingContext.cpp:
(WebCore::Layout::InlineFormattingContext::setDisplayBoxesForLine):
* layout/inlineformatting/InlineFormattingContextQuirks.cpp:
(WebCore::Layout::InlineFormattingContext::Quirks::lineHeightConstraints const):
* layout/inlineformatting/InlineLineBox.h:
(WebCore::Layout::AscentAndDescent::height const):
(WebCore::Layout::LineBox::ascentAndDescent const):
(WebCore::Layout::LineBox::LineBox):
(WebCore::Layout::LineBox::InlineBox::InlineBox):
(WebCore::Layout::LineBox::setBaselineIfGreater):
(WebCore::Layout::LineBox::setAscentIfGreater):
(WebCore::Layout::LineBox::setDescentIfGreater):
(WebCore::Layout::LineBox::baseline const):
(WebCore::Layout::LineBox::resetBaseline):
(WebCore::Layout::LineBox::InlineBox::Baseline::height const): Deleted.
(WebCore::Layout::LineBox::setBaselineOffsetIfGreater): Deleted.
(WebCore::Layout::LineBox::baselineOffset const): Deleted.
(WebCore::Layout::LineBox::InlineBox::Baseline::Baseline): Deleted.
(WebCore::Layout::LineBox::InlineBox::Baseline::setAscent): Deleted.
(WebCore::Layout::LineBox::InlineBox::Baseline::setDescent): Deleted.
(WebCore::Layout::LineBox::InlineBox::Baseline::reset): Deleted.
(WebCore::Layout::LineBox::InlineBox::Baseline::ascent const): Deleted.
(WebCore::Layout::LineBox::InlineBox::Baseline::descent const): Deleted.
* layout/inlineformatting/InlineLineBuilder.cpp:
(WebCore::Layout::LineBuilder::initialize):
(WebCore::Layout::LineBuilder::alignContentVertically):
(WebCore::Layout::LineBuilder::adjustBaselineAndLineHeight):
(WebCore::Layout::LineBuilder::halfLeadingMetrics):
* layout/inlineformatting/InlineLineBuilder.h:
(WebCore::Layout::LineBuilder::baseline const):
(WebCore::Layout::LineBuilder::baselineOffset const): Deleted.
* layout/integration/LayoutIntegrationLineLayout.cpp:
(WebCore::LayoutIntegration::LineLayout::paint):
* layout/tableformatting/TableFormattingContext.cpp:
(WebCore::Layout::TableFormattingContext::setUsedGeometryForCells):
(WebCore::Layout::TableFormattingContext::setUsedGeometryForRows):
(WebCore::Layout::TableFormattingContext::computeAndDistributeExtraSpace):
* layout/tableformatting/TableFormattingContextGeometry.cpp:
(WebCore::Layout::TableFormattingContext::Geometry::usedBaselineForCell):
* layout/tableformatting/TableGrid.h:
(WebCore::Layout::TableGrid::Row::setBaseline):
(WebCore::Layout::TableGrid::Row::baseline const):
(WebCore::Layout::TableGrid::Cell::setBaseline):
(WebCore::Layout::TableGrid::Cell::baseline const):
(WebCore::Layout::TableGrid::Row::setBaselineOffset): Deleted.
(WebCore::Layout::TableGrid::Row::baselineOffset const): Deleted.
(WebCore::Layout::TableGrid::Cell::setBaselineOffset): Deleted.
(WebCore::Layout::TableGrid::Cell::baselineOffset const): Deleted.
* layout/tableformatting/TableLayout.cpp:
(WebCore::Layout::TableFormattingContext::TableLayout::distributedVerticalSpace):

Modified Paths

trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/layout/displaytree/DisplayPainter.cpp
trunk/Source/WebCore/layout/inlineformatting/InlineFormattingContext.cpp
trunk/Source/WebCore/layout/inlineformatting/InlineFormattingContextQuirks.cpp
trunk/Source/WebCore/layout/inlineformatting/InlineLineBox.h
trunk/Source/WebCore/layout/inlineformatting/InlineLineBuilder.cpp
trunk/Source/WebCore/layout/inlineformatting/InlineLineBuilder.h
trunk/Source/WebCore/layout/integration/LayoutIntegrationLineLayout.cpp
trunk/Source/WebCore/layout/tableformatting/TableFormattingContext.cpp
trunk/Source/WebCore/layout/tableformatting/TableFormattingContextGeometry.cpp
trunk/Source/WebCore/layout/tableformatting/TableGrid.h
trunk/Source/WebCore/layout/tableformatting/TableLayout.cpp




Diff

Modified: trunk/Source/WebCore/ChangeLog (266055 => 266056)

--- trunk/Source/WebCore/ChangeLog	2020-08-24 13:10:57 UTC (rev 266055)
+++ trunk/Source/WebCore/ChangeLog	2020-08-24 14:50:31 UTC (rev 266056)
@@ -1,3 +1,66 @@
+2020-08-24  Zalan Bujtas  
+
+[LFC][IFC] Use the term 'baseline' to indicate alignment baseline
+https://bugs.webkit.org/show_bug.cgi?id=214779
+
+Reviewed by Antti Koivisto.
+
+This patch is in preparation for decoupling the line box and the root inline box geometry.
+Inline boxes have ascent/descent pairs, while line boxes have inline boxes and a baseline. 
+
+* layout/displaytree/DisplayPainter.cpp:
+(WebCore::Display::paintInlineContent):
+* 

[webkit-changes] [266055] trunk

2020-08-24 Thread carlosgc
Title: [266055] trunk








Revision 266055
Author carlo...@webkit.org
Date 2020-08-24 06:10:57 -0700 (Mon, 24 Aug 2020)


Log Message
Unreviewed. Fix GTK4 build

Source/WebCore:

* SourcesGTK.txt:
* platform/gtk/GtkUtilities.cpp:
(WebCore::monitorWorkArea):
* platform/gtk/GtkUtilities.h:
* platform/gtk/PlatformScreenGtk.cpp:
(WebCore::screenAvailableRect):

Source/WebKit:

* PlatformGTK.cmake:
* UIProcess/API/gtk/WebKitAuthenticationDialog.cpp:
(webkitAuthenticationDialogInitialize):
* UIProcess/API/gtk/WebKitScriptDialogImpl.cpp:
(webkitScriptDialogImplConstructed):
(webkit_script_dialog_impl_class_init):
* UIProcess/API/gtk/WebKitWebViewBase.cpp:
(webkitWebViewBaseEvent):
(webkit_web_view_base_class_init):
* UIProcess/API/gtk/WebKitWebViewDialog.cpp:
(webkitWebViewDialogConstructed):
(webkitWebViewDialogSetChild):
* UIProcess/Inspector/gtk/WebKitInspectorWindow.cpp:
(webkit_inspector_window_init):
* UIProcess/gtk/WebDataListSuggestionsDropdownGtk.cpp:
(WebKit::WebDataListSuggestionsDropdownGtk::show):
* UIProcess/gtk/WebPageProxyGtk.cpp:
(WebKit::WebPageProxy::bindAccessibilityTree):
* UIProcess/gtk/WebPopupMenuProxyGtk.cpp:
(WebKit::WebPopupMenuProxyGtk::showPopupMenu):

Tools:

* MiniBrowser/gtk/BrowserSearchBox.c:
* MiniBrowser/gtk/BrowserWindow.c:

Modified Paths

trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/SourcesGTK.txt
trunk/Source/WebCore/platform/gtk/GtkUtilities.cpp
trunk/Source/WebCore/platform/gtk/GtkUtilities.h
trunk/Source/WebCore/platform/gtk/PlatformScreenGtk.cpp
trunk/Source/WebKit/ChangeLog
trunk/Source/WebKit/PlatformGTK.cmake
trunk/Source/WebKit/UIProcess/API/gtk/WebKitAuthenticationDialog.cpp
trunk/Source/WebKit/UIProcess/API/gtk/WebKitScriptDialogImpl.cpp
trunk/Source/WebKit/UIProcess/API/gtk/WebKitWebViewBase.cpp
trunk/Source/WebKit/UIProcess/API/gtk/WebKitWebViewDialog.cpp
trunk/Source/WebKit/UIProcess/Inspector/gtk/WebKitInspectorWindow.cpp
trunk/Source/WebKit/UIProcess/gtk/WebDataListSuggestionsDropdownGtk.cpp
trunk/Source/WebKit/UIProcess/gtk/WebPageProxyGtk.cpp
trunk/Source/WebKit/UIProcess/gtk/WebPopupMenuProxyGtk.cpp
trunk/Tools/ChangeLog
trunk/Tools/MiniBrowser/gtk/BrowserSearchBox.c
trunk/Tools/MiniBrowser/gtk/BrowserWindow.c




Diff

Modified: trunk/Source/WebCore/ChangeLog (266054 => 266055)

--- trunk/Source/WebCore/ChangeLog	2020-08-24 12:29:24 UTC (rev 266054)
+++ trunk/Source/WebCore/ChangeLog	2020-08-24 13:10:57 UTC (rev 266055)
@@ -1,3 +1,14 @@
+2020-08-24  Carlos Garcia Campos  
+
+Unreviewed. Fix GTK4 build
+
+* SourcesGTK.txt:
+* platform/gtk/GtkUtilities.cpp:
+(WebCore::monitorWorkArea):
+* platform/gtk/GtkUtilities.h:
+* platform/gtk/PlatformScreenGtk.cpp:
+(WebCore::screenAvailableRect):
+
 2020-08-24  Rob Buis  
 
 Make window.find not default the search string to undefined


Modified: trunk/Source/WebCore/SourcesGTK.txt (266054 => 266055)

--- trunk/Source/WebCore/SourcesGTK.txt	2020-08-24 12:29:24 UTC (rev 266054)
+++ trunk/Source/WebCore/SourcesGTK.txt	2020-08-24 13:10:57 UTC (rev 266055)
@@ -107,7 +107,7 @@
 platform/gtk/DragDataGtk.cpp
 platform/gtk/DragImageGtk.cpp
 platform/gtk/GRefPtrGtk.cpp
-platform/gtk/GtkUtilities.cpp
+platform/gtk/GtkUtilities.cpp @no-unify
 platform/gtk/LocalizedStringsGtk.cpp
 platform/gtk/PasteboardGtk.cpp
 platform/gtk/PlatformKeyboardEventGtk.cpp


Modified: trunk/Source/WebCore/platform/gtk/GtkUtilities.cpp (266054 => 266055)

--- trunk/Source/WebCore/platform/gtk/GtkUtilities.cpp	2020-08-24 12:29:24 UTC (rev 266054)
+++ trunk/Source/WebCore/platform/gtk/GtkUtilities.cpp	2020-08-24 13:10:57 UTC (rev 266055)
@@ -24,6 +24,10 @@
 #include 
 #include 
 
+#if USE(GTK4) && PLATFORM(X11)
+#include 
+#endif
+
 namespace WebCore {
 
 static IntPoint gtkWindowGetOrigin(GtkWidget* window)
@@ -187,4 +191,20 @@
 return static_cast(0);
 }
 
+void monitorWorkArea(GdkMonitor* monitor, GdkRectangle* area)
+{
+#if USE(GTK4)
+#if PLATFORM(X11)
+if (GDK_IS_X11_MONITOR(monitor)) {
+gdk_x11_monitor_get_workarea(monitor, area);
+return;
+}
+#endif
+
+gdk_monitor_get_geometry(monitor, area);
+#else
+gdk_monitor_get_workarea(monitor, area);
+#endif
+}
+
 } // namespace WebCore


Modified: trunk/Source/WebCore/platform/gtk/GtkUtilities.h (266054 => 266055)

--- trunk/Source/WebCore/platform/gtk/GtkUtilities.h	2020-08-24 12:29:24 UTC (rev 266054)
+++ trunk/Source/WebCore/platform/gtk/GtkUtilities.h	2020-08-24 13:10:57 UTC (rev 266055)
@@ -55,4 +55,6 @@
 WEBCORE_EXPORT GdkDragAction dragOperationToGdkDragActions(OptionSet);
 WEBCORE_EXPORT GdkDragAction dragOperationToSingleGdkDragAction(OptionSet);
 
+void monitorWorkArea(GdkMonitor*, GdkRectangle*);
+
 } // namespace WebCore


Modified: trunk/Source/WebCore/platform/gtk/PlatformScreenGtk.cpp (266054 => 266055)

--- trunk/Source/WebCore/platform/gtk/PlatformScreenGtk.cpp	2020-08-24 12:29:24 UTC (rev 266054)
+++ trunk/Source/WebCore/platform/gtk/PlatformScreenGtk.cpp	2020-08-24 13:10:57 

[webkit-changes] [266054] trunk

2020-08-24 Thread commit-queue
Title: [266054] trunk








Revision 266054
Author commit-qu...@webkit.org
Date 2020-08-24 05:29:24 -0700 (Mon, 24 Aug 2020)


Log Message
Make window.find not default the search string to undefined
https://bugs.webkit.org/show_bug.cgi?id=215757

Patch by Rob Buis  on 2020-08-24
Reviewed by Darin Adler.

Source/WebCore:

Make window.find not default the search string to undefined, instead
use the null string. Before this change window.find() would find a
hit in a page containing the text string "undefined".

Test: fast/text/window-find.html

* page/DOMWindow.idl:

LayoutTests:

Add tests to verify window.find() does not use "undefined"
as text string. This also fixes the logic of the test to
not always output PASS at the end.

* fast/text/window-find.html:

Modified Paths

trunk/LayoutTests/ChangeLog
trunk/LayoutTests/fast/text/window-find.html
trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/page/DOMWindow.idl




Diff

Modified: trunk/LayoutTests/ChangeLog (266053 => 266054)

--- trunk/LayoutTests/ChangeLog	2020-08-24 09:55:13 UTC (rev 266053)
+++ trunk/LayoutTests/ChangeLog	2020-08-24 12:29:24 UTC (rev 266054)
@@ -1,3 +1,16 @@
+2020-08-24  Rob Buis  
+
+Make window.find not default the search string to undefined
+https://bugs.webkit.org/show_bug.cgi?id=215757
+
+Reviewed by Darin Adler.
+
+Add tests to verify window.find() does not use "undefined"
+as text string. This also fixes the logic of the test to
+not always output PASS at the end.
+
+* fast/text/window-find.html:
+
 2020-08-23  Wenson Hsieh  
 
 Programmatic selection of text in a text field causes the highlight overlay to spill out


Modified: trunk/LayoutTests/fast/text/window-find.html (266053 => 266054)

--- trunk/LayoutTests/fast/text/window-find.html	2020-08-24 09:55:13 UTC (rev 266053)
+++ trunk/LayoutTests/fast/text/window-find.html	2020-08-24 12:29:24 UTC (rev 266054)
@@ -5,11 +5,16 @@
 if (window.testRunner)
 testRunner.dumpAsText();
 
+var passed = true;
+
 function fail(s) {
-document.body.innerHTML = "FAIL: " + s;
+document.body.innerHTML += "FAIL: " + s + "";
+passed = false;
 }
 
 function runTest() {
+if (window.find()) fail('found: find with no parameters');
+if (window.find('')) fail('found: empty string');
 if (window.find('nonsense')) fail('found: nonsense');
 if (window.find('nonsense', true)) fail('found: nonsense');
 if (window.find('nonsense', false)) fail('found: nonsense');
@@ -34,7 +39,7 @@
 if (!window.find('for', true, false, true)) fail('not found: for');
 if (window.find('FOR', true, false, true)) fail('found: FOR');
 
-document.body.innerHTML = "This is a test for window.find(). SUCCESS!";
+if (passed) document.body.innerHTML = "This is a test for window.find(). SUCCESS!";
 }
 
 


Modified: trunk/Source/WebCore/ChangeLog (266053 => 266054)

--- trunk/Source/WebCore/ChangeLog	2020-08-24 09:55:13 UTC (rev 266053)
+++ trunk/Source/WebCore/ChangeLog	2020-08-24 12:29:24 UTC (rev 266054)
@@ -1,3 +1,18 @@
+2020-08-24  Rob Buis  
+
+Make window.find not default the search string to undefined
+https://bugs.webkit.org/show_bug.cgi?id=215757
+
+Reviewed by Darin Adler.
+
+Make window.find not default the search string to undefined, instead
+use the null string. Before this change window.find() would find a
+hit in a page containing the text string "undefined".
+
+Test: fast/text/window-find.html
+
+* page/DOMWindow.idl:
+
 2020-08-24  Justin Uberti  
 
 RTCRtpSynchronizationSource.rtpTimestamp is not present


Modified: trunk/Source/WebCore/page/DOMWindow.idl (266053 => 266054)

--- trunk/Source/WebCore/page/DOMWindow.idl	2020-08-24 09:55:13 UTC (rev 266053)
+++ trunk/Source/WebCore/page/DOMWindow.idl	2020-08-24 12:29:24 UTC (rev 266054)
@@ -159,7 +159,7 @@
 [Replaceable, CustomGetter] readonly attribute Event event;
 attribute DOMString defaultStatus;
 [ImplementedAs=defaultStatus] attribute DOMString defaultstatus; // For compatibility with legacy content.
-boolean find(optional DOMString string = "undefined", optional boolean caseSensitive = false, optional boolean backwards = false, optional boolean wrap = false, optional boolean wholeWord = false, optional boolean searchInFrames = false, optional boolean showDialog = false); // FIXME: Using "undefined" as default parameter value is wrong.
+boolean find(optional DOMString string, optional boolean caseSensitive = false, optional boolean backwards = false, optional boolean wrap = false, optional boolean wholeWord = false, optional boolean searchInFrames = false, optional boolean showDialog = false); // FIXME: Using "undefined" as default parameter value is wrong.
 [Replaceable] readonly attribute  boolean offscreenBuffering;
 [Replaceable] readonly attribute long screenLeft;
 [Replaceable] readonly attribute long screenTop;







[webkit-changes] [266053] trunk/JSTests

2020-08-24 Thread ysuzuki
Title: [266053] trunk/JSTests








Revision 266053
Author ysuz...@apple.com
Date 2020-08-24 02:55:13 -0700 (Mon, 24 Aug 2020)


Log Message
Unreviewed, ignore RangeErrors when ICU is too old

Since ICU version is different (very sad), some of tests can throw an error.

* stress/intl-language-tag.js:
(let.shouldThrow):
(let.shouldNotThrow):
(suppressErrors):
(vm.icuVersion):
(shouldThrow): Deleted.
(shouldNotThrow): Deleted.

Modified Paths

trunk/JSTests/ChangeLog
trunk/JSTests/stress/intl-language-tag.js




Diff

Modified: trunk/JSTests/ChangeLog (266052 => 266053)

--- trunk/JSTests/ChangeLog	2020-08-24 09:26:58 UTC (rev 266052)
+++ trunk/JSTests/ChangeLog	2020-08-24 09:55:13 UTC (rev 266053)
@@ -1,3 +1,17 @@
+2020-08-24  Yusuke Suzuki  
+
+Unreviewed, ignore RangeErrors when ICU is too old
+
+Since ICU version is different (very sad), some of tests can throw an error.
+
+* stress/intl-language-tag.js:
+(let.shouldThrow):
+(let.shouldNotThrow):
+(suppressErrors):
+(vm.icuVersion):
+(shouldThrow): Deleted.
+(shouldNotThrow): Deleted.
+
 2020-08-22  Yusuke Suzuki  
 
 [JSC] Implement Intl Language Tag Parser


Modified: trunk/JSTests/stress/intl-language-tag.js (266052 => 266053)

--- trunk/JSTests/stress/intl-language-tag.js	2020-08-24 09:26:58 UTC (rev 266052)
+++ trunk/JSTests/stress/intl-language-tag.js	2020-08-24 09:55:13 UTC (rev 266053)
@@ -31,11 +31,17 @@
 shouldThrow(() => Intl.getCanonicalLocales("en-US-a-xxx"), RangeError);
 shouldThrow(() => Intl.getCanonicalLocales("en-US-a-ok-ok-$"), RangeError);
 shouldNotThrow(() => Intl.getCanonicalLocales("en-US-a-ok-ok"));
-shouldNotThrow(() => Intl.getCanonicalLocales("en-US-0-ok1"));
+if ($vm.icuVersion() < 64)
+shouldThrow(() => Intl.getCanonicalLocales("en-US-0-ok1"), RangeError);
+else
+shouldNotThrow(() => Intl.getCanonicalLocales("en-US-0-ok1"));
 shouldThrow(() => Intl.getCanonicalLocales("en-US-0"), RangeError);
 shouldThrow(() => Intl.getCanonicalLocales("en-US-0-xxx"), RangeError);
 shouldThrow(() => Intl.getCanonicalLocales("en-US-0-ok-ok-$"), RangeError);
-shouldNotThrow(() => Intl.getCanonicalLocales("en-US-0-ok-ok"));
+if ($vm.icuVersion() < 64)
+shouldThrow(() => Intl.getCanonicalLocales("en-US-0-ok-ok"), RangeError);
+else
+shouldNotThrow(() => Intl.getCanonicalLocales("en-US-0-ok-ok"));
 shouldNotThrow(() => Intl.getCanonicalLocales("de"));
 shouldNotThrow(() => Intl.getCanonicalLocales("fr"));
 shouldNotThrow(() => Intl.getCanonicalLocales("ja"));






___
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes


[webkit-changes] [266052] trunk

2020-08-24 Thread commit-queue
Title: [266052] trunk








Revision 266052
Author commit-qu...@webkit.org
Date 2020-08-24 02:26:58 -0700 (Mon, 24 Aug 2020)


Log Message
RTCRtpSynchronizationSource.rtpTimestamp is not present
https://bugs.webkit.org/show_bug.cgi?id=215722

Patch by Justin Uberti  on 2020-08-24
Reviewed by Youenn Fablet.

LayoutTests/imported/w3c:

Updated expectations file to indicate that tests checking for .rtpTimestamp now pass.

* LayoutTests/imported/w3c/web-platform-tests/webrtc/RTCRtpReceiver-getSynchronizationSources.https-expected.txt:

Source/WebCore:

Updated expected results in LayoutTests/imported/w3c/web-platform-tests/webrtc/RTCRtpReceiver-getSynchronizationSources.https-expected.txt.

* Modules/mediastream/RTCRtpContributingSource.idl:
* Modules/mediastream/RTCRtpContributingSource.idl:
* Modules/mediastream/RTCRtpSynchronizationSource.idl:
Minor modification to ensure JSRTCRtpSynchronizationSource.cpp gets regenerated.
* Modules/mediastream/libwebrtc/LibWebRTCRtpReceiverBackend.cpp:
(WebCore::fillRTCRtpContributingSource):

Modified Paths

trunk/LayoutTests/imported/w3c/ChangeLog
trunk/LayoutTests/imported/w3c/web-platform-tests/webrtc/RTCRtpReceiver-getSynchronizationSources.https-expected.txt
trunk/Source/WebCore/ChangeLog
trunk/Source/WebCore/Modules/mediastream/RTCRtpContributingSource.h
trunk/Source/WebCore/Modules/mediastream/RTCRtpContributingSource.idl
trunk/Source/WebCore/Modules/mediastream/RTCRtpSynchronizationSource.idl
trunk/Source/WebCore/Modules/mediastream/libwebrtc/LibWebRTCRtpReceiverBackend.cpp




Diff

Modified: trunk/LayoutTests/imported/w3c/ChangeLog (266051 => 266052)

--- trunk/LayoutTests/imported/w3c/ChangeLog	2020-08-24 01:51:56 UTC (rev 266051)
+++ trunk/LayoutTests/imported/w3c/ChangeLog	2020-08-24 09:26:58 UTC (rev 266052)
@@ -1,3 +1,14 @@
+2020-08-24  Justin Uberti  
+
+RTCRtpSynchronizationSource.rtpTimestamp is not present
+https://bugs.webkit.org/show_bug.cgi?id=215722
+
+Reviewed by Youenn Fablet.
+
+Updated expectations file to indicate that tests checking for .rtpTimestamp now pass.
+
+* LayoutTests/imported/w3c/web-platform-tests/webrtc/RTCRtpReceiver-getSynchronizationSources.https-expected.txt:
+
 2020-08-22  Emilio Cobos Álvarez  
 
 Import css-content tests.


Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/webrtc/RTCRtpReceiver-getSynchronizationSources.https-expected.txt (266051 => 266052)

--- trunk/LayoutTests/imported/w3c/web-platform-tests/webrtc/RTCRtpReceiver-getSynchronizationSources.https-expected.txt	2020-08-24 01:51:56 UTC (rev 266051)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/webrtc/RTCRtpReceiver-getSynchronizationSources.https-expected.txt	2020-08-24 09:26:58 UTC (rev 266052)
@@ -1,13 +1,13 @@
 
 PASS [audio] getSynchronizationSources() eventually returns a non-empty list 
 PASS [audio] RTCRtpSynchronizationSource.timestamp is a number 
-FAIL [audio] RTCRtpSynchronizationSource.rtpTimestamp is a number [0, 2^32-1] assert_equals: expected "number" but got "undefined"
+PASS [audio] RTCRtpSynchronizationSource.rtpTimestamp is a number [0, 2^32-1] 
 PASS [audio] getSynchronizationSources() does not contain SSRCs older than 10 seconds 
 FAIL [audio] RTCRtpSynchronizationSource.timestamp is comparable to performance.timeOrigin + performance.now() assert_true: expected true got false
 PASS [audio] RTCRtpSynchronizationSource.source is a number 
 PASS [video] getSynchronizationSources() eventually returns a non-empty list 
 PASS [video] RTCRtpSynchronizationSource.timestamp is a number 
-FAIL [video] RTCRtpSynchronizationSource.rtpTimestamp is a number [0, 2^32-1] assert_equals: expected "number" but got "undefined"
+PASS [video] RTCRtpSynchronizationSource.rtpTimestamp is a number [0, 2^32-1] 
 PASS [video] getSynchronizationSources() does not contain SSRCs older than 10 seconds 
 FAIL [video] RTCRtpSynchronizationSource.timestamp is comparable to performance.timeOrigin + performance.now() assert_true: expected true got false
 PASS [video] RTCRtpSynchronizationSource.source is a number 


Modified: trunk/Source/WebCore/ChangeLog (266051 => 266052)

--- trunk/Source/WebCore/ChangeLog	2020-08-24 01:51:56 UTC (rev 266051)
+++ trunk/Source/WebCore/ChangeLog	2020-08-24 09:26:58 UTC (rev 266052)
@@ -1,3 +1,19 @@
+2020-08-24  Justin Uberti  
+
+RTCRtpSynchronizationSource.rtpTimestamp is not present
+https://bugs.webkit.org/show_bug.cgi?id=215722
+
+Reviewed by Youenn Fablet.
+
+Updated expected results in LayoutTests/imported/w3c/web-platform-tests/webrtc/RTCRtpReceiver-getSynchronizationSources.https-expected.txt.
+
+* Modules/mediastream/RTCRtpContributingSource.idl:
+* Modules/mediastream/RTCRtpContributingSource.idl:
+* Modules/mediastream/RTCRtpSynchronizationSource.idl:
+Minor modification to ensure JSRTCRtpSynchronizationSource.cpp gets regenerated.
+*