Bug 113823

Summary: Generalize prior line break context state and names.
Product: WebKit Reporter: Glenn Adams <glenn>
Component: Layout and RenderingAssignee: Glenn Adams <glenn>
Status: RESOLVED FIXED    
Severity: Minor CC: darin, eric, esprehn+autocc, kojii, leviw, mitz, msaboff, ojan.autocc, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 105692    
Attachments:
Description Flags
Patch none

Description Glenn Adams 2013-04-02 13:46:20 PDT 
In order to better accommodate an expectation of more general state for prior line break context text and to facilitate use of prior context in non-ASCII line break path, change to a fixed UChar array for storage of prior context and change accessors/methods to use better names.

This change is to simplify (reduce the size of) the patch proposed for bug 105692 in order to ease the review and landing process.

No behavioral change is introduced, so no new tests are required.

Note that it is expected that this prior context state may be further generalized in the future in order to accommodate greater than the last two context characters. This is expected to be required to support Thai (and similar) dictionary based line breaking, where the prior context may consist of an entire word.
Comment 1 Glenn Adams 2013-04-02 14:33:54 PDT 
Created attachment 196227 [details]
Patch
Comment 2 WebKit Review Bot 2013-04-02 16:26:57 PDT 
Comment on attachment 196227 [details]
Patch

Clearing flags on attachment: 196227

Committed r147506: <http://trac.webkit.org/changeset/147506>
Comment 3 WebKit Review Bot 2013-04-02 16:27:00 PDT 
All reviewed patches have been landed.  Closing bug.