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
-~----------~----~----~----~------~----~------~--~---

Reply via email to