Bug 105196

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 RenderingAssignee: 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 Flags
Patch
none
Patch none

Anton Vayvod
Reported 2012-12-17 10:50:03 PST
TextAutosizing - Adjust the lowest common ancestor of the cluster's text nodes after the width of the first estimation is found.
Attachments
Patch (10.55 KB, patch)
2012-12-17 10:58 PST, Anton Vayvod 		no flags
DetailsFormatted DiffDiff
Patch (19.18 KB, patch)
2013-01-07 06:49 PST, Anton Vayvod 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Anton Vayvod
Comment 1 2012-12-17 10:58:54 PST
Created attachment 179770 [details] Patch
Anton Vayvod
Comment 2 2012-12-17 11:01:58 PST
This patch depends on the patch for bug 103627
Anton Vayvod
Comment 3 2013-01-07 06:49:46 PST
Created attachment 181502 [details] Patch
John Mellor
Comment 4 2013-03-27 03:24:43 PDT
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.
Bem Jones-Bey
Comment 5 2014-04-16 17:02:29 PDT
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.
Daniel Bates
Comment 6 2016-09-19 15:13:00 PDT
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.
Note You need to log in before you can comment on or make changes to this bug.