On Sun, 27 Feb 2022 05:35:23 GMT, Jay Bhaskar <d...@openjdk.java.net> wrote:
>> Issue: The current implementation of DOMWindow ::localStorage(..) returns >> null pointer in case of page is being closed. >> Fix: It should not return nullptr , as per the [w3c web storage >> spec](https://www.w3.org/TR/2016/REC-webstorage-20160419/) >> "User agents should expire data from the local storage areas only for >> security reasons or when requested to do so by the user. User agents should >> always avoid deleting data while a script that could access that data is >> running." > > Jay Bhaskar has updated the pull request incrementally with one additional > commit since the last revision: > > Add Review Changes modules/javafx.web/src/test/java/test/javafx/scene/web/LocalStorageTest.java line 55: > 53: > 54: private static final File LOCAL_STORAGE_DIR = new > File("LocalStorageDir"); > 55: private static final File PRE_LOCKED = new File("zoo_local_storage"); > LOCAL_STORAGE_DIR is used by web engine , to store local data in this case Yes, `LOCAL_STORAGE_DIR` is needed. I only meant that `PRE_LOCKED` was unused. GitHub decided to highlight both lines. ------------- PR: https://git.openjdk.java.net/jfx/pull/703