Skip to content

Commit

Permalink
LibWeb/HTML: Remove uneeded FIXME about use of USVString
Browse files Browse the repository at this point in the history
An AK::String works fine for a USVString as a USVString is just a more
strict version of DOMString. Maybe we will have a different String type
for it in the future, but for now using an AK::String is fine and we do
not need this FIXME.
  • Loading branch information
shannonbooth authored and trflynn89 committed Dec 16, 2024
1 parent a5bdc56 commit e5436ce
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Libraries/LibWeb/HTML/ErrorEvent.h
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ namespace Web::HTML {
// https://html.spec.whatwg.org/multipage/webappapis.html#erroreventinit
struct ErrorEventInit : public DOM::EventInit {
String message;
String filename; // FIXME: This should be a USVString.
String filename;
u32 lineno { 0 };
u32 colno { 0 };
JS::Value error { JS::js_null() };
Expand Down

0 comments on commit e5436ce

Please sign in to comment.