When flowing a line of justified, right-to-left text where only a single word can fit on the line, the word is placed flush left, instead of flush right. (Note that things work correctly in the case of the _last_ line flowed.)

Moving to bidi. Is this a dupe?

I don't think this is a dupe, I don't recall seeing it before.

Comment on attachment 229377 [details] [diff] [review] patch I don't like the code duplication, especially since I suspect you should duplicate it again in the else of "if (!aShrinkWrapWidth)". (I don't know in what circumstances aAllowJustify and aShrinkWrapWidth can both be true). I would like it better to put a break after ApplyFrameJustification and remove the else from else if (NS_STYLE_DIRECTION_RTL ...

... like this. Uri and I couldn't reach agreement on IRC which version was clearer. At the bottom line, whichever version dbaron prefers has r=smontagu.

I think I prefer neither. Why not move the JUSTIFY case up to before the DEFAULT case, and then only break if justification was performed? Sorry to take so long to get to this.

This is basically moving code, although I did wrap the overly long line, insert and remove the break statements, and change the comments.

Note also that aShrinkWrapWidth no longer exists post-reflow-branch.

Comment on attachment 248331 [details] [diff] [review] how 'bout this? That's great.

Checked in to trunk.

Mass-assigning the new rtl keyword to RTL-related (see bug 349193).