check in bidi code for layout/html/forms

VERIFIED FIXED in mozilla0.9

Status

()

Core
Layout
P1
normal
VERIFIED FIXED
17 years ago
13 years ago

People

(Reporter: Erik van der Poel, Assigned: Erik van der Poel)

Tracking

Trunk
mozilla0.9
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

17 years ago
This bug report will be used to attach the diffs for layout/html/forms from the
IBM bidi project.
(Assignee)

Comment 1

17 years ago
Created attachment 27161 [details] [diff] [review]
diffs for mozilla/layout/html/forms/src/*
(Assignee)

Comment 2

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

Comment 3

17 years ago
Wow, a lot of changes, I'll apply the patch and take a look.
(Assignee)

Comment 4

17 years ago
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.
(Assignee)

Updated

17 years ago
Priority: -- → P1
Target Milestone: --- → mozilla0.8.1

Comment 5

17 years ago
r=rods since they aren't turned on go ahead and check them in. But we should 
talk before they get turned on.
(Assignee)

Comment 6

17 years ago
Created attachment 27741 [details] [diff] [review]
ignore previous attachment; this one does not mess with white-space

Comment 7

17 years ago
this stuff goes after 0.8.1
Target Milestone: mozilla0.8.1 → mozilla0.9
(Assignee)

Comment 8

17 years ago
Changes checked in.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
(Assignee)

Comment 9

17 years ago
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

17 years ago
code change bug, reassign QA contact to ftang
v=ftang
Status: RESOLVED → VERIFIED
QA Contact: petersen → ftang
You need to log in before you can comment on or make changes to this bug.