Comment #2 on issue 24722 by bugdro...@chromium.org:
file_util::TrimTrailingSeparator is deprecated and broken and should be
removed
http://code.google.com/p/chromium/issues/detail?id=24722
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=29093
------------------------------------------------------------------------
r29093 | tk...@chromium.org | 2009-10-14 22:00:25 -0700 (Wed, 14 Oct 2009)
| 6 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/file_util.cc?r1=29093&r2=29092
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/file_util.h?r1=29093&r2=29092
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/file_util_posix.cc?r1=29093&r2=29092
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/file_util_unittest.cc?r1=29093&r2=29092
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/file_util_win.cc?r1=29093&r2=29092
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/mini_installer_test/chrome_mini_installer.cc?r1=29093&r2=29092
Remove file_util::TrimTrailingSeparators(), which is deprecated and
doesn't work well for Windows root drives.
BUG=24722
TEST=existing tests are enough.
Review URL: http://codereview.chromium.org/271086
------------------------------------------------------------------------
--
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
-~----------~----~----~----~------~----~------~--~---
