mirrors/electron
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions 1

fix: clean up callback handling in webContents.print() (#34894)

Browse source 
* refactor: clean up callback handling in webContents.print()

* chore: update patches

Co-authored-by: PatchUp <73610968+patchup[bot]@users.noreply.github.com>
This commit is contained in:
Shelley Vohr 2022-07-21 12:46:15 +02:00 • committed by GitHub
parent d4e97483aa
commit 1ed191114a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 99 additions and 45 deletions
Download patch file Download diff file Expand all files Collapse all files

144
patches/chromium/printing.patch
View file

@ -134,7 +134,7 @@ index 1e158ecd686e775f656d5a05a9d916ce8f075fa8..20613012d1e6f435c3211d78ec311cf0
void PrintJobWorkerOop::UnregisterServiceManagerClient() { void PrintJobWorkerOop::UnregisterServiceManagerClient() {
diff --git a/chrome/browser/printing/print_view_manager_base.cc b/chrome/browser/printing/print_view_manager_base.cc diff --git a/chrome/browser/printing/print_view_manager_base.cc b/chrome/browser/printing/print_view_manager_base.cc
index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d89882163c93ce89 100644 index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..78f3db8a6fb8b9184cfd41188b11851ac8b8df85 100644
--- a/chrome/browser/printing/print_view_manager_base.cc --- a/chrome/browser/printing/print_view_manager_base.cc
+++ b/chrome/browser/printing/print_view_manager_base.cc +++ b/chrome/browser/printing/print_view_manager_base.cc
@@ -30,8 +30,6 @@ @@ -30,8 +30,6 @@
@ -146,7 +146,22 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
#include "chrome/common/pref_names.h" #include "chrome/common/pref_names.h"
#include "chrome/grit/generated_resources.h" #include "chrome/grit/generated_resources.h"
#include "components/prefs/pref_service.h" #include "components/prefs/pref_service.h"
@@ -87,6 +85,8 @@ using PrintSettingsCallback = @@ -83,10 +81,23 @@ namespace printing {
namespace {
+std::string PrintReasonFromPrintStatus(PrintViewManager::PrintStatus status) {
+ if (status == PrintViewManager::PrintStatus::kInvalid) {
+ return "Invalid printer settings";
+ } else if (status == PrintViewManager::PrintStatus::kCanceled) {
+ return "Print job canceled";
+ } else if (status == PrintViewManager::PrintStatus::kFailed) {
+ return "Print job failed";
+ }
+ return "";
+}
+
using PrintSettingsCallback =
base::OnceCallback<void(std::unique_ptr<PrinterQuery>)>; base::OnceCallback<void(std::unique_ptr<PrinterQuery>)>;
void ShowWarningMessageBox(const std::u16string& message) { void ShowWarningMessageBox(const std::u16string& message) {
@ -155,7 +170,7 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
// Runs always on the UI thread. // Runs always on the UI thread.
static bool is_dialog_shown = false; static bool is_dialog_shown = false;
if (is_dialog_shown) if (is_dialog_shown)
@@ -95,6 +95,7 @@ void ShowWarningMessageBox(const std::u16string& message) { @@ -95,6 +106,7 @@ void ShowWarningMessageBox(const std::u16string& message) {
base::AutoReset<bool> auto_reset(&is_dialog_shown, true); base::AutoReset<bool> auto_reset(&is_dialog_shown, true);
chrome::ShowWarningMessageBox(nullptr, std::u16string(), message); chrome::ShowWarningMessageBox(nullptr, std::u16string(), message);
@ -163,7 +178,7 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
} }
void OnDidGetDefaultPrintSettings( void OnDidGetDefaultPrintSettings(
@@ -144,7 +145,9 @@ void OnDidUpdatePrintSettings( @@ -144,7 +156,9 @@ void OnDidUpdatePrintSettings(
DCHECK_CURRENTLY_ON(content::BrowserThread::UI); DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
DCHECK(printer_query); DCHECK(printer_query);
mojom::PrintPagesParamsPtr params = CreateEmptyPrintPagesParamsPtr(); mojom::PrintPagesParamsPtr params = CreateEmptyPrintPagesParamsPtr();
@ -174,7 +189,7 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
RenderParamsFromPrintSettings(printer_query->settings(), RenderParamsFromPrintSettings(printer_query->settings(),
params->params.get()); params->params.get());
params->params->document_cookie = printer_query->cookie(); params->params->document_cookie = printer_query->cookie();
@@ -172,6 +175,7 @@ void OnDidScriptedPrint( @@ -172,6 +186,7 @@ void OnDidScriptedPrint(
mojom::PrintManagerHost::ScriptedPrintCallback callback) { mojom::PrintManagerHost::ScriptedPrintCallback callback) {
DCHECK_CURRENTLY_ON(content::BrowserThread::UI); DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
mojom::PrintPagesParamsPtr params = CreateEmptyPrintPagesParamsPtr(); mojom::PrintPagesParamsPtr params = CreateEmptyPrintPagesParamsPtr();
@ -182,7 +197,7 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
if (printer_query->last_status() == mojom::ResultCode::kSuccess && if (printer_query->last_status() == mojom::ResultCode::kSuccess &&
printer_query->settings().dpi()) { printer_query->settings().dpi()) {
RenderParamsFromPrintSettings(printer_query->settings(), RenderParamsFromPrintSettings(printer_query->settings(),
@@ -181,7 +185,8 @@ void OnDidScriptedPrint( @@ -181,7 +196,8 @@ void OnDidScriptedPrint(
} }
bool has_valid_cookie = params->params->document_cookie; bool has_valid_cookie = params->params->document_cookie;
bool has_dpi = !params->params->dpi.IsEmpty(); bool has_dpi = !params->params->dpi.IsEmpty();
@ -192,7 +207,7 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
if (has_dpi && has_valid_cookie) { if (has_dpi && has_valid_cookie) {
queue->QueuePrinterQuery(std::move(printer_query)); queue->QueuePrinterQuery(std::move(printer_query));
@@ -196,12 +201,14 @@ PrintViewManagerBase::PrintViewManagerBase(content::WebContents* web_contents) @@ -196,12 +212,14 @@ PrintViewManagerBase::PrintViewManagerBase(content::WebContents* web_contents)
: PrintManager(web_contents), : PrintManager(web_contents),
queue_(g_browser_process->print_job_manager()->queue()) { queue_(g_browser_process->print_job_manager()->queue()) {
DCHECK(queue_); DCHECK(queue_);
@ -207,7 +222,7 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
} }
PrintViewManagerBase::~PrintViewManagerBase() { PrintViewManagerBase::~PrintViewManagerBase() {
@@ -209,7 +216,10 @@ PrintViewManagerBase::~PrintViewManagerBase() { @@ -209,7 +227,10 @@ PrintViewManagerBase::~PrintViewManagerBase() {
DisconnectFromCurrentPrintJob(); DisconnectFromCurrentPrintJob();
} }
@ -219,7 +234,7 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
// Remember the ID for `rfh`, to enable checking that the `RenderFrameHost` // Remember the ID for `rfh`, to enable checking that the `RenderFrameHost`
// is still valid after a possible inner message loop runs in // is still valid after a possible inner message loop runs in
// `DisconnectFromCurrentPrintJob()`. // `DisconnectFromCurrentPrintJob()`.
@@ -237,6 +247,9 @@ bool PrintViewManagerBase::PrintNow(content::RenderFrameHost* rfh) { @@ -237,6 +258,9 @@ bool PrintViewManagerBase::PrintNow(content::RenderFrameHost* rfh) {
#endif #endif
SetPrintingRFH(rfh); SetPrintingRFH(rfh);
@ -229,7 +244,7 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
#if BUILDFLAG(ENABLE_PRINT_CONTENT_ANALYSIS) #if BUILDFLAG(ENABLE_PRINT_CONTENT_ANALYSIS)
enterprise_connectors::ContentAnalysisDelegate::Data scanning_data; enterprise_connectors::ContentAnalysisDelegate::Data scanning_data;
@@ -405,7 +418,8 @@ void PrintViewManagerBase::GetDefaultPrintSettingsReply( @@ -405,7 +429,8 @@ void PrintViewManagerBase::GetDefaultPrintSettingsReply(
void PrintViewManagerBase::ScriptedPrintReply( void PrintViewManagerBase::ScriptedPrintReply(
ScriptedPrintCallback callback, ScriptedPrintCallback callback,
int process_id, int process_id,
@ -239,7 +254,7 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
DCHECK_CURRENTLY_ON(content::BrowserThread::UI); DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
#if BUILDFLAG(ENABLE_OOP_PRINTING) #if BUILDFLAG(ENABLE_OOP_PRINTING)
@@ -420,8 +434,11 @@ void PrintViewManagerBase::ScriptedPrintReply( @@ -420,8 +445,11 @@ void PrintViewManagerBase::ScriptedPrintReply(
return; return;
} }
@ -252,7 +267,7 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
} }
void PrintViewManagerBase::UpdatePrintingEnabled() { void PrintViewManagerBase::UpdatePrintingEnabled() {
@@ -429,8 +446,7 @@ void PrintViewManagerBase::UpdatePrintingEnabled() { @@ -429,8 +457,7 @@ void PrintViewManagerBase::UpdatePrintingEnabled() {
// The Unretained() is safe because ForEachRenderFrameHost() is synchronous. // The Unretained() is safe because ForEachRenderFrameHost() is synchronous.
web_contents()->GetPrimaryMainFrame()->ForEachRenderFrameHost( web_contents()->GetPrimaryMainFrame()->ForEachRenderFrameHost(
base::BindRepeating(&PrintViewManagerBase::SendPrintingEnabled, base::BindRepeating(&PrintViewManagerBase::SendPrintingEnabled,
@ -262,7 +277,7 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
} }
void PrintViewManagerBase::NavigationStopped() { void PrintViewManagerBase::NavigationStopped() {
@@ -546,11 +562,14 @@ void PrintViewManagerBase::DidPrintDocument( @@ -546,11 +573,14 @@ void PrintViewManagerBase::DidPrintDocument(
void PrintViewManagerBase::GetDefaultPrintSettings( void PrintViewManagerBase::GetDefaultPrintSettings(
GetDefaultPrintSettingsCallback callback) { GetDefaultPrintSettingsCallback callback) {
DCHECK_CURRENTLY_ON(content::BrowserThread::UI); DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
@ -277,7 +292,7 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
#if BUILDFLAG(ENABLE_OOP_PRINTING) #if BUILDFLAG(ENABLE_OOP_PRINTING)
if (printing::features::kEnableOopPrintDriversJobPrint.Get() && if (printing::features::kEnableOopPrintDriversJobPrint.Get() &&
!service_manager_client_id_.has_value()) { !service_manager_client_id_.has_value()) {
@@ -588,18 +607,20 @@ void PrintViewManagerBase::UpdatePrintSettings( @@ -588,18 +618,20 @@ void PrintViewManagerBase::UpdatePrintSettings(
base::Value::Dict job_settings, base::Value::Dict job_settings,
UpdatePrintSettingsCallback callback) { UpdatePrintSettingsCallback callback) {
DCHECK_CURRENTLY_ON(content::BrowserThread::UI); DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
@ -299,7 +314,7 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
content::BrowserContext* context = content::BrowserContext* context =
web_contents() ? web_contents()->GetBrowserContext() : nullptr; web_contents() ? web_contents()->GetBrowserContext() : nullptr;
PrefService* prefs = PrefService* prefs =
@@ -609,6 +630,7 @@ void PrintViewManagerBase::UpdatePrintSettings( @@ -609,6 +641,7 @@ void PrintViewManagerBase::UpdatePrintSettings(
if (value > 0) if (value > 0)
job_settings.Set(kSettingRasterizePdfDpi, value); job_settings.Set(kSettingRasterizePdfDpi, value);
} }
@ -307,7 +322,7 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
auto callback_wrapper = auto callback_wrapper =
base::BindOnce(&PrintViewManagerBase::UpdatePrintSettingsReply, base::BindOnce(&PrintViewManagerBase::UpdatePrintSettingsReply,
@@ -640,14 +662,14 @@ void PrintViewManagerBase::ScriptedPrint(mojom::ScriptedPrintParamsPtr params, @@ -640,14 +673,14 @@ void PrintViewManagerBase::ScriptedPrint(mojom::ScriptedPrintParamsPtr params,
// didn't happen for some reason. // didn't happen for some reason.
bad_message::ReceivedBadMessage( bad_message::ReceivedBadMessage(
render_process_host, bad_message::PVMB_SCRIPTED_PRINT_FENCED_FRAME); render_process_host, bad_message::PVMB_SCRIPTED_PRINT_FENCED_FRAME);
@ -324,7 +339,7 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
return; return;
} }
#endif #endif
@@ -685,7 +707,6 @@ void PrintViewManagerBase::PrintingFailed(int32_t cookie, @@ -685,7 +718,6 @@ void PrintViewManagerBase::PrintingFailed(int32_t cookie,
PrintManager::PrintingFailed(cookie, reason); PrintManager::PrintingFailed(cookie, reason);
#if !BUILDFLAG(IS_ANDROID) // Android does not implement this function. #if !BUILDFLAG(IS_ANDROID) // Android does not implement this function.
@ -332,19 +347,19 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
#endif #endif
ReleasePrinterQuery(); ReleasePrinterQuery();
@@ -700,6 +721,11 @@ void PrintViewManagerBase::RemoveObserver(Observer& observer) { @@ -700,6 +732,11 @@ void PrintViewManagerBase::RemoveObserver(Observer& observer) {
} }
void PrintViewManagerBase::ShowInvalidPrinterSettingsError() { void PrintViewManagerBase::ShowInvalidPrinterSettingsError() {
+ if (!callback_.is_null()) { + if (!callback_.is_null()) {
+ std::string cb_str = "Invalid printer settings"; + printing_status_ = PrintStatus::kInvalid;
+ std::move(callback_).Run(printing_succeeded_, cb_str); + TerminatePrintJob(true);
+ } + }
+ +
base::ThreadTaskRunnerHandle::Get()->PostTask( base::ThreadTaskRunnerHandle::Get()->PostTask(
FROM_HERE, base::BindOnce(&ShowWarningMessageBox, FROM_HERE, base::BindOnce(&ShowWarningMessageBox,
l10n_util::GetStringUTF16( l10n_util::GetStringUTF16(
@@ -710,10 +736,12 @@ void PrintViewManagerBase::RenderFrameHostStateChanged( @@ -710,10 +747,12 @@ void PrintViewManagerBase::RenderFrameHostStateChanged(
content::RenderFrameHost* render_frame_host, content::RenderFrameHost* render_frame_host,
content::RenderFrameHost::LifecycleState /*old_state*/, content::RenderFrameHost::LifecycleState /*old_state*/,
content::RenderFrameHost::LifecycleState new_state) { content::RenderFrameHost::LifecycleState new_state) {
@ -357,19 +372,30 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
} }
void PrintViewManagerBase::DidStartLoading() { void PrintViewManagerBase::DidStartLoading() {
@@ -773,6 +801,11 @@ void PrintViewManagerBase::OnJobDone() { @@ -769,7 +808,12 @@ void PrintViewManagerBase::OnJobDone() {
ReleasePrintJob(); // Printing is done, we don't need it anymore.
} // print_job_->is_job_pending() may still be true, depending on the order
// of object registration.
+void PrintViewManagerBase::UserInitCanceled() { - printing_succeeded_ = true;
+ printing_canceled_ = true; + printing_status_ = PrintStatus::kSucceeded;
+ ReleasePrintJob(); + ReleasePrintJob();
+} +}
+ +
void PrintViewManagerBase::OnFailed() { +void PrintViewManagerBase::UserInitCanceled() {
TerminatePrintJob(true); + printing_status_ = PrintStatus::kCanceled;
ReleasePrintJob();
} }
@@ -831,7 +864,10 @@ bool PrintViewManagerBase::CreateNewPrintJob(
@@ -783,7 +827,7 @@ bool PrintViewManagerBase::RenderAllMissingPagesNow() {
// Is the document already complete?
if (print_job_->document() && print_job_->document()->IsComplete()) {
- printing_succeeded_ = true;
+ printing_status_ = PrintStatus::kSucceeded;
return true;
}
@@ -831,7 +875,10 @@ bool PrintViewManagerBase::CreateNewPrintJob(
// Disconnect the current `print_job_`. // Disconnect the current `print_job_`.
auto weak_this = weak_ptr_factory_.GetWeakPtr(); auto weak_this = weak_ptr_factory_.GetWeakPtr();
@ -381,21 +407,37 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
if (!weak_this) if (!weak_this)
return false; return false;
@@ -912,6 +948,13 @@ void PrintViewManagerBase::ReleasePrintJob() { @@ -852,7 +899,7 @@ bool PrintViewManagerBase::CreateNewPrintJob(
#endif
print_job_->AddObserver(*this);
- printing_succeeded_ = false;
+ printing_status_ = PrintStatus::kFailed;
return true;
}
@@ -912,6 +959,11 @@ void PrintViewManagerBase::ReleasePrintJob() {
} }
#endif #endif
+ if (!callback_.is_null()) { + if (!callback_.is_null()) {
+ std::string cb_str = ""; + bool success = printing_status_ == PrintStatus::kSucceeded;
+ if (!printing_succeeded_) + std::move(callback_).Run(success, PrintReasonFromPrintStatus(printing_status_));
+ cb_str = printing_canceled_ ? "canceled" : "failed";
+ std::move(callback_).Run(printing_succeeded_, cb_str);
+ } + }
+ +
if (!print_job_) if (!print_job_)
return; return;
@@ -961,7 +1004,7 @@ bool PrintViewManagerBase::RunInnerMessageLoop() { @@ -919,7 +971,7 @@ void PrintViewManagerBase::ReleasePrintJob() {
// printing_rfh_ should only ever point to a RenderFrameHost with a live
// RenderFrame.
DCHECK(rfh->IsRenderFrameLive());
- GetPrintRenderFrame(rfh)->PrintingDone(printing_succeeded_);
+ GetPrintRenderFrame(rfh)->PrintingDone(printing_status_ == PrintStatus::kSucceeded);
}
print_job_->RemoveObserver(*this);
@@ -961,7 +1013,7 @@ bool PrintViewManagerBase::RunInnerMessageLoop() {
} }
bool PrintViewManagerBase::OpportunisticallyCreatePrintJob(int cookie) { bool PrintViewManagerBase::OpportunisticallyCreatePrintJob(int cookie) {
@ -404,7 +446,7 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
return true; return true;
if (!cookie) { if (!cookie) {
@@ -1069,7 +1112,7 @@ void PrintViewManagerBase::SendPrintingEnabled(bool enabled, @@ -1069,7 +1121,7 @@ void PrintViewManagerBase::SendPrintingEnabled(bool enabled,
} }
void PrintViewManagerBase::CompletePrintNow(content::RenderFrameHost* rfh) { void PrintViewManagerBase::CompletePrintNow(content::RenderFrameHost* rfh) {
@ -414,7 +456,7 @@ index 4bd871e5cadc693aea6e8c71b3fe3296b743d9ec..e6201cbf8d1ed64b1b53ed43d8988216
for (auto& observer : GetObservers()) for (auto& observer : GetObservers())
observer.OnPrintNow(rfh); observer.OnPrintNow(rfh);
diff --git a/chrome/browser/printing/print_view_manager_base.h b/chrome/browser/printing/print_view_manager_base.h diff --git a/chrome/browser/printing/print_view_manager_base.h b/chrome/browser/printing/print_view_manager_base.h
index 746df417a23f7760818ba265d4a7d589dec8bf34..0027387d4717c59f2df3f279caf7aa0de6669400 100644 index 746df417a23f7760818ba265d4a7d589dec8bf34..0d3e4491826be629c7251a69afc7ebde990e10e1 100644
--- a/chrome/browser/printing/print_view_manager_base.h --- a/chrome/browser/printing/print_view_manager_base.h
+++ b/chrome/browser/printing/print_view_manager_base.h +++ b/chrome/browser/printing/print_view_manager_base.h
@@ -41,6 +41,8 @@ namespace printing { @@ -41,6 +41,8 @@ namespace printing {
@ -446,7 +488,22 @@ index 746df417a23f7760818ba265d4a7d589dec8bf34..0027387d4717c59f2df3f279caf7aa0d
// Adds and removes observers for `PrintViewManagerBase` events. The order in // Adds and removes observers for `PrintViewManagerBase` events. The order in
// which notifications are sent to observers is undefined. Observers must be // which notifications are sent to observers is undefined. Observers must be
@@ -241,7 +247,8 @@ class PrintViewManagerBase : public PrintManager, public PrintJob::Observer { @@ -120,6 +126,14 @@ class PrintViewManagerBase : public PrintManager, public PrintJob::Observer {
void AddObserver(Observer& observer);
void RemoveObserver(Observer& observer);
+ enum class PrintStatus {
+ kSucceeded,
+ kCanceled,
+ kFailed,
+ kInvalid,
+ kUnknown
+ };
+
protected:
explicit PrintViewManagerBase(content::WebContents* web_contents);
@@ -241,7 +255,8 @@ class PrintViewManagerBase : public PrintManager, public PrintJob::Observer {
// Runs `callback` with `params` to reply to ScriptedPrint(). // Runs `callback` with `params` to reply to ScriptedPrint().
void ScriptedPrintReply(ScriptedPrintCallback callback, void ScriptedPrintReply(ScriptedPrintCallback callback,
int process_id, int process_id,
@ -456,7 +513,7 @@ index 746df417a23f7760818ba265d4a7d589dec8bf34..0027387d4717c59f2df3f279caf7aa0d
// Requests the RenderView to render all the missing pages for the print job. // Requests the RenderView to render all the missing pages for the print job.
// No-op if no print job is pending. Returns true if at least one page has // No-op if no print job is pending. Returns true if at least one page has
@@ -314,9 +321,15 @@ class PrintViewManagerBase : public PrintManager, public PrintJob::Observer { @@ -314,8 +329,11 @@ class PrintViewManagerBase : public PrintManager, public PrintJob::Observer {
// The current RFH that is printing with a system printing dialog. // The current RFH that is printing with a system printing dialog.
raw_ptr<content::RenderFrameHost> printing_rfh_ = nullptr; raw_ptr<content::RenderFrameHost> printing_rfh_ = nullptr;
@ -464,14 +521,11 @@ index 746df417a23f7760818ba265d4a7d589dec8bf34..0027387d4717c59f2df3f279caf7aa0d
+ CompletionCallback callback_; + CompletionCallback callback_;
+ +
// Indication of success of the print job. // Indication of success of the print job.
bool printing_succeeded_ = false; - bool printing_succeeded_ = false;
+ PrintStatus printing_status_ = PrintStatus::kUnknown;
+ // Indication of whether the print job was manually canceled
+ bool printing_canceled_ = false;
+
// Set while running an inner message loop inside RenderAllMissingPagesNow(). // Set while running an inner message loop inside RenderAllMissingPagesNow().
// This means we are _blocking_ until all the necessary pages have been // This means we are _blocking_ until all the necessary pages have been
// rendered or the print settings are being loaded.
diff --git a/chrome/browser/ui/webui/print_preview/fake_print_render_frame.cc b/chrome/browser/ui/webui/print_preview/fake_print_render_frame.cc diff --git a/chrome/browser/ui/webui/print_preview/fake_print_render_frame.cc b/chrome/browser/ui/webui/print_preview/fake_print_render_frame.cc
index f3c3f85edb19489d079dc93411be64828ae581e2..ff303dcbc034cd8f1530fe1543729e98d3035826 100644 index f3c3f85edb19489d079dc93411be64828ae581e2..ff303dcbc034cd8f1530fe1543729e98d3035826 100644
--- a/chrome/browser/ui/webui/print_preview/fake_print_render_frame.cc --- a/chrome/browser/ui/webui/print_preview/fake_print_render_frame.cc