[go: up one dir, main page]

Disable printer fetching from OOBE in Chrome OS

Resolves a crash from attempting to print from oobe in M76 due to
CupsPrintersManager not being instantiated on the lock screen. This is
a stopgap to prevent the crash. Followup CL's will resolve the
underlying issue to re-enable printing from OOBE.

(cherry picked from commit 00b83546e761c1bdfff55404d371350a5c919749)

Bug: 971185
Change-Id: Iaaf4b57a4c9509d1b54817891c4aebcca8e0ef37
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1648827
Commit-Queue: Bailey Berro <baileyberro@chromium.org>
Auto-Submit: Bailey Berro <baileyberro@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#668248}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1658928
Reviewed-by: Bailey Berro <baileyberro@chromium.org>
Cr-Commit-Position: refs/branch-heads/3809@{#292}
Cr-Branched-From: d82dec1a818f378c464ba307ddd9c92133eac355-refs/heads/master@{#665002}
diff --git a/chrome/browser/ui/webui/print_preview/local_printer_handler_chromeos.cc b/chrome/browser/ui/webui/print_preview/local_printer_handler_chromeos.cc
index e8d848a..65229b67 100644
--- a/chrome/browser/ui/webui/print_preview/local_printer_handler_chromeos.cc
+++ b/chrome/browser/ui/webui/print_preview/local_printer_handler_chromeos.cc
@@ -158,6 +158,12 @@
   // thread.
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
 
+  // TODO(crbug.com/971823): Re-enable printing from OOBE.
+  if (chromeos::ProfileHelper::IsSigninProfile(profile_)) {
+    std::move(done_callback).Run();
+    return;
+  }
+
   PrinterList printer_list;
   AddPrintersToList(printers_manager_->GetPrinters(PrinterClass::kSaved),
                     &printer_list);