Delete the utility process startup ping.

chrome/browser/printing/pwg_raster_converter.cc

 // Copyright 2013 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/printing/pwg_raster_converter.h"
 
 #include <algorithm>
 #include <memory>
 #include <utility>
 
@@ skipping 112 matching lines @@
                const PWGRasterConverter::ResultCallback& callback);
 
   // UtilityProcessHostClient implementation.
   void OnProcessCrashed(int exit_code) override;
   bool OnMessageReceived(const IPC::Message& message) override;
 
  private:
   ~PwgUtilityProcessHostClient() override;
 
   // Message handlers.
-  void OnProcessStarted();
   void OnSucceeded();
   void OnFailed();
 
   void RunCallback(bool success);
 
   void StartProcessOnIOThread();
 
   void RunCallbackOnUIThread(bool success);
   void OnFilesReadyOnUIThread();
 
@@ skipping 29 matching lines @@
 }
 
 void PwgUtilityProcessHostClient::OnProcessCrashed(int exit_code) {
   OnFailed();
 }
 
 bool PwgUtilityProcessHostClient::OnMessageReceived(
   const IPC::Message& message) {
   bool handled = true;
   IPC_BEGIN_MESSAGE_MAP(PwgUtilityProcessHostClient, message)
-    IPC_MESSAGE_HANDLER(ChromeUtilityHostMsg_ProcessStarted, OnProcessStarted)
     IPC_MESSAGE_HANDLER(
         ChromeUtilityHostMsg_RenderPDFPagesToPWGRaster_Succeeded, OnSucceeded)
     IPC_MESSAGE_HANDLER(ChromeUtilityHostMsg_RenderPDFPagesToPWGRaster_Failed,
                         OnFailed)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
   return handled;
 }
 
-void PwgUtilityProcessHostClient::OnProcessStarted() {
-  DCHECK_CURRENTLY_ON(BrowserThread::IO);
-  if (!utility_process_host_) {
-    RunCallbackOnUIThread(false);
-    return;
-  }
-
-  utility_process_host_->Send(new ChromeUtilityMsg_RenderPDFPagesToPWGRaster(
-      files_->GetPdfForProcess(), settings_, bitmap_settings_,
-      files_->GetPwgForProcess()));
-  utility_process_host_.reset();
-}
-
 void PwgUtilityProcessHostClient::OnSucceeded() {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   RunCallback(true);
 }
 
 void PwgUtilityProcessHostClient::OnFailed() {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   RunCallback(false);
 }
 
 void PwgUtilityProcessHostClient::OnFilesReadyOnUIThread() {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   if (!files_->IsValid()) {
     RunCallbackOnUIThread(false);
     return;
   }
   BrowserThread::PostTask(
       BrowserThread::IO, FROM_HERE,
       base::Bind(&PwgUtilityProcessHostClient::StartProcessOnIOThread, this));
 }
 
 void PwgUtilityProcessHostClient::StartProcessOnIOThread() {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   utility_process_host_ =
       content::UtilityProcessHost::Create(
           this, base::MessageLoop::current()->task_runner())->AsWeakPtr();
   utility_process_host_->SetName(l10n_util::GetStringUTF16(
       IDS_UTILITY_PROCESS_PWG_RASTER_CONVERTOR_NAME));
-  utility_process_host_->Send(new ChromeUtilityMsg_StartupPing);
+  utility_process_host_->Send(new ChromeUtilityMsg_RenderPDFPagesToPWGRaster(
+      files_->GetPdfForProcess(), settings_, bitmap_settings_,
+      files_->GetPwgForProcess()));
+  utility_process_host_.reset();

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

I have no idea how this code works, was just reviewing for color WRT
safe_browsing, but AFAICT the only reason utility_process_host_ exists is for
reference in the ping callback.  So maybe it can stop being a member variable?

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

Done.

 }
 
 void PwgUtilityProcessHostClient::RunCallback(bool success) {
   BrowserThread::PostTask(
       BrowserThread::UI, FROM_HERE,
       base::Bind(&PwgUtilityProcessHostClient::RunCallbackOnUIThread, this,
                  success));
 }
 
 void PwgUtilityProcessHostClient::RunCallbackOnUIThread(bool success) {
@@ skipping 110 matching lines @@
       }
   }
 
   result.rotate_all_pages = raster_capability.value().rotate_all_pages;
 
   result.reverse_page_order = raster_capability.value().reverse_order_streaming;
   return result;
 }
 
 }  // namespace printing