LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-10 13:00:29 +03:00
Code Issues Packages Projects Releases Wiki Activity

LibWeb: Let supported_property_names() return Vector<FlyString>

Browse Source 
Ultimately, this API should probably be replaced with something that
updates a cache on relevant DOM mutations instead of regenerating
the list of property names again and again.
This commit is contained in:
Andreas Kling 2023-12-24 20:59:00 +01:00
parent 0178929387
commit 41f56b0df9
Notes: sideshowbarker 2024-07-17 07:06:47 +09:00 
Author: https://github.com/awesomekling
Commit: https://github.com/SerenityOS/serenity/commit/41f56b0df9
Pull-request: https://github.com/SerenityOS/serenity/pull/22422
19 changed files with 42 additions and 48 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
Userland/Libraries/LibWeb/Bindings/LegacyPlatformObject.cpp
View File

@ -126,7 +126,7 @@ WebIDL::ExceptionOr<void> LegacyPlatformObject::invoke_indexed_property_setter(J
WebIDL::ExceptionOr<void> LegacyPlatformObject::invoke_named_property_setter(String const& property_name, JS::Value value)
{
// 1. Let creating be true if P is not a supported property name, and false otherwise.
Vector<String> supported_property_names = this->supported_property_names();
auto supported_property_names = this->supported_property_names();
bool creating = !supported_property_names.contains_slow(property_name);
// FIXME: We do not have this information at this point, so converting the value is left as an exercise to the inheritor of LegacyPlatformObject.
@ -219,8 +219,8 @@ JS::ThrowCompletionOr<bool> LegacyPlatformObject::internal_define_own_property(J
// 1. Let creating be true if P is not a supported property name, and false otherwise.
// NOTE: This is in it's own variable to enforce the type.
Vector<String> supported_property_names = this->supported_property_names();
bool creating = !supported_property_names.contains_slow(MUST(String::from_byte_string(property_name_as_string)));
auto supported_property_names = this->supported_property_names();
bool creating = !supported_property_names.contains_slow(MUST(FlyString::from_deprecated_fly_string(property_name_as_string)));
// 2. If O implements an interface with the [LegacyOverrideBuiltIns] extended attribute or O does not have an own property named P, then:
// NOTE: Own property lookup has to be done manually instead of using Object::has_own_property, as that would use the overridden internal_get_own_property.
@ -348,7 +348,7 @@ JS::ThrowCompletionOr<JS::MarkedVector<JS::Value>> LegacyPlatformObject::interna
// 3. If O supports named properties, then for each P of O’s supported property names that is visible according to the named property visibility algorithm, append P to keys.
if (supports_named_properties()) {
for (auto& named_property : supported_property_names()) {
if (TRY(WebIDL::is_named_property_exposed_on_object({ this }, named_property.to_byte_string())))
if (TRY(WebIDL::is_named_property_exposed_on_object({ this }, named_property.to_deprecated_fly_string())))
keys.append(JS::PrimitiveString::create(vm, named_property));
}
}
@ -381,7 +381,7 @@ WebIDL::ExceptionOr<JS::Value> LegacyPlatformObject::named_item_value(FlyString
return JS::js_undefined();
}
Vector<String> LegacyPlatformObject::supported_property_names() const
Vector<FlyString> LegacyPlatformObject::supported_property_names() const
{
return {};
}

2
Userland/Libraries/LibWeb/Bindings/LegacyPlatformObject.h
View File

@ -37,7 +37,7 @@ public:
virtual WebIDL::ExceptionOr<JS::Value> item_value(size_t index) const;
virtual WebIDL::ExceptionOr<JS::Value> named_item_value(FlyString const& name) const;
virtual Vector<String> supported_property_names() const;
virtual Vector<FlyString> supported_property_names() const;
virtual bool is_supported_property_index(u32) const;
// NOTE: These will crash if you make has_named_property_setter return true but do not override these methods.

6
Userland/Libraries/LibWeb/DOM/HTMLCollection.cpp
View File

@ -99,10 +99,10 @@ Element* HTMLCollection::named_item(FlyString const& name_) const
}
// https://dom.spec.whatwg.org/#ref-for-dfn-supported-property-names
Vector<String> HTMLCollection::supported_property_names() const
Vector<FlyString> HTMLCollection::supported_property_names() const
{
// 1. Let result be an empty list.
Vector<String> result;
Vector<FlyString> result;
// 2. For each element represented by the collection, in tree order:
auto elements = collect_matching_elements();
@ -119,7 +119,7 @@ Vector<String> HTMLCollection::supported_property_names() const
if (auto maybe_name = element->attribute(HTML::AttributeNames::name); maybe_name.has_value()) {
auto name = maybe_name.release_value();
if (!name.is_empty() && !result.contains_slow(name))
result.append(move(name));
result.append(name);
}
}
}

2
Userland/Libraries/LibWeb/DOM/HTMLCollection.h
View File

@ -46,7 +46,7 @@ public:
virtual WebIDL::ExceptionOr<JS::Value> item_value(size_t index) const override;
virtual WebIDL::ExceptionOr<JS::Value> named_item_value(FlyString const& name) const override;
virtual Vector<String> supported_property_names() const override;
virtual Vector<FlyString> supported_property_names() const override;
virtual bool is_supported_property_index(u32) const override;
protected:

4
Userland/Libraries/LibWeb/DOM/NamedNodeMap.cpp
View File

@ -49,10 +49,10 @@ bool NamedNodeMap::is_supported_property_index(u32 index) const
}
// https://dom.spec.whatwg.org/#ref-for-dfn-supported-property-names%E2%91%A0
Vector<String> NamedNodeMap::supported_property_names() const
Vector<FlyString> NamedNodeMap::supported_property_names() const
{
// 1. Let names be the qualified names of the attributes in this NamedNodeMap object’s attribute list, with duplicates omitted, in order.
Vector<String> names;
Vector<FlyString> names;
names.ensure_capacity(m_attributes.size());
for (auto const& attribute : m_attributes) {

2
Userland/Libraries/LibWeb/DOM/NamedNodeMap.h
View File

@ -26,7 +26,7 @@ public:
~NamedNodeMap() = default;
virtual bool is_supported_property_index(u32 index) const override;
virtual Vector<String> supported_property_names() const override;
virtual Vector<FlyString> supported_property_names() const override;
virtual WebIDL::ExceptionOr<JS::Value> item_value(size_t index) const override;
virtual WebIDL::ExceptionOr<JS::Value> named_item_value(FlyString const& name) const override;

6
Userland/Libraries/LibWeb/HTML/DOMStringMap.cpp
View File

@ -91,13 +91,13 @@ Vector<DOMStringMap::NameValuePair> DOMStringMap::get_name_value_pairs() const
// https://html.spec.whatwg.org/multipage/dom.html#concept-domstringmap-pairs
// NOTE: There isn't a direct link to this, so the link is to one of the algorithms above it.
Vector<String> DOMStringMap::supported_property_names() const
Vector<FlyString> DOMStringMap::supported_property_names() const
{
// The supported property names on a DOMStringMap object at any instant are the names of each pair returned from getting the DOMStringMap's name-value pairs at that instant, in the order returned.
Vector<String> names;
Vector<FlyString> names;
auto name_value_pairs = get_name_value_pairs();
for (auto& name_value_pair : name_value_pairs) {
names.append(name_value_pair.name.to_string());
names.append(name_value_pair.name);
}
return names;
}

2
Userland/Libraries/LibWeb/HTML/DOMStringMap.h
View File

@ -37,7 +37,7 @@ private:
// ^LegacyPlatformObject
virtual WebIDL::ExceptionOr<JS::Value> named_item_value(FlyString const&) const override;
virtual Vector<String> supported_property_names() const override;
virtual Vector<FlyString> supported_property_names() const override;
virtual bool supports_indexed_properties() const override { return false; }
virtual bool supports_named_properties() const override { return true; }

8
Userland/Libraries/LibWeb/HTML/MimeTypeArray.cpp
View File

@ -28,7 +28,7 @@ void MimeTypeArray::initialize(JS::Realm& realm)
}
// https://html.spec.whatwg.org/multipage/system-state.html#pdf-viewing-support:support-named-properties-2
Vector<String> MimeTypeArray::supported_property_names() const
Vector<FlyString> MimeTypeArray::supported_property_names() const
{
// The MimeTypeArray interface supports named properties. If the user agent's PDF viewer supported is true, then they are the PDF viewer mime types. Otherwise, they are the empty list.
auto const& window = verify_cast<HTML::Window>(HTML::relevant_global_object(*this));
@ -36,9 +36,9 @@ Vector<String> MimeTypeArray::supported_property_names() const
return {};
// https://html.spec.whatwg.org/multipage/system-state.html#pdf-viewer-mime-types
static Vector<String> const mime_types = {
"application/pdf"_string,
"text/pdf"_string,
static Vector<FlyString> const mime_types = {
"application/pdf"_fly_string,
"text/pdf"_fly_string,
};
return mime_types;

2
Userland/Libraries/LibWeb/HTML/MimeTypeArray.h
View File

@ -28,7 +28,7 @@ private:
virtual void initialize(JS::Realm&) override;
// ^Bindings::LegacyPlatformObject
virtual Vector<String> supported_property_names() const override;
virtual Vector<FlyString> supported_property_names() const override;
virtual WebIDL::ExceptionOr<JS::Value> item_value(size_t index) const override;
virtual WebIDL::ExceptionOr<JS::Value> named_item_value(FlyString const& name) const override;
virtual bool is_supported_property_index(u32) const override;

8
Userland/Libraries/LibWeb/HTML/Plugin.cpp
View File

@ -52,7 +52,7 @@ String Plugin::filename() const
}
// https://html.spec.whatwg.org/multipage/system-state.html#pdf-viewing-support:support-named-properties-3
Vector<String> Plugin::supported_property_names() const
Vector<FlyString> Plugin::supported_property_names() const
{
// The Plugin interface supports named properties. If the user agent's PDF viewer supported is true, then they are the PDF viewer mime types. Otherwise, they are the empty list.
auto const& window = verify_cast<HTML::Window>(HTML::relevant_global_object(*this));
@ -60,9 +60,9 @@ Vector<String> Plugin::supported_property_names() const
return {};
// https://html.spec.whatwg.org/multipage/system-state.html#pdf-viewer-mime-types
static Vector<String> const mime_types = {
"application/pdf"_string,
"text/pdf"_string,
static Vector<FlyString> const mime_types = {
"application/pdf"_fly_string,
"text/pdf"_fly_string,
};
return mime_types;

2
Userland/Libraries/LibWeb/HTML/Plugin.h
View File

@ -34,7 +34,7 @@ private:
virtual void initialize(JS::Realm&) override;
// ^Bindings::LegacyPlatformObject
virtual Vector<String> supported_property_names() const override;
virtual Vector<FlyString> supported_property_names() const override;
virtual WebIDL::ExceptionOr<JS::Value> item_value(size_t index) const override;
virtual WebIDL::ExceptionOr<JS::Value> named_item_value(FlyString const& name) const override;
virtual bool is_supported_property_index(u32) const override;

14
Userland/Libraries/LibWeb/HTML/PluginArray.cpp
View File

@ -34,7 +34,7 @@ void PluginArray::refresh() const
}
// https://html.spec.whatwg.org/multipage/system-state.html#pdf-viewing-support:support-named-properties
Vector<String> PluginArray::supported_property_names() const
Vector<FlyString> PluginArray::supported_property_names() const
{
// The PluginArray interface supports named properties. If the user agent's PDF viewer supported is true, then they are the PDF viewer plugin names. Otherwise, they are the empty list.
auto const& window = verify_cast<HTML::Window>(HTML::relevant_global_object(*this));
@ -42,12 +42,12 @@ Vector<String> PluginArray::supported_property_names() const
return {};
// https://html.spec.whatwg.org/multipage/system-state.html#pdf-viewer-plugin-names
static Vector<String> const plugin_names = {
"PDF Viewer"_string,
"Chrome PDF Viewer"_string,
"Chromium PDF Viewer"_string,
"Microsoft Edge PDF Viewer"_string,
"WebKit built-in PDF"_string,
static Vector<FlyString> const plugin_names = {
"PDF Viewer"_fly_string,
"Chrome PDF Viewer"_fly_string,
"Chromium PDF Viewer"_fly_string,
"Microsoft Edge PDF Viewer"_fly_string,
"WebKit built-in PDF"_fly_string,
};
return plugin_names;

2
Userland/Libraries/LibWeb/HTML/PluginArray.h
View File

@ -29,7 +29,7 @@ private:
virtual void initialize(JS::Realm&) override;
// ^Bindings::LegacyPlatformObject
virtual Vector<String> supported_property_names() const override;
virtual Vector<FlyString> supported_property_names() const override;
virtual WebIDL::ExceptionOr<JS::Value> item_value(size_t index) const override;
virtual WebIDL::ExceptionOr<JS::Value> named_item_value(FlyString const& name) const override;
virtual bool is_supported_property_index(u32) const override;

4
Userland/Libraries/LibWeb/HTML/Storage.cpp
View File

@ -149,10 +149,10 @@ void Storage::broadcast(StringView key, StringView old_value, StringView new_val
// FIXME: Implement.
}
Vector<String> Storage::supported_property_names() const
Vector<FlyString> Storage::supported_property_names() const
{
// The supported property names on a Storage object storage are the result of running get the keys on storage's map.
Vector<String> names;
Vector<FlyString> names;
names.ensure_capacity(m_map.size());
for (auto const& key : m_map.keys())
names.unchecked_append(key);

2
Userland/Libraries/LibWeb/HTML/Storage.h
View File

@ -40,7 +40,7 @@ private:
// ^LegacyPlatformObject
virtual WebIDL::ExceptionOr<JS::Value> named_item_value(FlyString const&) const override;
virtual WebIDL::ExceptionOr<DidDeletionFail> delete_value(String const&) override;
virtual Vector<String> supported_property_names() const override;
virtual Vector<FlyString> supported_property_names() const override;
virtual WebIDL::ExceptionOr<void> set_value_of_named_property(String const& key, JS::Value value) override;
virtual bool supports_indexed_properties() const override { return false; }

10
Userland/Libraries/LibWeb/HTML/Window.cpp
View File

@ -1548,7 +1548,7 @@ OrderedHashMap<String, JS::NonnullGCPtr<Navigable>> Window::document_tree_child_
}
// https://html.spec.whatwg.org/#named-access-on-the-window-object
Vector<String> Window::supported_property_names()
Vector<FlyString> Window::supported_property_names()
{
// The Window object supports named properties.
// The supported property names of a Window object window at any moment consist of the following,
@ -1575,13 +1575,7 @@ Vector<String> Window::supported_property_names()
return IterationDecision::Continue;
});
Vector<String> names;
names.ensure_capacity(property_names.size());
for (auto const& name : property_names) {
names.append(name.to_string());
}
return names;
return property_names.values();
}
// https://html.spec.whatwg.org/#named-access-on-the-window-object

2
Userland/Libraries/LibWeb/HTML/Window.h
View File

@ -203,7 +203,7 @@ public:
[[nodiscard]] OrderedHashMap<String, JS::NonnullGCPtr<Navigable>> document_tree_child_navigable_target_name_property_set();
[[nodiscard]] Vector<String> supported_property_names();
[[nodiscard]] Vector<FlyString> supported_property_names();
[[nodiscard]] WebIDL::ExceptionOr<JS::Value> named_item_value(FlyString const&);
private:

2
Userland/Libraries/LibWeb/WebIDL/AbstractOperations.cpp
View File

@ -317,7 +317,7 @@ JS::ThrowCompletionOr<bool> is_named_property_exposed_on_object(Variant<Bindings
// 1. If P is not a supported property name of O, then return false.
// NOTE: This is in it's own variable to enforce the type.
Vector<String> supported_property_names = variant.visit([](auto* o) { return o->supported_property_names(); });
auto supported_property_names = variant.visit([](auto* o) { return o->supported_property_names(); });
auto property_key_string = MUST(String::from_byte_string(property_key.to_string()));
if (!supported_property_names.contains_slow(property_key_string))
return false;