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

Issue 686283 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Ongoing automatic downloads snapshots should not cancel manual requests

Project Member Reported by carlosk@chromium.org, Jan 27 2017

Issue description

Note: this only applies once https://codereview.chromium.org/2655673005 lands.

RecentTabHelper ignores overlapping download requests to snapshot a page. This stops the user from creating a overwhelming number of snapshot saves by continuously pressing the downloads button.

It does also has an automatic trigger that allows the latest downloaded snapshot to be upgraded to a higher quality one if needed, once the page reaches what is considered "high" quality. For this to happen a downloads snapshot request is created silently.

The edge case here is that if the user decides to request a snapshot to be saved precisely at the moment when this automatic request is being served, the users' one will be ignored without clear feedback.

We have to make a decision on how to better handle this situation. A few options are:

* To schedule the user's request to be executed later.
* To stop automatically upgrading the latest saved downloads snapshot.
 

Comment 1 by chili@chromium.org, Feb 7 2017

Labels: Hotlist-Fixit
Owner: carlosk@chromium.org
Status: Assigned (was: Untriaged)
Note: https://codereview.chromium.org/2655673005 has landed.

Fixit update: a decision has as explained in the OP as to how to handle this situation. Me and dimich@ are the minimum quorum to make that decision.
Status: WontFix (was: Assigned)
After an offline discussion we decided to not work on this because:
* We don't have user reports on this so it seems it's not really affecting users.
* The edge case described here seems to be really rare.
* Once we update SnapshotController (based off Right Moment Detection ongoing work) there will only be one snapshot trigger per page load.

Sign in to add a comment