Summary: | Avoid constructing the string "all" repeatedly in MediaQueryParser | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||
Component: | CSS | Assignee: | Chris Dumez <cdumez> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, darin, ggaren, koivisto, simon.fraser, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Chris Dumez
2018-05-24 14:37:52 PDT
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. 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? |