Status: Assigned Owner: hb...@chromium.org CC: js...@chromium.org Labels: Type-Cleanup Pri-2 OS-All Area-BrowserUI Size-Medium I18N
New issue 9851 by hb...@chromium.org: Replace TrimWhitespace() with TrimWhitespaceASCII() or TrimWhitespaceUTF8() http://code.google.com/p/chromium/issues/detail?id=9851 * Expected Result: Replace all TrimWhitespace(const std::string&, TrimPositions, std::string*) calls with TrimWhitespaceASCII() or TrimWhitespaceUTF8(). * Actual Result: TrimWhitespace() calls are used for trimming whitespace characters of a UTF-8 string. * Additional information: The std::string version of the TrimWhitespace() function, i.e. TrimWhiteSpace(const std::string&, TrimPositions, std::string*), has been split into TrimWhitespaceASCII() and TrimWhiteSpaceUTF8() functions so that we can trim Unicode whitespace characters from a UTF-8 string. Even though the current TrimWhiteSpace() function just calls the TrimWhiteSpaceASCII() function for backward-compatibility, we should check each TrimWhitespace() call and replace it with TrimWhitespaceASCII() or TrimWhitespaceUTF8() so that we can trim Unicode whitespace characters from a UTF-8 string. -- 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 -~----------~----~----~----~------~----~------~--~---
