Change the NaCl loader and broker processes to use the ServiceManager.

chromecast/browser/cast_content_browser_client.h

Index: chromecast/browser/cast_content_browser_client.h
diff --git a/chromecast/browser/cast_content_browser_client.h b/chromecast/browser/cast_content_browser_client.h
index cc07dcb77a6ec003ba135fd856db63cb2b03d670..058d75591a20a669826976cd426646e42348d601 100644
--- a/chromecast/browser/cast_content_browser_client.h
+++ b/chromecast/browser/cast_content_browser_client.h
@@ -158,7 +158,7 @@ class CastContentBrowserClient : public content::ContentBrowserClient {
       content::RenderProcessHost* render_process_host) override;
   void RegisterInProcessServices(StaticServiceMap* services) override;
   std::unique_ptr<base::Value> GetServiceManifestOverlay(
-      const std::string& service_name) override;
+      base::StringPiece service_name) override;

[Mark Seaborn, 2016/12/19 19:17:05]

Could you explain in the commit message why this change involves converting
these other parts of the codebase to use StringPiece?  (Would it be better to do
this conversion as a separate change?)

[Sam McNally, 2016/12/19 23:05:22]

Done.

 #if defined(OS_ANDROID)
   void GetAdditionalMappedFilesForChildProcess(
       const base::CommandLine& command_line,