Summary: | TextAutosizing - Adjust the lowest common ancestor of the cluster's text nodes after the width of the first estimation is found. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Anton Vayvod <avayvod> | ||||||
Component: | Layout and Rendering | Assignee: | Anton Vayvod <avayvod> | ||||||
Status: | RESOLVED WONTFIX | ||||||||
Severity: | Normal | CC: | benjamin, dbates, eric, johnme, ojan.autocc, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | 103627 | ||||||||
Bug Blocks: | 84186 | ||||||||
Attachments: |
|
Description
Anton Vayvod
2012-12-17 10:50:03 PST
Created attachment 179770 [details]
Patch
This patch depends on the patch for bug 103627 Created attachment 181502 [details]
Patch
Update: the code seems good, and in theory it makes the algorithm more correct. However when we tested it on the Alexa 1000 homepages, it made barely any difference in practice. Hence Anton and I decided to put this patch on hold for now to avoid introducing unnecessary code complexity. Comment on attachment 181502 [details]
Patch
Cleaning up old review flags. Given the most recent comment on this one, I don't think this is still looking for a review.
Marking this bug Resolved WontFix because the TEXT_AUTOSIZING feature was removed in <https://trac.webkit.org/changeset/206119> (bug #162167). See bug 84186, comment 32 for more details. |