RFR: 8255940: localStorage is null after window.close() [v2]
Kevin Rushforth
kcr at openjdk.java.net
Sat Feb 5 15:13:15 UTC 2022
On Sat, 5 Feb 2022 12:54:43 GMT, Jay Bhaskar <duke at openjdk.java.net> wrote:
>> We must check if localstorage setting is disabled, then return nullptr first.,as below
>> if (!page->settings().localStorageEnabled())
>> return nullptr;
>>
>> and there after the section
>> if (page->isClosing() && m_localStorage)
>> return m_localStorage.get();
>> would become as below
>> if (m_localStorage)
>> return m_localStorage.get();
>
> in recent code base (webkit upgrade)the localstorage is false ,
> @@ -1579,7 +1579,7 @@ NeedsSiteSpecificQuirks:
> WebKit:
> default: true
> WebCore:
> - default: false
> + default: true
>
> So , it needs to be enable also.
1. My point was that if you initially check for `m_localStorage` being non-null, without the check for `isClosing` (see my earlier comment), then you don't need to check it here.
2. Regarding the enabling of local storage in WebCore, are you seeing any problems as a result of not having it enabled?
-------------
PR: https://git.openjdk.java.net/jfx/pull/703
More information about the openjfx-dev
mailing list