On Tue, 1 Dec 2020 03:12:10 GMT, Arun Joseph <ajos...@openjdk.org> wrote:

>> We should use the public_suffix_list.dat file in the JDK instead. Reading 
>> the public_suffix_list.dat file is modified to be similar to 
>> [DomainName.java](https://github.com/openjdk/jdk/blob/master/src/java.base/share/classes/sun/security/util/DomainName.java).
>>  If the file is not present, `isPublicSuffix()` returns `false`, which is 
>> similar to how WebKit ignores the public suffix check when it is disabled.
>> 
>> Test: Run PublicSuffixesTest.java
>
> Arun Joseph has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Concat log warning

Looks good to me

-------------

Marked as reviewed by ghb (Reviewer).

PR: https://git.openjdk.java.net/jfx/pull/324

Reply via email to