Bug 117485 - text-transform: capitalize shouldn't upconvert
Summary: text-transform: capitalize shouldn't upconvert
Alias: None
Product: WebKit
Classification: Unclassified
Component: Text (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ryosuke Niwa
Keywords: BlinkMergeCandidate
Depends on:
Reported: 2013-06-10 21:23 PDT by Ryosuke Niwa
Modified: 2013-06-10 22:40 PDT (History)
7 users (show)

See Also:

Merges the patch (2.83 KB, patch)
2013-06-10 21:36 PDT, Ryosuke Niwa
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.