mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-04-26 06:18:59 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

LibWeb/HTML: Update editorial spec comments for Storage

Browse source 
See: 8d2829a
This commit is contained in:
Shannon Booth 2025-01-07 10:31:28 +13:00 • committed by Tim Ledbetter
parent 7fee54b34e
commit b08db3dc1e
Notes: github-actions[bot] 2025-01-07 07:07:29 +00:00
1 changed files with 3 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

7
Libraries/LibWeb/HTML/Storage.cpp
View file

@ -144,8 +144,7 @@ WebIDL::ExceptionOr<void> Storage::set_item(String const& key, String const& val
// https://html.spec.whatwg.org/multipage/webstorage.html#dom-storage-removeitem
void Storage::remove_item(String const& key)
{
// 1. If this's map[key] does not exist, then return null.
// FIXME: Return null?
// 1. If this's map[key] does not exist, then return.
auto it = map().find(key);
if (it == map().end())
return;
@ -190,8 +189,8 @@ void Storage::broadcast(Optional<String> const& key, Optional<String> const& old
auto& relevant_global = relevant_global_object(*this);
auto const& this_document = verify_cast<Window>(relevant_global).associated_document();
// 2. Let url be thisDocument's URL.
auto url = this_document.url().to_string();
// 2. Let url be the serialization of thisDocument's URL.
auto url = this_document.url().serialize();
// 3. Let remoteStorages be all Storage objects excluding storage whose:
GC::RootVector<GC::Ref<Storage>> remote_storages(heap());