Convert GetSearchProviderInstallState to Mojo

chrome/browser/search_engines/search_provider_install_state_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 CHROME_BROWSER_SEARCH_ENGINES_SEARCH_PROVIDER_INSTALL_STATE_MESSAGE_FILTER_H_
-#define CHROME_BROWSER_SEARCH_ENGINES_SEARCH_PROVIDER_INSTALL_STATE_MESSAGE_FILTER_H_
+#ifndef CHROME_BROWSER_SEARCH_ENGINES_SEARCH_PROVIDER_INSTALL_STATE_IMPL_H_
+#define CHROME_BROWSER_SEARCH_ENGINES_SEARCH_PROVIDER_INSTALL_STATE_IMPL_H_
 
+#include "base/callback.h"

[Sam McNally, 2016/06/23 03:05:35]

callback_forward.h

[tibell, 2016/06/23 03:44:39]

Done.

 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
+#include "chrome/browser/profiles/profile.h"

[Sam McNally, 2016/06/23 03:05:35]

Remove.

[tibell, 2016/06/23 03:44:38]

Done.

 #include "chrome/browser/search_engines/search_provider_install_data.h"
 #include "chrome/common/search_provider.h"

[Sam McNally, 2016/06/23 03:05:35]

Remove.

[tibell, 2016/06/23 03:44:39]

Done.

+#include "chrome/common/search_provider.mojom.h"
 #include "content/public/browser/browser_message_filter.h"
+#include "content/public/browser/render_process_host.h"

[Sam McNally, 2016/06/23 03:05:35]

Remove and forward declare RenderProcessHost instead.

[tibell, 2016/06/23 03:44:38]

Done.

+#include "content/public/common/service_registry.h"

[Sam McNally, 2016/06/23 03:05:35]

Move this into the .cc file.

[tibell, 2016/06/23 03:44:39]

Done.

+#include "mojo/public/cpp/bindings/binding_set.h"
 
 class GURL;
 class Profile;
 
 // Handles messages regarding search provider install state on the I/O thread.
-class SearchProviderInstallStateMessageFilter
-    : public content::BrowserMessageFilter {
+class SearchProviderInstallStateImpl
+    : public chrome::mojom::SearchProviderInstallState {
  public:
   // Unlike the other methods, the constructor must be called on the UI thread.
-  SearchProviderInstallStateMessageFilter(int render_process_id,
-                                          Profile* profile);
+  SearchProviderInstallStateImpl(int render_process_id, Profile* profile);
+  ~SearchProviderInstallStateImpl() override;
 
-  // content::BrowserMessageFilter implementation.
-  bool OnMessageReceived(const IPC::Message& message) override;
+  // Create impl and register it in the host's service registry.
+  static void InstallService(content::RenderProcessHost* host);
 
  private:
-  ~SearchProviderInstallStateMessageFilter() override;
+  void Bind(chrome::mojom::SearchProviderInstallStateRequest request);
 
   // Figures out the install state for the search provider.
-   search_provider::InstallState GetSearchProviderInstallState(
+  chrome::mojom::InstallState GetSearchProviderInstallState(
       const GURL& page_location,
       const GURL& requested_host);
 
   // Starts handling the message requesting the search provider install state.
-  void OnGetSearchProviderInstallState(const GURL& page_location,
-                                       const GURL& requested_host,
-                                       IPC::Message* reply_msg);
+  // chrome::mojom::SearchProviderInstallState override.
+  void GetInstallState(const GURL& page_location,
+                       const GURL& requested_host,
+                       const GetInstallStateCallback& callback) override;
 
   // Sends the reply message about the search provider install state.
   void ReplyWithProviderInstallState(const GURL& page_location,
                                      const GURL& requested_host,
-                                     IPC::Message* reply_msg);
+                                     const GetInstallStateCallback& callback);
 
   // Used to do a load and get information about install states.
   SearchProviderInstallData provider_data_;
 
   // Copied from the profile since the profile can't be accessed on the I/O
   // thread.
   const bool is_off_the_record_;
 
+  mojo::BindingSet<chrome::mojom::SearchProviderInstallState> binding_set_;
+
   // Used to schedule invocations of ReplyWithProviderInstallState.
-  base::WeakPtrFactory<SearchProviderInstallStateMessageFilter> weak_factory_;
+  base::WeakPtrFactory<SearchProviderInstallStateImpl> weak_factory_;
 
-  DISALLOW_COPY_AND_ASSIGN(SearchProviderInstallStateMessageFilter);
+  DISALLOW_COPY_AND_ASSIGN(SearchProviderInstallStateImpl);
 };
 
-#endif  // CHROME_BROWSER_SEARCH_ENGINES_SEARCH_PROVIDER_INSTALL_STATE_MESSAGE_FILTER_H_
+#endif  // CHROME_BROWSER_SEARCH_ENGINES_SEARCH_PROVIDER_INSTALL_STATE_IMPL_H_