From 359983179a1070cd758db5febd5ddbae4de749cb Mon Sep 17 00:00:00 2001 From: Shelley Vohr Date: Tue, 4 Feb 2020 19:06:03 +0000 Subject: [PATCH] chore: wrap cb with default invocation --- .../browser/printing/print_preview_message_handler.cc | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/shell/browser/printing/print_preview_message_handler.cc b/shell/browser/printing/print_preview_message_handler.cc index 3a555c4107e97..6823c6047b7c0 100644 --- a/shell/browser/printing/print_preview_message_handler.cc +++ b/shell/browser/printing/print_preview_message_handler.cc @@ -23,6 +23,7 @@ #include "content/public/browser/browser_thread.h" #include "content/public/browser/render_frame_host.h" #include "content/public/browser/web_contents.h" +#include "mojo/public/cpp/bindings/callback_helpers.h" #include "shell/common/api/locker.h" #include "shell/common/node_includes.h" @@ -98,10 +99,16 @@ void PrintPreviewMessageHandler::OnMetafileReadyForPrinting( auto* client = printing::PrintCompositeClient::FromWebContents(web_contents()); DCHECK(client); + + auto callback = + base::BindOnce(&PrintPreviewMessageHandler::OnCompositePdfDocumentDone, + weak_ptr_factory_.GetWeakPtr(), ids); client->DoCompositeDocumentToPdf( params.document_cookie, render_frame_host, content, - base::BindOnce(&PrintPreviewMessageHandler::OnCompositePdfDocumentDone, - weak_ptr_factory_.GetWeakPtr(), ids)); + mojo::WrapCallbackWithDefaultInvokeIfNotRun( + std::move(callback), + printing::mojom::PdfCompositor::Status::kCompositingFailure, + base::ReadOnlySharedMemoryRegion())); } else { ResolvePromise(ids.request_id, base::RefCountedSharedMemoryMapping::CreateFromWholeRegion(