[Webkit-unassigned] [Bug 123960] New: Bidi-Isolate inlines break layout with collapsed whitespace

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=123960

           Summary: Bidi-Isolate inlines break layout with collapsed
                    whitespace
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Keywords: BlinkMergeCandidate
          Severity: Normal
          Priority: P2
         Component: Text
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: rniwa at webkit.org
                CC: mitz at webkit.org, enrica at apple.com, jonlee at apple.com,
                    igor.oliveira at webkit.org, mmaxfield at apple.com


Consider merging https://chromium.googlesource.com/chromium/blink/+/d1cf2bc1fb535d697d9060d9f8d5aa5906558ceb

Collapsed space never worked properly if we have spaces inside and/or after a Bidi-isolate.
The problem happens in two parts of the code:

1. The midpointState is not updated properly in addFakeRunIfNecessary. So when 
an isolated run is added the next midpoint state is wrong.
2. Since isolated runs have a two pass algorithm. In the second pass the 
midpoint state is wrong.

This patches fixes the first problem implementing a method called 
adjustMidpointsAndAppendRunsForObjectIfNeeded, when a fake run is added it basically 
updates the midpointState properly. And the second problem is fixed
saving and restoring the midpointState for Bidi-isolate inlines.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.