RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-26 17:47:44 +00:00
Code Issues Activity Actions

LibWeb: Make Document::set_cookie take a StringView

Browse source 
Enabled by also making the same change to ParsedCookie::parse_cookie :^)
This commit is contained in:
Shannon Booth 2023-09-12 22:11:24 +12:00 • committed by Andreas Kling
parent 642802d339
commit 8ad05fff4a
4 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Userland/Libraries/LibWeb/Cookie/ParsedCookie.cpp
View file

@ -29,7 +29,7 @@ static void on_http_only_attribute(ParsedCookie& parsed_cookie);
static void on_same_site_attribute(ParsedCookie& parsed_cookie, StringView attribute_value);
static Optional<UnixDateTime> parse_date_time(StringView date_string);
Optional<ParsedCookie> parse_cookie(DeprecatedString const& cookie_string)
Optional<ParsedCookie> parse_cookie(StringView cookie_string)
{
// https://tools.ietf.org/html/rfc6265#section-5.2

2
Userland/Libraries/LibWeb/Cookie/ParsedCookie.h
View file

@ -26,7 +26,7 @@ struct ParsedCookie {
bool http_only_attribute_present { false };
};
Optional<ParsedCookie> parse_cookie(DeprecatedString const& cookie_string);
Optional<ParsedCookie> parse_cookie(StringView cookie_string);
}

2
Userland/Libraries/LibWeb/DOM/Document.cpp
View file

@ -1967,7 +1967,7 @@ DeprecatedString Document::cookie(Cookie::Source source)
return {};
}
void Document::set_cookie(DeprecatedString const& cookie_string, Cookie::Source source)
void Document::set_cookie(StringView cookie_string, Cookie::Source source)
{
auto cookie = Cookie::parse_cookie(cookie_string);
if (!cookie.has_value())

2
Userland/Libraries/LibWeb/DOM/Document.h
View file

@ -98,7 +98,7 @@ public:
JS::GCPtr<Selection::Selection> get_selection() const;
DeprecatedString cookie(Cookie::Source = Cookie::Source::NonHttp);
void set_cookie(DeprecatedString const&, Cookie::Source = Cookie::Source::NonHttp);
void set_cookie(StringView, Cookie::Source = Cookie::Source::NonHttp);
DeprecatedString referrer() const;
void set_referrer(DeprecatedString);