Comment #21 on issue 2780 by [email protected]: RTL: Omnibar -
message "Press Tab to search Google" doesn't show correctly in a "New Tab"
for RTL locales
http://code.google.com/p/chromium/issues/detail?id=2780
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=21667
------------------------------------------------------------------------
r21667 | [email protected] | 2009-07-27 11:43:55 -0700 (Mon, 27 Jul
2009) | 16 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/branches/195/src/chrome/browser/views/location_bar_view.cc?r1=21667&r2=21666
M
http://src.chromium.org/viewvc/chrome/branches/195/src/chrome/browser/views/location_bar_view.h?r1=21667&r2=21666
Merge 21312 - This CL fixes issue 2780 RTL: Omnibar message "Press Tab to
search Google" doesn't show correctly in a "New Tab" for RTL locales
Because of the bug in PosFromChar() (see bug report for detailed
information), we actually calculate the width of the string ourselves using
font.GetStringWidth().
BUG=http://crbug.com/2780
TEST=
1 Start Chrome with a *new user data directory* and make sure the UI
language is Hebrew.
2 Type www.google.com in the omnibox and press Enter.
3 Close and reopen the browser.
4 Type character "h" in Ominibox
5 Message "Press Tab to search Google" should show correctly, not
only "Tab" is displayed
Review URL: http://codereview.chromium.org/155789
[email protected]
Review URL: http://codereview.chromium.org/160180
------------------------------------------------------------------------
--
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
-~----------~----~----~----~------~----~------~--~---
