Comment #9 on issue 24672 by [email protected]: Use FilePath instead of wstring for file paths http://code.google.com/p/chromium/issues/detail?id=24672
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=29472
------------------------------------------------------------------------
r29472 | [email protected] | 2009-10-19 15:54:41 -0700 (Mon, 19 Oct 2009) |
9 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/base_paths_win.cc?r1=29472&r2=29471
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/i18n/icu_util.cc?r1=29472&r2=29471
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/path_service.cc?r1=29472&r2=29471
M
http://src.chromium.org/viewvc/chrome/trunk/src/base/path_service.h?r1=29472&r2=29471
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/importer/firefox_importer_unittest.cc?r1=29472&r2=29471
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/importer/firefox_importer_unittest_utils.h?r1=29472&r2=29471
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/importer/firefox_importer_unittest_utils_mac.cc?r1=29472&r2=29471
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/nacl_process_host.cc?r1=29472&r2=29471
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/renderer_host/gtk_key_bindings_handler_unittest.cc?r1=29472&r2=29471
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/user_data_manager.cc?r1=29472&r2=29471
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/zygote_host_linux.cc?r1=29472&r2=29471
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/ui/ui_test.cc?r1=29472&r2=29471
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/plugins/plugin_list_win.cc?r1=29472&r2=29471
Deprecate PathService::Get(..., wstring*) and use FilePath instead.
I tried fixing all the Windows code but there's a *ton* of it.
This change will at least prevent people from adding new code that
uses the deprecated version (as that won't compile on Lin/Mac).
BUG=24672
Review URL: http://codereview.chromium.org/293013
------------------------------------------------------------------------
--
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
-~----------~----~----~----~------~----~------~--~---
