Add PDF compositor service

content/browser/utility_process_host_impl.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_UTILITY_PROCESS_HOST_IMPL_H_
 #define CONTENT_BROWSER_UTILITY_PROCESS_HOST_IMPL_H_
 
 #include <memory>
 #include <string>
 #include <vector>
@@ skipping 42 matching lines @@
   void ElevatePrivileges() override;
 #endif
   const ChildProcessData& GetData() override;
 #if defined(OS_POSIX)
   void SetEnv(const base::EnvironmentMap& env) override;
 #endif
   bool Start() override;
   void BindInterface(const std::string& interface_name,
                      mojo::ScopedMessagePipeHandle interface_pipe) override;
   void SetName(const base::string16& name) override;
+  void RegisterMojoServices() override;

[Ken Rockot(use gerrit already), 2017/05/04 17:27:18]

Agree with thestig@'s comments. This doesn't need to be tied to
UtilityProcessHostImpl.

RenderProcessHostImpl listens for the same memory-related interfaces using its
own ConnectionFilter, but that too is unnecessary. We should move the filter to
some common location so that *all* connections to the browser can bind it.

AFAICT though your patch should not depend on this stuff working for utility
processes -- the error messages that result during utility process launch are
innocuous and can be ignored -- so we should just treat this as a separate bug
and not address it here.

[Wei Li, 2017/05/04 18:18:28]

Acknowledged.

 
   void set_child_flags(int flags) { child_flags_ = flags; }
 
  private:
+  class ConnectionFilterImpl;
+
   // Starts a process if necessary.  Returns true if it succeeded or a process
   // has already been started via StartBatchMode().
   bool StartProcess();
 
   // BrowserChildProcessHost:
   bool OnMessageReceived(const IPC::Message& message) override;
   void OnProcessLaunchFailed(int error_code) override;
   void OnProcessCrashed(int exit_code) override;
 
   // Cleans up |this| as a result of a failed Start().
@@ skipping 37 matching lines @@
 
   // Used to vend weak pointers, and should always be declared last.
   base::WeakPtrFactory<UtilityProcessHostImpl> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(UtilityProcessHostImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_UTILITY_PROCESS_HOST_IMPL_H_