RESOLVED FIXED 223300
Use ASCIILiteral over String for CSSValue::separatorCSSText() to avoid allocation 
https://bugs.webkit.org/show_bug.cgi?id=223300
Summary Use ASCIILiteral over String for CSSValue::separatorCSSText() to avoid alloca...
Tyler Wilcock
Reported 2021-03-16 20:06:20 PDT
See Darin's comment in this bug: https://bugs.webkit.org/show_bug.cgi?id=223205#c8
Attachments
Patch (3.96 KB, patch)
2021-03-16 20:12 PDT, Tyler Wilcock 		no flags
DetailsFormatted DiffDiff
Patch (3.97 KB, patch)
2021-03-17 07:40 PDT, Tyler Wilcock 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Tyler Wilcock
Comment 1 2021-03-16 20:12:33 PDT
Created attachment 423426 [details] Patch
Yusuke Suzuki
Comment 2 2021-03-16 20:31:54 PDT
Comment on attachment 423426 [details] Patch r=me
EWS
Comment 3 2021-03-16 23:36:28 PDT
commit-queue failed to commit attachment 423426 [details] to WebKit repository. To retry, please set cq+ flag again.
Tyler Wilcock
Comment 4 2021-03-17 07:40:04 PDT
Created attachment 423485 [details] Patch
Tyler Wilcock
Comment 5 2021-03-17 09:51:03 PDT
OK, uploaded a new patch with a rebased changelog and another cq?
Darin Adler
Comment 6 2021-03-17 10:05:08 PDT
Comment on attachment 423485 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=423485&action=review > Source/WebCore/css/CSSValue.h:30 > +#include <wtf/text/ASCIILiteral.h> A forward declaration of ASCIILiteral should be added to Forward.h. Then we could remove this include. Not critical to do right now, of course.
EWS
Comment 7 2021-03-17 10:22:48 PDT
Committed r274566: <https://commits.webkit.org/r274566> All reviewed patches have been landed. Closing bug and clearing flags on attachment 423485 [details].
Radar WebKit Bug Importer
Comment 8 2021-03-17 10:23:15 PDT
<rdar://problem/75532517>
Note You need to log in before you can comment on or make changes to this bug.