Bug 117485

Summary: text-transform: capitalize shouldn't upconvert
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: TextAssignee: Ryosuke Niwa <rniwa>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, benjamin, commit-queue, darin, esprehn+autocc, glenn, msaboff
Priority: P2 Keywords: BlinkMergeCandidate
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Merges the patch none

Description Ryosuke Niwa 2013-06-10 21:23:01 PDT
Consider merging https://chromium.googlesource.com/chromium/blink/+/c5a221d6da2443df0639c01c40aac6040908ec79

In looking at STRING_STATS for Mobile New York Times, I noticed that
text-transform: capitalize is bloating memory by upconverting strings. It's a
small amount of memory, but there's no reason to upconvert the strings.

After this CL, all the strings used by Mobile New York Times are fully
optimized (or at least they will be after
https://codereview.chromium.org/16158008/ lands).
Comment 1 Ryosuke Niwa 2013-06-10 21:36:50 PDT
Created attachment 204282 [details]
Merges the patch
Comment 2 WebKit Commit Bot 2013-06-10 22:40:38 PDT
Comment on attachment 204282 [details]
Merges the patch

Clearing flags on attachment: 204282

Committed r151422: <http://trac.webkit.org/changeset/151422>
Comment 3 WebKit Commit Bot 2013-06-10 22:40:40 PDT
All reviewed patches have been landed.  Closing bug.