Bug 15068 – [S60] VBRY-763RLF : RTL brackets and alt text are displayed incorrecting

• Status: CLOSED
• Resolution: FIXED
• Component: DOM

[Pat]

TSW ID: VBRY-763RLF
For RTL languages, special characters, such as brackets are displayed incorrectly. Also Alternative text for images are displayed LTR, instead of RTL. The alt text issue is on Safari also, but they always display a broken image icon, so the text is not displayed.

[Pat]

Created attachment 16132 [details]
3.2 patch for RTL text in body and alt text

This patch needs to be reviewed by Sachin (WebKit part) and Zalan (WebCore part).

[Pat]

Comment on attachment 16132 [details]
3.2 patch for RTL text in body and alt text

Sachin requested that Yongjun review the WebKit part of patch, since he wrote that module. Reassigning review to Yongjun.

[Yongjun Zhang]

Comment on attachment 16132 [details]
3.2 patch for RTL text in body and alt text

Possible memory leak here:
+                // We have a RTL string
+                TBidiText* bidiText = TBidiText::NewL( newStr, 1, TBidiText::ERightToLeft );
+                bidiText->WrapText( WidthForRun( aRun, aStyle ), *iFont );
+                bidiText->DrawText( bitgc, startPos );

/ bidiText is never deleted.

[Pat]

Created attachment 16219 [details]
3.2 patch for RTL text in body and alt image text

Fixes memory leak, by deleting TBidiText.

[Yongjun Zhang]

Comment on attachment 16219 [details]
3.2 patch for RTL text in body and alt image text

r = me

[Bradley Morrison]

Bulk closing of all s60 platform bugs. 

Sorry for the noise!

[Joel Parks]

re-purposing InTSW keyword for use by QtWebkit team