Delete the utility process startup ping.

chrome/browser/safe_browsing/sandboxed_dmg_analyzer_mac.cc

 // Copyright 2015 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.
 
 #include "chrome/browser/safe_browsing/sandboxed_dmg_analyzer_mac.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "chrome/common/chrome_utility_messages.h"
@@ skipping 47 matching lines @@
 void SandboxedDMGAnalyzer::StartAnalysis() {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 
   utility_process_host_ =
       content::UtilityProcessHost::Create(this,
           BrowserThread::GetMessageLoopProxyForThread(BrowserThread::IO))
       ->AsWeakPtr();
 
   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) {
   OnAnalysisFinished(zip_analyzer::Results());
 }
 
 void SandboxedDMGAnalyzer::OnProcessLaunchFailed() {
   OnAnalysisFinished(zip_analyzer::Results());
 }
 
 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)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
   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);
   if (callback_called_)
     return;
 
   callback_called_ = true;
   BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
                           base::Bind(callback_, results));
 }
 
 }  // namespace safe_browsing