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

See Also:

Attachments
Merges the patch (2.83 KB, patch)
2013-06-10 21:36 PDT, Ryosuke Niwa 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

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.