New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 713916 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: May 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Print Jobs Restart when Opening the Print Dialog

Project Member Reported by skau@chromium.org, Apr 20 2017

Issue description

Because we reinitialize a printer every time we open the print dialog, the queues are restarting jobs.  This means that if a multi-page job was printing, it restarts.
 

Comment 1 by skau@chromium.org, Apr 28 2017

Status: Started (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, May 10 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d571f3812f6e6031ce0da38c6a667cbabfd31e36

commit d571f3812f6e6031ce0da38c6a667cbabfd31e36
Author: skau <skau@chromium.org>
Date: Wed May 10 20:27:49 2017

Track printer installations for each configuration.

Track which printers have been configured in the system by tracking
the timestamp for their most recently installed configuration.  This
prevents reinstalling printers which can cause print queues to restart.

BUG= 713916 

Review-Url: https://codereview.chromium.org/2858353004
Cr-Commit-Position: refs/heads/master@{#470686}

[modify] https://crrev.com/d571f3812f6e6031ce0da38c6a667cbabfd31e36/chrome/browser/chromeos/printing/printers_manager.cc
[modify] https://crrev.com/d571f3812f6e6031ce0da38c6a667cbabfd31e36/chrome/browser/chromeos/printing/printers_manager.h
[modify] https://crrev.com/d571f3812f6e6031ce0da38c6a667cbabfd31e36/chrome/browser/chromeos/printing/printers_manager_unittest.cc
[modify] https://crrev.com/d571f3812f6e6031ce0da38c6a667cbabfd31e36/chrome/browser/chromeos/printing/specifics_translation.cc
[modify] https://crrev.com/d571f3812f6e6031ce0da38c6a667cbabfd31e36/chrome/browser/chromeos/printing/specifics_translation_unittest.cc
[modify] https://crrev.com/d571f3812f6e6031ce0da38c6a667cbabfd31e36/chrome/browser/ui/webui/print_preview/printer_backend_proxy_chromeos.cc
[modify] https://crrev.com/d571f3812f6e6031ce0da38c6a667cbabfd31e36/chrome/browser/ui/webui/settings/chromeos/cups_printers_handler.cc
[modify] https://crrev.com/d571f3812f6e6031ce0da38c6a667cbabfd31e36/chromeos/printing/printer_configuration.cc
[modify] https://crrev.com/d571f3812f6e6031ce0da38c6a667cbabfd31e36/chromeos/printing/printer_configuration.h
[modify] https://crrev.com/d571f3812f6e6031ce0da38c6a667cbabfd31e36/chromeos/printing/printer_translator.cc
[modify] https://crrev.com/d571f3812f6e6031ce0da38c6a667cbabfd31e36/chromeos/printing/printer_translator.h
[modify] https://crrev.com/d571f3812f6e6031ce0da38c6a667cbabfd31e36/chromeos/printing/printer_translator_unittest.cc

Comment 3 by skau@chromium.org, May 10 2017

Status: Fixed (was: Started)
Chrome now remembers what printers were installed and when.  So printers won't be installed repeatedly if the configuration doesn't change.

Comment 4 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

Comment 5 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment