Convert GetSearchProviderInstallState to Mojo

chrome/browser/chrome_content_browser_client.cc

Index: chrome/browser/chrome_content_browser_client.cc
diff --git a/chrome/browser/chrome_content_browser_client.cc b/chrome/browser/chrome_content_browser_client.cc
index 74b64ae03340f1ba0357c30a365580dc117a9827..098b2d1effa1e8ca5d164fef05ccb850511bf0a3 100644
--- a/chrome/browser/chrome_content_browser_client.cc
+++ b/chrome/browser/chrome_content_browser_client.cc
@@ -72,7 +72,7 @@
 #include "chrome/browser/search/instant_service.h"
 #include "chrome/browser/search/instant_service_factory.h"
 #include "chrome/browser/search/search.h"
-#include "chrome/browser/search_engines/search_provider_install_state_message_filter.h"
+#include "chrome/browser/search_engines/search_provider_install_state_impl.h"
 #include "chrome/browser/speech/chrome_speech_recognition_manager_delegate.h"
 #include "chrome/browser/speech/tts_controller.h"
 #include "chrome/browser/speech/tts_message_filter.h"
@@ -949,7 +949,7 @@ void ChromeContentBrowserClient::RenderProcessWillLaunch(
 #if defined(ENABLE_PRINTING)
   host->AddFilter(new printing::PrintingMessageFilter(id, profile));
 #endif
-  host->AddFilter(new SearchProviderInstallStateMessageFilter(id, profile));
+  SearchProviderInstallStateImpl::InstallService(host);

[dcheng, 2016/07/04 06:46:29]

Nit: Let's give mojo stuff it's own distinct section.

[dcheng, 2016/07/04 07:42:48]

Actually it looks like there's already a specific callback for registering mojo
interfaces:
https://cs.chromium.org/chromium/src/content/public/browser/content_browser_c...

So let's use that (regardless of whether we use the
ChromeContentBrowserClientPart implementation strategy or not)

[tibell, 2016/07/15 00:47:08]

Now when registration is encapsulated (see AddService on RPH) I think this makes
less sense. What do you think?

 #if defined(ENABLE_SPELLCHECK)
   host->AddFilter(new SpellCheckMessageFilter(id));
 #endif