Bug 53311

Summary: Changing unicode-bidi doesn’t force layout
Product: WebKit Reporter: mitz
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, andersca, eric, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Force layout when unicode-bidi changes andersca: review+

mitz
Reported 2011-01-28 10:42:51 PST
Patch forthcoming.
Attachments
Force layout when unicode-bidi changes (19.65 KB, patch)
2011-01-28 10:44 PST, mitz 		andersca: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2011-01-28 10:44:40 PST
Created attachment 80465 [details] Force layout when unicode-bidi changes
Anders Carlsson
Comment 2 2011-01-28 10:45:51 PST
Comment on attachment 80465 [details] Force layout when unicode-bidi changes View in context: https://bugs.webkit.org/attachment.cgi?id=80465&action=review > LayoutTests/ChangeLog:5 > + Changing unicode-bidi doesnât force layout 8-bit character or something here.
mitz
Comment 3 2011-01-28 10:47:47 PST
(In reply to comment #2) > (From update of attachment 80465 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=80465&action=review > > > LayoutTests/ChangeLog:5 > > + Changing unicode-bidi doesnât force layout > > 8-bit character or something here. Maybe even more bits. There’s no use pointing out this long-standing bug in the bugs.webkit.org website in a patch review, in my opinion.
mitz
Comment 4 2011-01-28 10:48:17 PST
Fixed in r76954. <http://trac.webkit.org/changeset/76954>
Anders Carlsson
Comment 5 2011-01-28 10:49:56 PST
(In reply to comment #3) > (In reply to comment #2) > > (From update of attachment 80465 [details] [details]) > > View in context: https://bugs.webkit.org/attachment.cgi?id=80465&action=review > > > > > LayoutTests/ChangeLog:5 > > > + Changing unicode-bidi doesnât force layout > > > > 8-bit character or something here. > > Maybe even more bits. There’s no use pointing out this long-standing bug in the bugs.webkit.org website in a patch review, in my opinion. I didn't know that there was a bug, sorry.
WebKit Review Bot
Comment 6 2011-01-28 12:17:58 PST
http://trac.webkit.org/changeset/76954 might have broken Leopard Intel Release (Tests)
Note You need to log in before you can comment on or make changes to this bug.