Comment #4 on issue 7377 by [email protected]: Regression: Omnibox
trims URL ending with 0x85
http://code.google.com/p/chromium/issues/detail?id=7377
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=10456
------------------------------------------------------------------------
r10456 | [email protected] | 2009-02-26 02:02:52 -0800 (Thu, 26 Feb 2009)
| 6 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/string_util.cc?r1=10456&r2=10455
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/string_util.h?r1=10456&r2=10455
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/string_util_unittest.cc?r1=10456&r2=10455
M
http://src.chromium.org/viewvc/chrome/trunk/src/build/googleurl.xcodeproj/project.pbxproj?r1=10456&r2=10455
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/net/url_fixer_upper.cc?r1=10456&r2=10455
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/net/url_fixer_upper_unittest.cc?r1=10456&r2=10455
Fixes Issue 7377: Regression: Omnibox trims URL ending with 0x85
To fix this issue, this change adds a new function TrimWhitespaceUTF8(),
which trims space characters (including non-printable characters and broken
UTF-8 characters) from either end of a UTF-8 string.
Please feel free to give me your comments since I'm not sure this
implimentation is correct. (Maybe this implementation trims too
aggressively.)
BUG=7377
Review URL: http://codereview.chromium.org/20219
------------------------------------------------------------------------
--
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
-~----------~----~----~----~------~----~------~--~---
