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

Issue 601858 link

Starred by 4 users

Issue metadata

Status: Archived
Owner:
Closed: Sep 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Feature



Sign in to add a comment

Feature Request: Adjust time to live for "Recent Destinations" in print dialog

Project Member Reported by mshe...@chromium.org, Apr 8 2016

Issue description

The time to live for recent destinations in the print dialog seems quite short in practice. The FR is for this to be extended or changed in some way (perhaps not removing the last 3 used printers)...

@Lei - Do you know what the current behavior is here precisely? Or who can we talk to to figure it out?

I'm not sure that this is the most urgent issue, but is one worth looking into (perhaps via an informal user study)
 
Cc: -thestig@chromium.org dbeam@chromium.org rltoscano@chromium.org abodenha@chromium.org dpa...@chromium.org
Does anyone who implemented this / reviewed the CLs e.g. [1] remember what Recent Destinations was suppose to be like? It appears I was CC'd on [1], but I probably ignored it because I was not working on printing at the time.

[1] https://chromiumcodereview.appspot.com/10450022
For reference in terms of where this FR comes from, this was based off of this original feedback report by mbp@: https://feedback.corp.google.com/product/208/neutron?lView=rd&lRSort=1&lROrder=2&lRFilter=1&lReportSearch=user%3Dmbp@google.com&lReport=7768401019
The feedback page doesn't load for me, based on what mbp said in a related email thread, it sounds like maybe "Recent Destinations" is only remembered for the duration of the browser session, and not saved between restarts.
Owner: rbpotter@chromium.org
Status: Started (was: Assigned)
Project Member

Comment 5 by bugdroid1@chromium.org, Sep 22 2016

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

commit 6d799882e49984de8620d41c53893101eaed92f6
Author: rbpotter <rbpotter@chromium.org>
Date: Thu Sep 22 02:06:12 2016

Save most recent 3 destinations for print across multiple
prints and multiple browser sessions. Note: can increase from
3 easily if needed.

Add browser test to check behavior.

BUG= 601858 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[modify] https://crrev.com/6d799882e49984de8620d41c53893101eaed92f6/chrome/browser/resources/print_preview/data/app_state.js
[modify] https://crrev.com/6d799882e49984de8620d41c53893101eaed92f6/chrome/browser/resources/print_preview/data/destination_store.js
[modify] https://crrev.com/6d799882e49984de8620d41c53893101eaed92f6/chrome/test/data/webui/print_preview.js

Status: Fixed (was: Started)

Comment 7 by dchan@chromium.org, Oct 7 2016

Labels: VerifyIn-55

Comment 8 by dchan@google.com, Nov 19 2016

Labels: VerifyIn-56

Comment 9 by dchan@google.com, Jan 21 2017

Labels: VerifyIn-57

Comment 10 by jawag@chromium.org, Jan 31 2017

Cc: rbpotter@chromium.org
 Issue 147762  has been merged into this issue.

Comment 11 by dchan@google.com, Mar 4 2017

Labels: VerifyIn-58

Comment 12 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

Comment 13 by dchan@google.com, May 30 2017

Labels: VerifyIn-60
Labels: VerifyIn-61

Comment 15 by dchan@chromium.org, Oct 14 2017

Status: Archived (was: Fixed)

Sign in to add a comment