diff options
author | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-03-11 23:24:17 +0000 |
---|---|---|
committer | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-03-11 23:24:17 +0000 |
commit | 6df853c758ee98981354428c52107545d743fceb (patch) | |
tree | 8537b289bebb6c45baa65b377b7cc09bddb1ed60 /webkit/data | |
parent | 865f447f8aa94000f9669a0d1e24c27d20b0c79c (diff) | |
download | chromium_src-6df853c758ee98981354428c52107545d743fceb.zip chromium_src-6df853c758ee98981354428c52107545d743fceb.tar.gz chromium_src-6df853c758ee98981354428c52107545d743fceb.tar.bz2 |
Fix some issues with text size calculation and message boxes.
First, size calculations for multi-line, character-breaking strings with no prespecified width were returning useless values, and when given pathological input, could take extremely long time.
Second, the flag calculations for text layout were a bit odd. For example, we wouldn't add an ending ellipsis on elided text if it was multi-line, and we wouldn't add an ellipsis on a clipped word if character breaking was off. Fixed these and reordered the calculations to match the order in which the header declares the flags.
Third, message boxes shouldn't need to trim their inputs, or add an ellipsis character, as both of these should now be handled safely by the underlying code.
BUG=34721
TEST=alert("AAAAA") for strings of As that are, say, 4000 characters long should not cause the UI to hang up for noticeable lengths of time
Review URL: http://codereview.chromium.org/810003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@41343 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/data')
0 files changed, 0 insertions, 0 deletions