Convert utility process ParseMediaMetadata IPC to mojo

chrome/utility/extensions/extensions_handler.h

 // Copyright 2014 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 CHROME_UTILITY_EXTENSIONS_EXTENSIONS_HANDLER_H_
 #define CHROME_UTILITY_EXTENSIONS_EXTENSIONS_HANDLER_H_
 
 #include <stdint.h>
 
 #include "base/base64.h"
 #include "base/compiler_specific.h"
 #include "base/macros.h"
 #include "build/build_config.h"
 #include "chrome/common/media_galleries/picasa_types.h"
 #include "chrome/utility/utility_message_handler.h"
 #include "extensions/features/features.h"
 #include "extensions/utility/utility_handler.h"
 
 #if !BUILDFLAG(ENABLE_EXTENSIONS)
 #error "Extensions must be enabled"
 #endif
 
 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;
 
  private:
   // IPC message handlers.
   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)
 
 #if defined(OS_WIN) || defined(OS_MACOSX)
   void OnParseITunesLibraryXmlFile(
       const IPC::PlatformFileForTransit& itunes_library_file);
 
   void OnParsePicasaPMPDatabase(
       const picasa::AlbumTableFilesForTransit& album_table_files);
 
   void OnIndexPicasaAlbumsContents(
       const picasa::AlbumUIDSet& album_uids,
       const std::vector<picasa::FolderINIContents>& folders_inis);
 #endif  // defined(OS_WIN) || defined(OS_MACOSX)
 
 #if defined(OS_WIN)
   void OnGetWiFiCredentials(const std::string& network_guid);
 #endif  // defined(OS_WIN)
 
   UtilityHandler utility_handler_;
 
-  // The client that owns this.
-  ChromeContentUtilityClient* const utility_client_;
-
   DISALLOW_COPY_AND_ASSIGN(ExtensionsHandler);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_UTILITY_EXTENSIONS_EXTENSIONS_HANDLER_H_