Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2044)

Unified Diff: chrome/browser/nacl_process_host.cc

Issue 293013: Deprecate PathService::Get(..., wstring*) and use FilePath instead. (Closed)
Patch Set: Created 11 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/nacl_process_host.cc
diff --git a/chrome/browser/nacl_process_host.cc b/chrome/browser/nacl_process_host.cc
index 133f17f412c96e703a8d6cafec8dab1dd7f1000d..9d7f4455ff333714943ff455018036df249e5120 100644
--- a/chrome/browser/nacl_process_host.cc
+++ b/chrome/browser/nacl_process_host.cc
@@ -116,8 +116,8 @@ bool NaClProcessHost::LaunchSelLdr(ResourceMessageFilter* renderer_msg_filter,
// Build command line for nacl.
const CommandLine& browser_command_line = *CommandLine::ForCurrentProcess();
- std::wstring exe_path =
- browser_command_line.GetSwitchValue(switches::kBrowserSubprocessPath);
+ FilePath exe_path =
+ browser_command_line.GetSwitchValuePath(switches::kBrowserSubprocessPath);
if (exe_path.empty() && !PathService::Get(base::FILE_EXE, &exe_path))
return false;
@@ -125,8 +125,8 @@ bool NaClProcessHost::LaunchSelLdr(ResourceMessageFilter* renderer_msg_filter,
if (logging::DialogsAreSuppressed())
cmd_line.AppendSwitch(switches::kNoErrorDialogs);
- // propagate the following switches to the plugin command line (along with
- // any associated values) if present in the browser command line
+ // Propagate the following switches to the plugin command line (along with
+ // any associated values) if present in the browser command line.
// TODO(gregoryd): check which flags of those below can be supported.
static const char* const switch_names[] = {
switches::kNoSandbox,

Powered by Google App Engine
This is Rietveld 408576698