Delete the utility process startup ping.

chrome/browser/safe_browsing/sandboxed_dmg_analyzer_mac.cc

Index: chrome/browser/safe_browsing/sandboxed_dmg_analyzer_mac.cc
diff --git a/chrome/browser/safe_browsing/sandboxed_dmg_analyzer_mac.cc b/chrome/browser/safe_browsing/sandboxed_dmg_analyzer_mac.cc
index cfc709f8ae6837775ea0649560f6e2ac2e1b10a4..26e08dbc593efef12d5f01c0b1a818e4cede2cb6 100644
--- a/chrome/browser/safe_browsing/sandboxed_dmg_analyzer_mac.cc
+++ b/chrome/browser/safe_browsing/sandboxed_dmg_analyzer_mac.cc
@@ -65,7 +65,9 @@ void SandboxedDMGAnalyzer::StartAnalysis() {
 
   utility_process_host_->SetName(l10n_util::GetStringUTF16(
       IDS_UTILITY_PROCESS_SAFE_BROWSING_ZIP_FILE_ANALYZER_NAME));
-  utility_process_host_->Send(new ChromeUtilityMsg_StartupPing);
+  utility_process_host_->Send(
+      new ChromeUtilityMsg_AnalyzeDmgFileForDownloadProtection(
+          IPC::TakePlatformFileForTransit(std::move(file_))));
 }

[Scott Hess - ex-Googler, 2016/04/13 17:44:43]

I suspect this may also no longer need to be an instance variable, though since
it's not reset() after, it's less obvious.  I guess it depends on how
utility_process_host_ operates internally.

[Anand Mistry (off Chromium), 2016/04/13 23:42:57]

It's unneeded (and reset does nothing since it's a weakptr). Removed.

 
 void SandboxedDMGAnalyzer::OnProcessCrashed(int exit_code) {
@@ -79,8 +81,6 @@ void SandboxedDMGAnalyzer::OnProcessLaunchFailed() {
 bool SandboxedDMGAnalyzer::OnMessageReceived(const IPC::Message& message) {
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(SandboxedDMGAnalyzer, message)
-    IPC_MESSAGE_HANDLER(ChromeUtilityHostMsg_ProcessStarted,
-                        OnUtilityProcessStarted)
     IPC_MESSAGE_HANDLER(
         ChromeUtilityHostMsg_AnalyzeDmgFileForDownloadProtection_Finished,
         OnAnalysisFinished)
@@ -89,13 +89,6 @@ bool SandboxedDMGAnalyzer::OnMessageReceived(const IPC::Message& message) {
   return handled;
 }
 
-void SandboxedDMGAnalyzer::OnUtilityProcessStarted() {
-  DCHECK_CURRENTLY_ON(BrowserThread::IO);
-  utility_process_host_->Send(
-      new ChromeUtilityMsg_AnalyzeDmgFileForDownloadProtection(
-          IPC::TakePlatformFileForTransit(std::move(file_))));
-}
-
 void SandboxedDMGAnalyzer::OnAnalysisFinished(
     const zip_analyzer::Results& results) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);