Convert utility process ParseMediaMetadata IPC to mojo

chrome/utility/extensions/extensions_handler.h

Index: chrome/utility/extensions/extensions_handler.h
diff --git a/chrome/utility/extensions/extensions_handler.h b/chrome/utility/extensions/extensions_handler.h
index 4098d77a708172a9d3f2c9f5b6b6d586b16dbd59..acd987b0ecb1c7551ef3030fdf59adc71a0bf944 100644
--- a/chrome/utility/extensions/extensions_handler.h
+++ b/chrome/utility/extensions/extensions_handler.h
@@ -22,16 +22,25 @@
 
 class ChromeContentUtilityClient;
 
+namespace service_manager {
+class InterfaceRegistry;
+}
+
 namespace extensions {
 
 // Dispatches IPCs for Chrome extensions utility messages.
 class ExtensionsHandler : public UtilityMessageHandler {
  public:
-  explicit ExtensionsHandler(ChromeContentUtilityClient* utility_client);
+  ExtensionsHandler();
   ~ExtensionsHandler() override;
 
   static void PreSandboxStartup();
 
+  static void ExposeInterfacesToBrowser(

[Sam McNally, 2017/01/10 06:55:14]

I feel like we might want to put this in the UtilityMessageHandler interface. A
TODO seems fine until the next UtilityMessageHandler is converted to mojo.

[Noel Gordon, 2017/01/10 22:52:37]

OK, added // TODO(noel): consider moving this API to the UtilityMessageHandler
interface.

I also just spotted https://codereview.chromium.org/1773233002 ... fascinating.

+      service_manager::InterfaceRegistry* registry,
+      ChromeContentUtilityClient* utility_client,
+      bool running_elevated);
+
   // UtilityMessageHandler:
   bool OnMessageReceived(const IPC::Message& message) override;
 
@@ -40,10 +49,6 @@ class ExtensionsHandler : public UtilityMessageHandler {
   void OnCheckMediaFile(int64_t milliseconds_of_decoding,
                         const IPC::PlatformFileForTransit& media_file);
 
-  void OnParseMediaMetadata(const std::string& mime_type,
-                            int64_t total_size,
-                            bool get_attached_images);
-
 #if defined(OS_WIN)
   void OnParseITunesPrefXml(const std::string& itunes_xml_data);
 #endif  // defined(OS_WIN)
@@ -66,9 +71,6 @@ class ExtensionsHandler : public UtilityMessageHandler {
 
   UtilityHandler utility_handler_;
 
-  // The client that owns this.
-  ChromeContentUtilityClient* const utility_client_;
-
   DISALLOW_COPY_AND_ASSIGN(ExtensionsHandler);
 };