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

Unified Diff: content/ppapi_plugin/ppapi_thread.cc

Issue 8414020: Expose the sandbox related code through the content API. I did a bit of cleanup while I was doing... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 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: content/ppapi_plugin/ppapi_thread.cc
===================================================================
--- content/ppapi_plugin/ppapi_thread.cc (revision 107639)
+++ content/ppapi_plugin/ppapi_thread.cc (working copy)
@@ -11,11 +11,11 @@
#include "base/rand_util.h"
#include "base/stringprintf.h"
#include "content/common/child_process.h"
-#include "content/common/sandbox_init_wrapper.h"
#include "content/ppapi_plugin/broker_process_dispatcher.h"
#include "content/ppapi_plugin/plugin_process_dispatcher.h"
#include "content/ppapi_plugin/ppapi_webkit_thread.h"
#include "content/public/common/content_switches.h"
+#include "content/public/common/sandbox_init.h"
#include "ipc/ipc_channel_handle.h"
#include "ipc/ipc_sync_channel.h"
#include "ppapi/c/dev/ppp_network_state_dev.h"
@@ -201,10 +201,7 @@
// We need to do this after getting |PPP_GetInterface()| (or presumably
// doing something nontrivial with the library), else the sandbox
// intercedes.
- CommandLine* parsed_command_line = CommandLine::ForCurrentProcess();
- SandboxInitWrapper sandbox_wrapper;
- if (!sandbox_wrapper.InitializeSandbox(*parsed_command_line,
- switches::kPpapiPluginProcess))
+ if (!content::InitializeSandbox())
cpu_(ooo_6.6-7.5) 2011/10/29 01:02:35 I normally use that if() macro, needs to be if ()
jam 2011/10/29 01:49:10 ok. done. i think in this case it's fine though be
LOG(WARNING) << "Failed to initialize sandbox";
#endif

Powered by Google App Engine
This is Rietveld 408576698