Deprecate PathService::Get(..., wstring*) and use FilePath instead.

webkit/glue/plugins/plugin_list_win.cc

Index: webkit/glue/plugins/plugin_list_win.cc
diff --git a/webkit/glue/plugins/plugin_list_win.cc b/webkit/glue/plugins/plugin_list_win.cc
index 95762381c5b33cdc206395da90b70172a8212edf..7506ed03adae5f75ecfad4eb0e2843453371ff87 100644
--- a/webkit/glue/plugins/plugin_list_win.cc
+++ b/webkit/glue/plugins/plugin_list_win.cc
@@ -121,10 +121,11 @@ void GetFirefoxDirectory(std::set<FilePath>* plugin_dirs) {
   GetPluginsInRegistryDirectory(
       HKEY_LOCAL_MACHINE, kRegistryMozillaPlugins, plugin_dirs);
 
-  std::wstring firefox_app_data_plugin_path;
+  FilePath firefox_app_data_plugin_path;
   if (PathService::Get(base::DIR_APP_DATA, &firefox_app_data_plugin_path)) {
-    firefox_app_data_plugin_path += L"\\Mozilla\\plugins";
-    plugin_dirs->insert(FilePath(firefox_app_data_plugin_path));
+    firefox_app_data_plugin_path =
+        firefox_app_data_plugin_path.AppendASCII("Mozilla/plugins");

[tony, 2009/10/19 22:46:49]

Is it safe to embed the '/' here or do you need to call AppendASCII twice?

[Evan Stade, 2009/10/19 22:54:31]

<bikeshedding>

I agree AppendASCII twice is probably better, but since this is Windows-only you
can use Append with a wide literal string

</bikeshedding>

+    plugin_dirs->insert(firefox_app_data_plugin_path);
   }
 }