From 9439388b7b424c8b09954851274fd4b13faf797e Mon Sep 17 00:00:00 2001 From: Shelley Vohr Date: Wed, 5 Feb 2020 18:24:41 +0000 Subject: [PATCH] chore: wrap cb with default invocation (#22036) --- .../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(