Title: [277087] trunk
- Revision
- 277087
- Author
- cdu...@apple.com
- Date
- 2021-05-06 07:52:13 -0700 (Thu, 06 May 2021)
Log Message
imported/w3c/web-platform-tests/eventsource/format-utf-8.htm is failing on some platforms
https://bugs.webkit.org/show_bug.cgi?id=225416
<rdar://77529801>
Reviewed by Youenn Fablet.
Source/WebCore:
The HTML specification [1] states that EventSource streams should always be decoded as UTF-8.
However, we had logic in WebKit that was checking the charset of the response and aborting
when the charset is not UTF-8. It looks like we were lucky until now that we would fail to
determine the charset of the response and proceed anyway. However, due to underlying system
changes, in the context of this test, we are now properly detecting the response charset as
"windows-1252" and (incorrectly) aborting. To address the issue, I no longer abort the
connection if the charset is not "utf-8". We do still log a console warning but we proceed
decoding the stream as UTF-8 no matter the response's charset, as per the specification.
[1] https://html.spec.whatwg.org/multipage/server-sent-events.html#event-stream-interpretation
No new tests, covered by existing test that is failing on some platforms.
* page/EventSource.cpp:
(WebCore::EventSource::responseIsValid const):
LayoutTests:
Dump JS Console messages to stderr for this test as it will only get logged on some
platforms.
* TestExpectations:
Modified Paths
Diff
Modified: trunk/LayoutTests/ChangeLog (277086 => 277087)
--- trunk/LayoutTests/ChangeLog 2021-05-06 13:59:33 UTC (rev 277086)
+++ trunk/LayoutTests/ChangeLog 2021-05-06 14:52:13 UTC (rev 277087)
@@ -1,3 +1,16 @@
+2021-05-06 Chris Dumez <cdu...@apple.com>
+
+ imported/w3c/web-platform-tests/eventsource/format-utf-8.htm is failing on some platforms
+ https://bugs.webkit.org/show_bug.cgi?id=225416
+ <rdar://77529801>
+
+ Reviewed by Youenn Fablet.
+
+ Dump JS Console messages to stderr for this test as it will only get logged on some
+ platforms.
+
+ * TestExpectations:
+
2021-05-06 Tim Nguyen <n...@apple.com>
Re-import css/css-flexbox WPT
Modified: trunk/LayoutTests/TestExpectations (277086 => 277087)
--- trunk/LayoutTests/TestExpectations 2021-05-06 13:59:33 UTC (rev 277086)
+++ trunk/LayoutTests/TestExpectations 2021-05-06 14:52:13 UTC (rev 277087)
@@ -387,6 +387,7 @@
imported/w3c/web-platform-tests/server-timing/navigation_timing_idl.https.html [ Skip ]
# Console log lines may appear in a different order so we silence them.
+imported/w3c/web-platform-tests/eventsource/format-utf-8.htm [ DumpJSConsoleLogInStdErr ]
imported/w3c/web-platform-tests/html/browsers/history/the-location-interface/location-protocol-setter-non-broken.html [ DumpJSConsoleLogInStdErr Failure Pass ]
imported/w3c/web-platform-tests/html/browsers/history/the-location-interface/location-protocol-setter-non-broken-weird.html [ DumpJSConsoleLogInStdErr ]
imported/w3c/web-platform-tests/html/browsers/sandboxing/sandbox-disallow-popups.html [ DumpJSConsoleLogInStdErr ]
Modified: trunk/LayoutTests/http/tests/eventsource/eventsource-content-type-charset-expected.txt (277086 => 277087)
--- trunk/LayoutTests/http/tests/eventsource/eventsource-content-type-charset-expected.txt 2021-05-06 13:59:33 UTC (rev 277086)
+++ trunk/LayoutTests/http/tests/eventsource/eventsource-content-type-charset-expected.txt 2021-05-06 14:52:13 UTC (rev 277087)
@@ -1,4 +1,4 @@
-CONSOLE MESSAGE: EventSource's response has a charset ("windows-1251") that is not UTF-8. Aborting the connection.
+CONSOLE MESSAGE: EventSource's response has a charset ("windows-1251") that is not UTF-8. The response will be decoded as UTF-8.
CONSOLE MESSAGE: EventSource's response has a MIME type ("text/event-stream-foobar") that is not "text/event-stream". Aborting the connection.
Test for bug 45372: https://bugs.webkit.org/show_bug.cgi?id=45372
Modified: trunk/LayoutTests/http/tests/eventsource/eventsource-content-type-charset.html (277086 => 277087)
--- trunk/LayoutTests/http/tests/eventsource/eventsource-content-type-charset.html 2021-05-06 13:59:33 UTC (rev 277086)
+++ trunk/LayoutTests/http/tests/eventsource/eventsource-content-type-charset.html 2021-05-06 14:52:13 UTC (rev 277087)
@@ -39,9 +39,9 @@
];
var expectedResultCallback = [ shouldGetMessage,
- shouldFail,
shouldGetMessage,
shouldGetMessage,
+ shouldGetMessage,
shouldFail
];
Modified: trunk/Source/WebCore/ChangeLog (277086 => 277087)
--- trunk/Source/WebCore/ChangeLog 2021-05-06 13:59:33 UTC (rev 277086)
+++ trunk/Source/WebCore/ChangeLog 2021-05-06 14:52:13 UTC (rev 277087)
@@ -1,3 +1,27 @@
+2021-05-06 Chris Dumez <cdu...@apple.com>
+
+ imported/w3c/web-platform-tests/eventsource/format-utf-8.htm is failing on some platforms
+ https://bugs.webkit.org/show_bug.cgi?id=225416
+ <rdar://77529801>
+
+ Reviewed by Youenn Fablet.
+
+ The HTML specification [1] states that EventSource streams should always be decoded as UTF-8.
+ However, we had logic in WebKit that was checking the charset of the response and aborting
+ when the charset is not UTF-8. It looks like we were lucky until now that we would fail to
+ determine the charset of the response and proceed anyway. However, due to underlying system
+ changes, in the context of this test, we are now properly detecting the response charset as
+ "windows-1252" and (incorrectly) aborting. To address the issue, I no longer abort the
+ connection if the charset is not "utf-8". We do still log a console warning but we proceed
+ decoding the stream as UTF-8 no matter the response's charset, as per the specification.
+
+ [1] https://html.spec.whatwg.org/multipage/server-sent-events.html#event-stream-interpretation
+
+ No new tests, covered by existing test that is failing on some platforms.
+
+ * page/EventSource.cpp:
+ (WebCore::EventSource::responseIsValid const):
+
2021-05-06 Martin Robinson <mrobin...@webkit.org>
[css-scroll-snap] Compute proximity information while snapping
Modified: trunk/Source/WebCore/page/EventSource.cpp (277086 => 277087)
--- trunk/Source/WebCore/page/EventSource.cpp 2021-05-06 13:59:33 UTC (rev 277086)
+++ trunk/Source/WebCore/page/EventSource.cpp 2021-05-06 14:52:13 UTC (rev 277087)
@@ -183,13 +183,13 @@
return false;
}
- // If we have a charset, the only allowed value is UTF-8 (case-insensitive).
+ // The specification states we should always decode as UTF-8. If there is a provided charset and it is not UTF-8, then log a warning
+ // message but keep going anyway.
auto& charset = response.textEncodingName();
if (!charset.isEmpty() && !equalLettersIgnoringASCIICase(charset, "utf-8")) {
- auto message = makeString("EventSource's response has a charset (\"", charset, "\") that is not UTF-8. Aborting the connection.");
+ auto message = makeString("EventSource's response has a charset (\"", charset, "\") that is not UTF-8. The response will be decoded as UTF-8.");
// FIXME: Console message would be better with a source code location; where would we get that?
scriptExecutionContext()->addConsoleMessage(MessageSource::JS, MessageLevel::Error, WTFMove(message));
- return false;
}
return true;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
