71339 - check in bidi code for layout/html/forms

Status: VERIFIED FIXED

(Core :: Layout, defect, P1)

Description

[Erik van der Poel]

This bug report will be used to attach the diffs for layout/html/forms from the
IBM bidi project.

Comment 1

[Erik van der Poel]

Attachment: diffs for mozilla/layout/html/forms/src/*

Comment 2

[Erik van der Poel]

Rod, please (have someone) review these changes. Thanks!

Comment 3

[rods (gone)]

Wow, a lot of changes, I'll apply the patch and take a look.

Comment 4

[Erik van der Poel]

Rod, how's it going? Note that the changes are wrapped in #ifdef IBMBIDI, and
we won't be turning on the ifdef until later, so you shouldn't see anything
when you apply these patches.

Comment 5

[rods (gone)]

r=rods since they aren't turned on go ahead and check them in. But we should 
talk before they get turned on.

Comment 6

[Erik van der Poel]

Attachment: ignore previous attachment; this one does not mess with white-space

Comment 7

[chris hofmann]

this stuff goes after 0.8.1

Comment 8

[Erik van der Poel]

Changes checked in.

Comment 9

[Erik van der Poel]

Rod, we are still reviewing and checking in #ifdef IBMBIDI changes in other
parts of the tree, but we would like to turn on the ifdef soon afterwards, so
would you please let me know what other concerns you might have with this part
(i.e. forms)? Let me know whether it's OK to turn on the ifdef for forms. Thx.

Comment 10

[Frank Tang]

code change bug, reassign QA contact to ftang
v=ftang