Comment #9 on issue 24672 by bugdro...@chromium.org: 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 | e...@chromium.org | 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 -~----------~----~----~----~------~----~------~--~---