Avoid constructing the string "all" repeatedly in MediaQueryParser.
Created attachment 341225 [details] Patch
Comment on attachment 341225 [details] Patch Maybe we should use AtomicStrings for media types?
Comment on attachment 341225 [details] Patch Clearing flags on attachment: 341225 Committed r232173: <https://trac.webkit.org/changeset/232173>
All reviewed patches have been landed. Closing bug.
<rdar://problem/40540318>
This change is OK, but I wonder why we need std::optional<String> since String already has distinct null and empty values. Maybe use a null string instead of std::nullopt?