Bug 125824
| Summary: | WebKit creates unnecessary fake runs for isolates | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> |
| Component: | Layout and Rendering | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | CC: | aharon, igor.oliveira, jonlee, koivisto, mmaxfield |
| Priority: | P2 | Keywords: | BlinkMergeCandidate |
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Ryosuke Niwa
Consider merging https://chromium.googlesource.com/chromium/blink/+/02bca0ac6f09e150df9f8bf210959cc4112322a1
Currently Blink creates a fake run for a given isolated span once during each call
to createBidiRunsForLine. However in some cases the added fake run is not necessary.
This patch uses the same logic in RenderBlockFlow::appendRunsObject to create
fake runs just when they are really needed.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Ryosuke Niwa
Also see https://chromium.googlesource.com/chromium/blink/+/9ae837bc9e1d8a74472ec0c67bfa25f47eee484e
Ryosuke Niwa
Also see https://chromium.googlesource.com/chromium/blink/+/6f6e18ee7069f34e026c6be33ce7966780d08184
Ryosuke Niwa
And https://chromium.googlesource.com/chromium/blink/+/8db277450da9b79d43a62fe745bb87a35e73f76b
Aharon (Vladimir) Lanin
Please also see https://code.google.com/p/chromium/issues/detail?id=330974 (now fixed there).