Bug 183060

Summary:

[Curl][WinCairo] Add Public Suffix support to WinCairo

Product:

WebKit

Reporter:

Christopher Reid <chris.reid>

Component:

Platform

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

achristensen, basuke, bfulgham, cgarcia, chris.reid, commit-queue, csaavedra, darin, don.olmstead, ews-watchlist, mcatanzaro, pvollan, webkit-bug-importer, wilander

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none
Patch	ews-watchlist: commit-queue-
Archive of layout-test-results from ews114 for mac-sierra	none
Updated patch	none
Patch	none
Patch	none

Christopher Reid

Reported 2018-02-22 13:43:46 PST

Tracking a FIXME in CookieJarDB.cpp: // FIXME: Need to check that a domain doesn't a set cookie for a tld when wincairo supports PSL

Attachments
Patch (55.47 KB, patch) 2018-04-04 11:28 PDT, Christopher Reid	no flags	Details Formatted Diff Diff
Patch (55.80 KB, patch) 2018-04-04 12:24 PDT, Christopher Reid	ews-watchlist: commit-queue-	Details Formatted Diff Diff
Archive of layout-test-results from ews114 for mac-sierra (3.14 MB, application/zip) 2018-04-04 14:06 PDT, EWS Watchlist	no flags	Details
Updated patch (60.81 KB, patch) 2018-04-19 17:01 PDT, Christopher Reid	no flags	Details Formatted Diff Diff
Patch (35.38 KB, patch) 2018-10-11 18:28 PDT, Christopher Reid	no flags	Details Formatted Diff Diff
Patch (35.35 KB, patch) 2018-10-11 20:36 PDT, Christopher Reid	no flags	Details Formatted Diff Diff
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.

Christopher Reid

Comment 1 2018-04-04 11:28:59 PDT

Created attachment 337196 [details] Patch

EWS Watchlist

Comment 2 2018-04-04 11:30:13 PDT

Attachment 337196 [details] did not pass style-queue: ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:29: Found header this file implements before WebCore config.h. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:98: Code inside a namespace should not be indented. [whitespace/indent] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:43: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:44: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:47: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:47: bytes_consumed is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:49: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:48: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:49: One space before end of line comments [whitespace/comments] [5] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:50: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:51: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:52: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:53: One space before end of line comments [whitespace/comments] [5] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:54: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:55: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:56: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:58: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:59: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:60: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:61: Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons. [readability/comparison_to_zero] [5] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:61: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:63: One line control clauses should not use braces. [whitespace/braces] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:63: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:65: One line control clauses should not use braces. [whitespace/braces] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:65: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:66: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:70: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:71: Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons. [readability/comparison_to_zero] [5] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:71: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:76: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:77: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:82: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:82: return_value is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:83: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:84: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:85: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:86: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:87: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:90: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:91: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:94: One space before end of line comments [whitespace/comments] [5] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:97: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:98: Missing space inside { }. [whitespace/braces] [5] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:108: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:109: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:112: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:114: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:115: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:116: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:117: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:118: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:120: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:121: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:122: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:122: is_last_char_in_label is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:123: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:123: is_match is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:124: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:132: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:134: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:135: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:136: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:137: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:151: is_last_char_in_label is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:152: is_match is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:155: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:156: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:157: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:158: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:159: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:160: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:161: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:162: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:164: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:166: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:168: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:169: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:170: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:171: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:174: Place brace on its own line for function definitions. [whitespace/braces] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:175: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:176: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:177: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:181: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:188: temp_pos is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:195: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:196: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:197: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:200: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:201: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:205: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:206: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:207: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:207: key_length is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:208: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:209: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:210: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:211: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:211: key_end is incorrectly named. Don't use underscores in your identifier names. [readability/naming/undersc ores] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:212: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:214: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:216: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:217: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:218: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:219: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.cpp:222: One space before end of line comments [whitespace/comments] [5] ERROR: Source/WebCore/platform/curl/PublicSuffixCurl.cpp:34: Bad include order. Mixing system and custom headers. [build/include_order] [4] ERROR: Source/WebCore/platform/curl/PublicSuffixCurl.cpp:54: One line control clauses should not use braces. [whitespace/braces] [4] ERROR: Source/WebCore/platform/curl/PublicSuffixCurl.cpp:56: domain_next is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4] ERROR: Source/WebCore/platform/curl/normalize_psl.py:83: expected 2 blank lines, found 1 [pep8/E302] [5] ERROR: Source/WebCore/platform/curl/normalize_psl.py:89: expected 2 blank lines, found 1 [pep8/E302] [5] ERROR: Source/WebCore/platform/curl/normalize_psl.py:102: expected 2 blank lines, found 1 [pep8/E302] [5] ERROR: Source/WebCore/platform/curl/normalize_psl.py:149: expected 2 blank lines, found 1 [pep8/E302] [5] ERROR: Source/WebCore/platform/curl/normalize_psl.py:153: expected 2 blank lines, found 1 [pep8/E302] [5] ERROR: Source/WebCore/platform/curl/normalize_psl.py:172: expected 2 blank lines, found 1 [pep8/E302] [5] ERROR: Source/WebCore/platform/Curl.cmake:7: Alphabetical sorting problem. "platform/curl/LookupStringInFixedSet.cpp" should be before "platform/curl/PublicSuffixCurl.cpp". [list/order] [5] ERROR: Source/WebCore/platform/URL.cpp:440: Missing space inside { }. [whitespace/braces] [5] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:36: One space before end of line comments [whitespace/comments] [5] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:36: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:37: One space before end of line comments [whitespace/comments] [5] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:37: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:38: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:39: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:40: One space before end of line comments [whitespace/comments] [5] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:40: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:41: One space before end of line comments [whitespace/comments] [5] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:41: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:42: One space before end of line comments [whitespace/comments] [5] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:42: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:51: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:57: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:58: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:59: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:59: key_length is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:101: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:102: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:103: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:104: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:105: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:106: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:107: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:108: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:109: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:110: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:111: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:112: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:114: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:115: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:116: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:117: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:118: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:119: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:121: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:123: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:124: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:125: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:126: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:127: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:128: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:129: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:130: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:131: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:132: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:133: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:134: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:135: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:137: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:138: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:139: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:140: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:141: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:142: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:143: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:144: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:145: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:146: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:147: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:148: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:149: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:151: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:152: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:153: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:154: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:154: pos_ is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:156: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:157: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:158: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:158: end_ is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:160: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:161: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:162: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:163: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:163: pos_is_label_character_ is incorrectly named. Don't use underscores in your identifier names. [readability/naming/underscores] [4] ERROR: Source/WebCore/platform/curl/LookupStringInFixedSet.h:166: One space before end of line comments [whitespace/comments] [5] Total errors found: 194 in 15 files If any of these errors are false positives, please file a bug against check-webkit-style.

Christopher Reid

Comment 3 2018-04-04 12:24:49 PDT

Created attachment 337202 [details] Patch

EWS Watchlist

Comment 4 2018-04-04 12:26:54 PDT

Attachment 337202 [details] did not pass style-queue: ERROR: Source/WebCore/platform/curl/PublicSuffixCurl.cpp:39: Bad include order. Mixing system and custom headers. [build/include_order] [4] Total errors found: 1 in 15 files If any of these errors are false positives, please file a bug against check-webkit-style.

Christopher Reid

Comment 5 2018-04-04 12:30:48 PDT

Sorry for that buildbot wall of text. This change uses files from the chromium project. > * platform/curl/LookupStringInFixedSet.cpp: Added. https://cs.chromium.org/chromium/src/net/base/lookup_string_in_fixed_set.cc?q=lookup_string_&sq=package:chromium&l=1 > * platform/curl/LookupStringInFixedSet.h: Added. https://cs.chromium.org/chromium/src/net/base/lookup_string_in_fixed_set.h?q=lookup_st&sq=package:chromium&l=1 > * platform/curl/make_dafsa.py: Added. https://cs.chromium.org/chromium/src/net/tools/dafsa/make_dafsa.py?q=make_dafsa&sq=package:chromium&dr=C&l=421 > * platform/curl/normalize_psl.py: Added. Based on https://cs.chromium.org/chromium/src/net/tools/tld_cleanup/tld_cleanup.cc?q=tld_&sq=package:chromium&l=1 Chromium's BSD license file: https://cs.chromium.org/chromium/src/LICENSE?q=license&sq=package:chromium&l=1 Right now I have the feature enabled but needs a new release of WinCairoRequirements before landing. I was wanting to wait on review feedback first.

Alex Christensen

Comment 6 2018-04-04 13:08:43 PDT

Comment on attachment 337202 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=337202&action=review > Source/WebCore/platform/URL.cpp:440 > + return { }; std::nullopt > Source/WebCore/platform/URL.cpp:441 > + return std::optional<StringView> {StringView(encodedHostName.data(), encodedHostName.size())}; encodedHostName goes out of scope with this. Why not just return an optional<UCharBuffer>? Also, if we know it's ASCII, why don't we just make it an LCharBuffer? Also, appendEncodedHostname should return a std::optional<something> instead of a bool and returning by reference. > Source/WebCore/platform/curl/PublicSuffixCurl.cpp:39 > +#include "PSLDafsa.cpp" Let's not.

EWS Watchlist

Comment 7 2018-04-04 14:06:25 PDT

Comment on attachment 337202 [details] Patch Attachment 337202 [details] did not pass mac-debug-ews (mac): Output: http://webkit-queues.webkit.org/results/7207567 New failing tests: imported/w3c/web-platform-tests/html/browsers/browsing-the-web/history-traversal/window-name-after-cross-origin-aux-frame-navigation.sub.html fast/dom/HTMLAnchorElement/set-href-attribute-prevents-rebase.html imported/w3c/web-platform-tests/html/browsers/browsing-the-web/history-traversal/window-name-after-cross-origin-sub-frame-navigation.sub.html imported/w3c/web-platform-tests/html/browsers/origin/relaxing-the-same-origin-restriction/document_domain_setter.html fast/dom/HTMLAnchorElement/set-href-attribute-hostname.html fast/dom/DOMURL/set-href-attribute-hostname.html imported/w3c/web-platform-tests/url/url-setters.html http/tests/security/window-name-after-cross-origin-main-frame-navigation.html http/tests/dom/set-document-location-hostname-to-accepted-values.html

EWS Watchlist

Comment 8 2018-04-04 14:06:27 PDT

Created attachment 337224 [details] Archive of layout-test-results from ews114 for mac-sierra The attached test failures were seen while running run-webkit-tests on the mac-debug-ews. Bot: ews114 Port: mac-sierra Platform: Mac OS X 10.12.6

Alex Christensen

Comment 9 2018-04-04 15:24:51 PDT

I'm not sure I like introducing all this Chromium-style code into WebKit. I'm not opposed to the use of a DFA of some sort for this, though. There still needs to be some kind of input containing a list of public suffixes, though, right? Where is that here?

Christopher Reid

Comment 10 2018-04-04 16:17:13 PDT

> Comment on attachment 337202 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=337202&action=review > > > Source/WebCore/platform/URL.cpp:440 > > + return { }; > > std::nullopt > Will do. > > Source/WebCore/platform/URL.cpp:441 > > + return std::optional<StringView> {StringView(encodedHostName.data(), encodedHostName.size())}; > > encodedHostName goes out of scope with this. Why not just return an > optional<UCharBuffer>? > Also, if we know it's ASCII, why don't we just make it an LCharBuffer? > Also, appendEncodedHostname should return a std::optional<something> instead > of a bool and returning by reference. Ah right, I should have realized StringView doesn't copy. Thanks for catching that. I'll look at returning LCharBuffers for those functions. > > Source/WebCore/platform/curl/PublicSuffixCurl.cpp:39 > > +#include "PSLDafsa.cpp" > > Let's not. I'll do something else here.

Christopher Reid

Comment 11 2018-04-04 16:26:52 PDT

(In reply to Alex Christensen from comment #9) > I'm not sure I like introducing all this Chromium-style code into WebKit. > I'm not opposed to the use of a DFA of some sort for this, though. > There still needs to be some kind of input containing a list of public suffixes, > though, right? Where is that here? We were planning on packaging it in with WinCairoRequirements. When we move WinCairoRequirements to use vcpkg I feel that the psl file is easier to maintain there. Currently, WinCairoRequirements will package the latest psl file. vcpkg WinCairoRequirements branch: https://github.com/WebKitForWindows/WinCairoRequirements/commit/fa64fbd26d27d2d183e0d75cd99086ce42bcc68f master WinCairoRequirements: https://github.com/WebKitForWindows/WinCairoRequirements/commit/98669bb28d0abd2be1d1ba6b1c1ae2b08b1e4db0

Christopher Reid

Comment 12 2018-04-19 17:01:08 PDT

Created attachment 338381 [details] Updated patch I ended up keeping a return of UCharBuffer instead of an LCharBuffer as there was a case where hostnames over 2048 characters won't get encoded to punycode. I also tried to modernize chromium's lookup code a bit. The dafsa data is now using a static NeverDestroyed vector instead of a raw array. Also this is a bit out of scope from these changes but one thing I noticed is that URL.cpp is still using the deprecated uidna_IDNToASCII for encoding hostnames. Is there a reason it wasn't updated to use IDN2008 yet? I tried it out in trunk and setting window.location.host or window.location.href to a hostname containing ß can resolve to different domains.

Alex Christensen

Comment 13 2018-04-24 10:57:47 PDT

Comment on attachment 338381 [details] Updated patch View in context: https://bugs.webkit.org/attachment.cgi?id=338381&action=review > Source/WebCore/platform/URL.cpp:411 > + return std::optional<UCharBuffer> {buffer}; This should have a WTFMove. > Source/WebCore/platform/URL.cpp:429 > + return std::optional<UCharBuffer> {buffer}; ditto.

Christopher Reid

Comment 14 2018-10-11 18:28:56 PDT

Created attachment 352118 [details] Patch We decided to switch to using libpsl for WinCairo's public suffix implementation now that it compiles on Windows. It is licensed under MIT and uses chromium sources licensed under BSD-3.

Don Olmstead

Comment 15 2018-10-11 19:00:25 PDT

Comment on attachment 352118 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=352118&action=review This is great work! I just think there's an issue with the finding of it since the bot is saying it can't find the library. > Source/cmake/FindLibPSL.cmake:37 > + NAMES libpsl Should this just be psl? I see psl.lib in the WinCairoRequirements.

Christopher Reid

Comment 16 2018-10-11 20:36:38 PDT

Created attachment 352132 [details] Patch WinCairo build fix

Alex Christensen

Comment 17 2018-10-12 18:41:32 PDT

Comment on attachment 352132 [details] Patch Cool!

WebKit Commit Bot

Comment 18 2018-10-15 09:34:05 PDT

Comment on attachment 352132 [details] Patch Clearing flags on attachment: 352132 Committed r237118: <https://trac.webkit.org/changeset/237118>

WebKit Commit Bot

Comment 19 2018-10-15 09:34:07 PDT

All reviewed patches have been landed. Closing bug.

Radar WebKit Bug Importer

Comment 20 2018-10-15 09:36:11 PDT

<rdar://problem/45273475>

Christopher Reid

Comment 21 2018-10-15 11:19:22 PDT

It looks like some of these imported mac test cases are failing on GTK Here's the result of my run: > **FAIL** PublicSuffix.IsPublicSuffix > > ../../Tools/TestWebKitAPI/Tests/WebCore/PublicSuffix.cpp:81 > Value of: isPublicSuffix("COM") > Actual: false > Expected: true > > **FAIL** PublicSuffix.TopPrivatelyControlledDomain > > ../../Tools/TestWebKitAPI/Tests/WebCore/PublicSuffix.cpp:156 > Expected equality of these values: > utf16String(u"\u6803\u6728.jp") > Which is: xn--4pvxs.jp > topPrivatelyControlledDomain(utf16String(u"\u6803\u6728.jp")) > Which is: > > > ../../Tools/TestWebKitAPI/Tests/WebCore/PublicSuffix.cpp:177 > Expected equality of these values: > String("example.com") > Which is: example.com > topPrivatelyControlledDomain("ExamPle.com") > Which is: ExamPle.com > > > ../../Tools/TestWebKitAPI/Tests/WebCore/PublicSuffix.cpp:178 > Expected equality of these values: > String("example.com") > Which is: example.com > topPrivatelyControlledDomain("SUB.dOmain.ExamPle.com") > Which is: ExamPle.com > > > ../../Tools/TestWebKitAPI/Tests/WebCore/PublicSuffix.cpp:179 > Expected equality of these values: > String("localhost") > Which is: localhost > topPrivatelyControlledDomain("localhost") > Which is: > > > ../../Tools/TestWebKitAPI/Tests/WebCore/PublicSuffix.cpp:180 > Expected equality of these values: > String("localhost") > Which is: localhost > topPrivatelyControlledDomain("LocalHost") > Which is: > > > ../../Tools/TestWebKitAPI/Tests/WebCore/PublicSuffix.cpp:181 > Expected equality of these values: > String("åäö") > Which is: Ã¥Ã¤Ã¶ > topPrivatelyControlledDomain("åäö") > Which is: > > > ../../Tools/TestWebKitAPI/Tests/WebCore/PublicSuffix.cpp:182 > Expected equality of these values: > String("ÅÄÖ") > Which is: Ã > Ã > Ã > topPrivatelyControlledDomain("ÅÄÖ") > Which is:

Michael Catanzaro

Comment 22 2018-10-15 11:43:31 PDT

CC: claudio for failing public suffix tests

Claudio Saavedra

Comment 23 2018-10-16 00:02:42 PDT

(In reply to Michael Catanzaro from comment #22) > CC: claudio for failing public suffix tests For libsoup's libpsl-powered public suffix api we need 2.63.1, but currently we're depending on a much older version and jhbuild fetches 2.57.1. When I have some time I'll check if bumping libsoup's version fixes these tests.

Claudio Saavedra

Comment 24 2018-10-29 09:00:45 PDT

(In reply to Christopher Reid from comment #21) > It looks like some of these imported mac test cases are failing on GTK > > Here's the result of my run: > > > **FAIL** PublicSuffix.IsPublicSuffix > > > > ../../Tools/TestWebKitAPI/Tests/WebCore/PublicSuffix.cpp:81 > > Value of: isPublicSuffix("COM") > > Actual: false > > Expected: true This is failing because we don't downcase the string before passing it to libsoup. Libsoup expects it to be downcased (it's not documented anywhere for some reason but so it is). > > > > **FAIL** PublicSuffix.TopPrivatelyControlledDomain > > > > ../../Tools/TestWebKitAPI/Tests/WebCore/PublicSuffix.cpp:156 > > Expected equality of these values: > > utf16String(u"\u6803\u6728.jp") > > Which is: xn--4pvxs.jp > > topPrivatelyControlledDomain(utf16String(u"\u6803\u6728.jp")) > > Which is: 栃木.jp is in the public-domain suffix list, from what I understand this is not a registrable domain (it's a public one). So I think this test is wrong and should be removed. > > > > > > ../../Tools/TestWebKitAPI/Tests/WebCore/PublicSuffix.cpp:177 > > Expected equality of these values: > > String("example.com") > > Which is: example.com > > topPrivatelyControlledDomain("ExamPle.com") > > Which is: ExamPle.com > > Same as first one. > > > > ../../Tools/TestWebKitAPI/Tests/WebCore/PublicSuffix.cpp:178 > > Expected equality of these values: > > String("example.com") > > Which is: example.com > > topPrivatelyControlledDomain("SUB.dOmain.ExamPle.com") > > Which is: ExamPle.com > > > > Ditto. > > ../../Tools/TestWebKitAPI/Tests/WebCore/PublicSuffix.cpp:179 > > Expected equality of these values: > > String("localhost") > > Which is: localhost > > topPrivatelyControlledDomain("localhost") > > Which is: > > > > > > ../../Tools/TestWebKitAPI/Tests/WebCore/PublicSuffix.cpp:180 > > Expected equality of these values: > > String("localhost") > > Which is: localhost > > topPrivatelyControlledDomain("LocalHost") > > Which is: > > > > These two are special-cased in the Mac implementation. We should do the same. > > ../../Tools/TestWebKitAPI/Tests/WebCore/PublicSuffix.cpp:181 > > Expected equality of these values: > > String("åäö") > > Which is: Ã¥Ã¤Ã¶ > > topPrivatelyControlledDomain("åäö") > > Which is: > > > > > > ../../Tools/TestWebKitAPI/Tests/WebCore/PublicSuffix.cpp:182 > > Expected equality of these values: > > String("ÅÄÖ") > > Which is: Ã > > Ã > > Ã > > topPrivatelyControlledDomain("ÅÄÖ") > > Which is: The error checking in the soup implementation is wrong so that's why these fail. Fixing those raises a few more failures. I'll file a new bug for this.

Note You need to log in before you can comment on or make changes to this bug.