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

Unified Diff: chrome/browser/plugin_process_host.cc

Issue 164177: Provide a separate app bundle for subprocesses like the renderer on the Mac.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 11 years, 4 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/plugin_process_host.cc
===================================================================
--- chrome/browser/plugin_process_host.cc (revision 22918)
+++ chrome/browser/plugin_process_host.cc (working copy)
@@ -335,11 +335,10 @@
// build command line for plugin, we have to quote the plugin's path to deal
// with spaces.
- const CommandLine& browser_command_line = *CommandLine::ForCurrentProcess();
- std::wstring exe_path =
- browser_command_line.GetSwitchValue(switches::kBrowserSubprocessPath);
- if (exe_path.empty() && !PathService::Get(base::FILE_EXE, &exe_path))
+ std::wstring exe_path = GetChildPath();
+ if (exe_path.empty()) {
return false;
+ }
CommandLine cmd_line(exe_path);
if (logging::DialogsAreSuppressed())
@@ -368,6 +367,8 @@
switches::kEnableStatsTable,
};
+ const CommandLine& browser_command_line = *CommandLine::ForCurrentProcess();
+
for (size_t i = 0; i < arraysize(switch_names); ++i) {
if (browser_command_line.HasSwitch(switch_names[i])) {
cmd_line.AppendSwitchWithValue(

Powered by Google App Engine
This is Rietveld 408576698