discardable_memory: Using mojo IPC to replace Chrome IPC

content/ppapi_plugin/ppapi_thread.cc

Index: content/ppapi_plugin/ppapi_thread.cc
diff --git a/content/ppapi_plugin/ppapi_thread.cc b/content/ppapi_plugin/ppapi_thread.cc
index 5829f837086061c51d729fed63072f4311e472b5..8a452c16417a060d67d6f622679209a9c3bdd598 100644
--- a/content/ppapi_plugin/ppapi_thread.cc
+++ b/content/ppapi_plugin/ppapi_thread.cc
@@ -107,7 +107,11 @@ typedef int32_t (*InitializeBrokerFunc)
     (PP_ConnectInstance_Func* connect_instance_func);
 
 PpapiThread::PpapiThread(const base::CommandLine& command_line, bool is_broker)
-    : is_broker_(is_broker),
+    : ChildThreadImpl(Options::Builder()
+                          .InitDiscardableMemory(
+                              !command_line.HasSwitch(switches::kSingleProcess))

[reveman, 2016/11/18 21:55:26]

sorry if I asked this already but would it be possible to just init this even in
the single process case? and in that way remove the need for a builder option..

[Peng, 2016/11/21 15:56:39]

Done

[Peng, 2016/11/21 18:31:42]

Sorry. I just tried it. But the content_browsertest crashes. It is because the
content browsertest will set this flag, but we still need initial the client
discardable shared memory manager for it. So we cannot test this flags in
ChildThreadImpl.

+                          .Build()),
+      is_broker_(is_broker),
       plugin_globals_(GetIOTaskRunner()),
       connect_instance_func_(NULL),
       local_pp_module_(base::RandInt(0, std::numeric_limits<PP_Module>::max())),