Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(178)

Unified Diff: chrome/browser/printing/pwg_raster_converter.cc

Issue 1885813002: Delete the utility process startup ping. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/printing/pwg_raster_converter.cc
diff --git a/chrome/browser/printing/pwg_raster_converter.cc b/chrome/browser/printing/pwg_raster_converter.cc
index c457a983df2f5bb77d8a2c398f0d0a61d84acd2b..eb220c08a8f026abf6260c00cfbc245ad26dd7c0 100644
--- a/chrome/browser/printing/pwg_raster_converter.cc
+++ b/chrome/browser/printing/pwg_raster_converter.cc
@@ -130,7 +130,6 @@ class PwgUtilityProcessHostClient : public content::UtilityProcessHostClient {
~PwgUtilityProcessHostClient() override;
// Message handlers.
- void OnProcessStarted();
void OnSucceeded();
void OnFailed();
@@ -145,7 +144,6 @@ class PwgUtilityProcessHostClient : public content::UtilityProcessHostClient {
PdfRenderSettings settings_;
PwgRasterSettings bitmap_settings_;
PWGRasterConverter::ResultCallback callback_;
- base::WeakPtr<content::UtilityProcessHost> utility_process_host_;
DISALLOW_COPY_AND_ASSIGN(PwgUtilityProcessHostClient);
};
@@ -180,7 +178,6 @@ 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,
@@ -190,19 +187,6 @@ bool PwgUtilityProcessHostClient::OnMessageReceived(
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);
@@ -226,12 +210,14 @@ void PwgUtilityProcessHostClient::OnFilesReadyOnUIThread() {
void PwgUtilityProcessHostClient::StartProcessOnIOThread() {
DCHECK_CURRENTLY_ON(BrowserThread::IO);
- utility_process_host_ =
+ content::UtilityProcessHost* utility_process_host =
content::UtilityProcessHost::Create(
- this, base::MessageLoop::current()->task_runner())->AsWeakPtr();
- utility_process_host_->SetName(l10n_util::GetStringUTF16(
+ this, base::MessageLoop::current()->task_runner());
+ 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()));
}
void PwgUtilityProcessHostClient::RunCallback(bool success) {
« no previous file with comments | « chrome/browser/printing/pdf_to_emf_converter.cc ('k') | chrome/browser/safe_browsing/sandboxed_dmg_analyzer_mac.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698