From 0fa06d6dae64b1a14851e088b9b66da429382c7e Mon Sep 17 00:00:00 2001 From: Keeley Hammond Date: Mon, 26 Jan 2026 14:32:31 -0800 Subject: [PATCH] fix(patch-conflict): update printing.patch for base::DictValue rename Updated printing.patch to use the new base::DictValue type name instead of base::Value::Dict following Chromium's type renaming change. This affects CompleteUpdatePrintSettings() signature and related code. Ref: https://chromium-review.googlesource.com/c/chromium/src/+/7509820 --- patches/chromium/printing.patch | 26 +++++++++++++------------- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/patches/chromium/printing.patch b/patches/chromium/printing.patch index 0a0cee12a5..d413e97edf 100644 --- a/patches/chromium/printing.patch +++ b/patches/chromium/printing.patch @@ -68,7 +68,7 @@ index f91857eb0b6ad385721b8224100de26dfdd7dd8d..45e8766fcb8d46d8edc3bf8d21d3f826 : PdfRenderSettings::Mode::POSTSCRIPT_LEVEL3; } diff --git a/chrome/browser/printing/print_view_manager_base.cc b/chrome/browser/printing/print_view_manager_base.cc -index 57d09902647c4f257728589e494f71793121f015..c6b544eb72b11baca99770b6b98566b4ab7adc5d 100644 +index aa79c324af2cec50019bca3bccff5d420fb30ffd..3bba39372750b3e3efd1c5a4747f41415ae3ede7 100644 --- a/chrome/browser/printing/print_view_manager_base.cc +++ b/chrome/browser/printing/print_view_manager_base.cc @@ -80,6 +80,20 @@ namespace printing { @@ -187,7 +187,7 @@ index 57d09902647c4f257728589e494f71793121f015..c6b544eb72b11baca99770b6b98566b4 void PrintViewManagerBase::CompleteUpdatePrintSettings( + std::unique_ptr printer_query, - base::Value::Dict job_settings, + base::DictValue job_settings, std::unique_ptr print_settings, UpdatePrintSettingsCallback callback) { @@ -332,7 +353,8 @@ void PrintViewManagerBase::CompleteUpdatePrintSettings( @@ -497,7 +497,7 @@ index 57d09902647c4f257728589e494f71793121f015..c6b544eb72b11baca99770b6b98566b4 scoped_refptr print_data, const gfx::Size& page_size, diff --git a/chrome/browser/printing/print_view_manager_base.h b/chrome/browser/printing/print_view_manager_base.h -index 824d6d8a9242302c2ab0c0c517d0cc329e607100..d3b25c90c90fe860990789aa96c872b45132720b 100644 +index cdd25ff24507cc49684454820c7d385b8c67259e..85b80282902b17aad49d4ffe40fddb1a0125d4ef 100644 --- a/chrome/browser/printing/print_view_manager_base.h +++ b/chrome/browser/printing/print_view_manager_base.h @@ -46,6 +46,8 @@ namespace printing { @@ -560,7 +560,7 @@ index 824d6d8a9242302c2ab0c0c517d0cc329e607100..d3b25c90c90fe860990789aa96c872b4 #endif void CompleteUpdatePrintSettings( + std::unique_ptr printer_query, - base::Value::Dict job_settings, + base::DictValue job_settings, std::unique_ptr print_settings, UpdatePrintSettingsCallback callback); @@ -376,8 +391,11 @@ class PrintViewManagerBase : public PrintManager, public PrintJob::Observer { @@ -577,10 +577,10 @@ index 824d6d8a9242302c2ab0c0c517d0cc329e607100..d3b25c90c90fe860990789aa96c872b4 // Indication that the job is getting canceled. bool canceling_job_ = false; diff --git a/chrome/browser/printing/printer_query.cc b/chrome/browser/printing/printer_query.cc -index c31c873db21ef06b6d0f19c789c9a097f3cfeb2b..94a05c657a895cadc98324f7fef133c3d206fbd9 100644 +index 15bd25f0d99b38c2d3a01b5bab78173ea9fef23c..27de3e2b4de7052af5d60a0c60b8ceec79869a49 100644 --- a/chrome/browser/printing/printer_query.cc +++ b/chrome/browser/printing/printer_query.cc -@@ -356,17 +356,19 @@ void PrinterQuery::UpdatePrintSettings(base::Value::Dict new_settings, +@@ -356,17 +356,19 @@ void PrinterQuery::UpdatePrintSettings(base::DictValue new_settings, #endif // BUILDFLAG(IS_LINUX) && BUILDFLAG(USE_CUPS) } @@ -605,7 +605,7 @@ index c31c873db21ef06b6d0f19c789c9a097f3cfeb2b..94a05c657a895cadc98324f7fef133c3 #if BUILDFLAG(IS_CHROMEOS) diff --git a/chrome/browser/printing/printer_query_oop.cc b/chrome/browser/printing/printer_query_oop.cc -index f58d494764ea77c997770ae0257b7c21b248a0e0..6eb4a9436a9bee5420a29ec643b984cea0700faa 100644 +index 0dd564edd45425c75830f5e078f6fb375d10460b..61e0af99682502056f77426f6d17ee33463df6ae 100644 --- a/chrome/browser/printing/printer_query_oop.cc +++ b/chrome/browser/printing/printer_query_oop.cc @@ -126,7 +126,7 @@ void PrinterQueryOop::OnDidAskUserForSettings( @@ -666,7 +666,7 @@ index ac2f719be566020d9f41364560c12e6d6d0fe3d8..16d758a6936f66148a196761cfb875f6 PrintingFailed(int32 cookie, PrintFailureReason reason); diff --git a/components/printing/renderer/print_render_frame_helper.cc b/components/printing/renderer/print_render_frame_helper.cc -index 1750c48c5dcff0ec3f45497a0949bdefe01bc730..73ac20c027c44fa0e1d5df94ce050ebb992c10e9 100644 +index 60b5e83a8bc1ed07970be4cdfdc19962698bd754..23db93ae0528d96b8be14bbbb64483cbdd9c46ec 100644 --- a/components/printing/renderer/print_render_frame_helper.cc +++ b/components/printing/renderer/print_render_frame_helper.cc @@ -54,6 +54,7 @@ @@ -735,7 +735,7 @@ index 1750c48c5dcff0ec3f45497a0949bdefe01bc730..73ac20c027c44fa0e1d5df94ce050ebb if (render_frame_gone_) { return; -@@ -1483,6 +1487,8 @@ void PrintRenderFrameHelper::PrintPreview(base::Value::Dict settings) { +@@ -1483,6 +1487,8 @@ void PrintRenderFrameHelper::PrintPreview(base::DictValue settings) { if (ipc_nesting_level_ > kAllowedIpcDepthForPrint) return; @@ -844,7 +844,7 @@ index 1750c48c5dcff0ec3f45497a0949bdefe01bc730..73ac20c027c44fa0e1d5df94ce050ebb } diff --git a/components/printing/renderer/print_render_frame_helper.h b/components/printing/renderer/print_render_frame_helper.h -index 37f84d2001e7e1dff029f1556b7676a71ac2f6f2..8c89e826165db6de2c681c8fa97cd91a8706b0ae 100644 +index 8b4cea7d43fe16d7414e017b11ee1cda722e1a8d..eb7982ac4fcba9b40cbff83ebe71e42493c97fba 100644 --- a/components/printing/renderer/print_render_frame_helper.h +++ b/components/printing/renderer/print_render_frame_helper.h @@ -258,7 +258,7 @@ class PrintRenderFrameHelper @@ -887,19 +887,19 @@ index 37f84d2001e7e1dff029f1556b7676a71ac2f6f2..8c89e826165db6de2c681c8fa97cd91a ScriptingThrottler scripting_throttler_; diff --git a/printing/printing_context.cc b/printing/printing_context.cc -index 746a056e1c7980803323739c51cedb930c62de25..34a9e45a3407612d6960a7cad798cf30989f065e 100644 +index 4b004cf94b93137a4a64b79eab49b8d216891846..45c14773307612708c487e29e2cced8429f12850 100644 --- a/printing/printing_context.cc +++ b/printing/printing_context.cc @@ -157,7 +157,6 @@ void PrintingContext::UsePdfSettings() { mojom::ResultCode PrintingContext::UpdatePrintSettings( - base::Value::Dict job_settings) { + base::DictValue job_settings) { - ResetSettings(); { std::unique_ptr settings = PrintSettingsFromJobSettings(job_settings); diff --git a/printing/printing_context.h b/printing/printing_context.h -index 2f34f45aaf89e6f4600be1d2b8444c636b0cc83f..5bfd45c5efbf300a36e016af80e18fb9ad376d41 100644 +index 01970ac4c5c38182774353762ee138dbd9af184c..fa261eec18ee102d9b3ab152ce5540ffa5d6512f 100644 --- a/printing/printing_context.h +++ b/printing/printing_context.h @@ -208,6 +208,9 @@ class COMPONENT_EXPORT(PRINTING) PrintingContext {