164469 2016-11-06 19:38:14 -0800 Web Automation: don't disable localStorage for ephemeral sessions that are being controlled by automation 2017-05-18 11:52:42 -0700 1 1 1 Unclassified WebKit Web Inspector WebKit Nightly Build All All RESOLVED CONFIGURATION CHANGED InRadar P2 Normal --- 1 bburg webkit-unassigned andersca bburg beidson darin inspector-bugzilla-changes webkit-bug-importer oldest_to_newest 1248676 0 bburg 2016-11-06 19:38:14 -0800 . 1248683 1 bburg 2016-11-06 20:49:08 -0800 <rdar://problem/28137091> 1248759 2 beidson 2016-11-07 09:15:55 -0800 Ephemeral sessions should *not* use the storage backend that actually stores to disk. So this bug is, by necessity, more than just "don't disable localStorage for ..." We'll need a mode for localStorage to use an in-memory backing store. 1248773 3 bburg 2016-11-07 09:52:31 -0800 (In reply to comment #2) > Ephemeral sessions should *not* use the storage backend that actually stores > to disk. > > So this bug is, by necessity, more than just "don't disable localStorage for > ..." > > We'll need a mode for localStorage to use an in-memory backing store. Does such a backing store already exist? It seems sketchy to me that the file-based backend is hooked up to ephemeral sessions right now, and we rely on this check to prevent code from actually using it. 1248776 4 beidson 2016-11-07 10:11:09 -0800 (In reply to comment #3) > (In reply to comment #2) > > Ephemeral sessions should *not* use the storage backend that actually stores > > to disk. > > > > So this bug is, by necessity, more than just "don't disable localStorage for > > ..." > > > > We'll need a mode for localStorage to use an in-memory backing store. > > Does such a backing store already exist? Yes - sessionStorage uses a per-WebView, in-memory backing store. And that's exactly what we'll want to use for localStorage in a WebView whose storage session is ephemeral. 1310173 5 bburg 2017-05-18 11:52:42 -0700 This bug is no longer needed.