Add PDF compositor service

content/public/browser/utility_process_host.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_PUBLIC_BROWSER_UTILITY_PROCESS_HOST_H_
 #define CONTENT_PUBLIC_BROWSER_UTILITY_PROCESS_HOST_H_
 
 #include "base/environment.h"
 #include "base/process/launch.h"
 #include "base/threading/thread.h"
@@ skipping 67 matching lines @@
   virtual void SetEnv(const base::EnvironmentMap& env) = 0;
 #endif
 
   // Starts the utility process.
   virtual bool Start() = 0;
 
   // Bind an interface exposed by the utility process.
   virtual void BindInterface(const std::string& interface_name,
                              mojo::ScopedMessagePipeHandle interface_pipe) = 0;
 
+  // Register mojo services that the utility process can use.

[Lei Zhang, 2017/05/04 02:13:13]

Mention this needs to run on the UI thread.

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

Done.

+  virtual void RegisterMojoServices() = 0;

[Lei Zhang, 2017/05/04 02:13:13]

It's weird to have this method as is:

1) The impl can be static.
2) The rest of the class runs on the IO thread. This is a bit of an odd ball as
a class method.

Can it leave elsewhere? Is it really tied to UtilityProcessHost?

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

As Ken said, once we can refactor those memory interfaces to some common
location, we can remove this. 

+
   // Set the name of the process to appear in the task manager.
   virtual void SetName(const base::string16& name) = 0;
 };
 
 };  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_UTILITY_PROCESS_HOST_H_