Comment #5 on issue 16279 by [email protected]: Problems in typing Thai in  
an input box on Linux Chrome
http://code.google.com/p/chromium/issues/detail?id=16279

agl,

To investigate this issue quickly today, it seems your  
Font::selectionRectForComplexText() returns a wrong position
for LTR strings.

604:    if (toX == -1 && !to)
605:        toX = rightEdge;
606:    else if (!walker.rtl())
607:        toX += truncateFixedPointToInteger(toAdvance);

This code adds |toAdvance| to |toX| when the given text is LTR. Even though  
I'm not totally sure about this code,
I'm wondering if it is correct to add |toAdvance| to |toX| here for LTR  
strings. (As far as I tested this function
with some LTR strings, |toX| becomes the correct value before reaching to  
this code.)

Since I'm not sure the context of this code, it is better for you to fix  
this problem. Anyway, I'm happy to help
you if you need my help.

Regards,

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to