139297 - Bidi text reversed in print headers

Status: RESOLVED FIXED

(Core :: Layout: Text and Fonts, defect)

Description

[Simon Montagu :smontagu]

When printing a URL with Bidi text in the page title, the Bidi text in the page
header appears reversed.

Steps to reproduce: go to http://www.google.com/advanced_search?hl=iw or any
page with Hebrew, Arabic etc in the title; Print, or Print Preview.

Expected results: correctly ordered text.

Actual results: All text is the title is rendered left-to-right.

Giving platform/OS as All/All, but note that this doesn't happen on platforms
like Win2000 with native Bidi support.

Comment 1

[Simon Montagu :smontagu]

Attachment: patch v.1

Comment 2

[Simon Montagu :smontagu]

mkaply, can you review the patch?

Comment 3

[Roland Mainz]

Comment on attachment 80495 [details] [diff] [review]
patch v.1

r=[e-mail] ; tested the patch with Xprint
module - the headers now print correctly with this patch... :)

Comment 4

[Marc Attinasi]

Comment on attachment 80495 [details] [diff] [review]
patch v.1

sr=attinasi

Comment 5

[Simon Montagu :smontagu]

Fix checked into trunk

Comment 6

[Roland Mainz]

smontagu:
What about requesting approval for the 1.0-branch ?

Comment 7

[Randell Jesup [:jesup] (needinfo me)]

Comment on attachment 80495 [details] [diff] [review]
patch v.1

a=[e-mail] for branch checkin.	Make sure it's on the trunk as well

Comment 8

[Simon Montagu :smontagu]

Fix is on the trunk:
http://bonsai.mozilla.org/cvslog.cgi?file=mozilla/layout/html/base/src/nsPageFrame.cpp

Comment 9

[Jaime Rodriguez, Jr.]

Let's get this one in after RC2. adt1.0.0- [adt2 RTM]

Comment 10

[scottputterman]

adding adt1.0.0+.  Please get drivers approval again since it's been more than 3
days since the previous approval and afterwards checkin to the 1.0 branch.

Comment 11

[chris hofmann]

reapproved for the 1.0 branch.

Comment 12

[Simon Montagu :smontagu]

Fix checked in to 1.0 branch