Add PDF compositor service

content/browser/utility_process_host_impl.h

Index: content/browser/utility_process_host_impl.h
diff --git a/content/browser/utility_process_host_impl.h b/content/browser/utility_process_host_impl.h
index 76dc3550712997100bd693a902aaa05c91e5409f..de66e0e42fe8fe519ae58d123fb93798dc3a5773 100644
--- a/content/browser/utility_process_host_impl.h
+++ b/content/browser/utility_process_host_impl.h
@@ -60,10 +60,13 @@ class CONTENT_EXPORT UtilityProcessHostImpl
   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();