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

Unified Diff: chrome/app/chrome_main_delegate.cc

Issue 8589001: Load mac sandbox definitions from resources instead of the bundle. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: different approach Created 9 years, 1 month 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
« no previous file with comments | « no previous file | chrome/browser/browser_resources.grd » ('j') | chrome/common/chrome_content_client.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/app/chrome_main_delegate.cc
diff --git a/chrome/app/chrome_main_delegate.cc b/chrome/app/chrome_main_delegate.cc
index 618b8942dd45635e78c82326dbf0459e0d542bd4..dce757d14d7b85be4bf4211e0f70720f1c97f37e 100644
--- a/chrome/app/chrome_main_delegate.cc
+++ b/chrome/app/chrome_main_delegate.cc
@@ -282,8 +282,11 @@ bool SubprocessNeedsResourceBundle(const std::string& process_type) {
process_type == switches::kZygoteProcess ||
#endif
#if defined(OS_MACOSX)
- // Mac needs them to for scrollbar related images.
+ // Mac needs them to for scrollbar related images and for the sandbox.
jeremy 2011/11/23 07:02:17 nit: *too and for the sandbox -> and for sandbox p
jochen (gone - plz use gerrit) 2011/11/23 10:57:28 Done.
process_type == switches::kWorkerProcess ||
+ process_type == switches::kNaClLoaderProcess ||
+ process_type == switches::kPpapiPluginProcess ||
+ process_type == switches::kPpapiBrokerProcess ||
#endif
process_type == switches::kRendererProcess ||
process_type == switches::kUtilityProcess;
@@ -607,7 +610,10 @@ void ChromeMainDelegate::PreSandboxStartup() {
// sources. The language should have been passed in to us from the
// browser process as a command line flag.
DCHECK(command_line.HasSwitch(switches::kLang) ||
- process_type == switches::kZygoteProcess);
+ process_type == switches::kZygoteProcess ||
+ process_type == switches::kNaClLoaderProcess ||
+ process_type == switches::kPpapiBrokerProcess ||
+ process_type == switches::kPpapiPluginProcess);
// TODO(markusheintz): The command line flag --lang is actually processed
// by the CommandLinePrefStore, and made available through the PrefService
« no previous file with comments | « no previous file | chrome/browser/browser_resources.grd » ('j') | chrome/common/chrome_content_client.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698