Bug 45606 - Neglect unlikely hyphenation opportunities
Summary: Neglect unlikely hyphenation opportunities
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Text (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: mitz
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-11 18:45 PDT by mitz
Modified: 2010-09-12 16:04 PDT (History)
0 users

See Also:

Attachments
Patch (2.02 KB, patch)
2010-09-11 18:57 PDT, mitz 		no flags Details | Formatted Diff | Diff
Patch (2.02 KB, patch)
2010-09-11 19:01 PDT, mitz 		adele: review+
Details | Formatted Diff | Diff
Show Obsolete (1) 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 mitz 2010-09-11 18:45:09 PDT 
Neglect unlikely hyphenation opportunities
Comment 1 mitz 2010-09-11 18:57:57 PDT 
Created attachment 67324 [details]
Patch
Comment 2 mitz 2010-09-11 19:01:13 PDT 
Created attachment 67325 [details]
Patch
Comment 3 Adele Peterson 2010-09-12 15:12:44 PDT 
Comment on attachment 67325 [details]
Patch

> +    // If the maximum width available for the prefix before the hyphen is small, then it is very unlikely
> +    // that an hyphenation opportunity exists, so do bother to look for it.
Did you mean "don't bother"?
Comment 4 mitz 2010-09-12 15:53:41 PDT 
(In reply to comment #3)
> (From update of attachment 67325 [details])
> > +    // If the maximum width available for the prefix before the hyphen is small, then it is very unlikely
> > +    // that an hyphenation opportunity exists, so do bother to look for it.
> Did you mean "don't bother"?

Yes :)
Comment 5 mitz 2010-09-12 16:04:59 PDT 
Committed <http://trac.webkit.org/changeset/67350>.