Closed Bug 82093 Opened 23 years ago Closed 23 years ago

bidi: arabic: Conv_FE_06 in intl/unicharutil/src/nsBidiUtilsImp.cpp should be turn on

Categories

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

Product:
Core
Component:
Layout: Text and Fonts
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: ftang, Assigned: smontagu)

References

(
URL
)

Details

The code in 
nsBidiUtilsImp::Conv_FE_06 should be turn on . Currently it is #ifdef 
FULL_ARABIC_SHAPING out because the code was not fully reviewed.

We should treat this seperate bug from ArabicShaping since ArabicShaping is used 
for display and this one have lower priority than it.

Currently, the code is ued by nsCopySupport::HTMLCopy
layout/base/src/nsCopySupport.cpp for bidi system 
and nsFormFrame::UnicodeToNewBytes in layout/html/forms/src/nsFormFrame.cpp

I don't see any bad ARB in this code. We should remove that #ifdef 
FULL_ARABIC_SHAPING after we address the nsString API change issue.
Blocks: 81194
QA Contact: andreasb → mahar
depends on 82383
Depends on: 82383
Component: Internationalization → BiDi Hebrew & Arabic
Mass-move all BiDi Hebrew and Arabic qa to me, zach@zachlipton.com. 
Thank you Gilad for your service to this component, and best of luck to you 
in the future.

Sholom.
QA Contact: mahar → zach
Arabic QA to mahar.
QA Contact: zach → mahar
This bug has been superseded by bug 92797
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Component: Layout: BiDi Hebrew & Arabic → Layout: Text
QA Contact: mahar → layout.fonts-and-text
You need to log in before you can comment on or make changes to this bug.