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

Description Ryosuke Niwa 2013-12-16 18:42:51 PST

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.

Comment 1 Ryosuke Niwa 2013-12-19 22:04:57 PST

Also see https://chromium.googlesource.com/chromium/blink/+/9ae837bc9e1d8a74472ec0c67bfa25f47eee484e

Comment 2 Ryosuke Niwa 2013-12-19 22:16:47 PST

Also see https://chromium.googlesource.com/chromium/blink/+/6f6e18ee7069f34e026c6be33ce7966780d08184

Comment 3 Ryosuke Niwa 2013-12-20 18:00:16 PST

And https://chromium.googlesource.com/chromium/blink/+/8db277450da9b79d43a62fe745bb87a35e73f76b

Comment 4 Aharon (Vladimir) Lanin 2014-01-08 02:16:36 PST

Please also see https://code.google.com/p/chromium/issues/detail?id=330974 (now fixed there).