Issue 685878 link

Starred by 1 user

Issue metadata

Status:	Archived
Owner:	tbarzic@chromium.org
Closed:	Jan 2017
Cc:	xiy...@chromium.org r...@chromium.org █ sduraisamy@chromium.org █ mnilsson@chromium.org
Components:	UI>Shell>Kiosk
EstimatedDays:	----
NextAction:	----
OS:	Chrome
Pri:	1
Type:	Bug
M-58

Long delay before app is launched from network config screen when a network is connected

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

Issue description

On kiosk app launch network config screen (which is shown for non-offline-enabled apps when there is no network connection when kiosk launch is attempted), there can be a long delay between the time a network connection is established and the time kiosk app is launched - I observed delays up to 5 minutes, which is way too long.

Project Member

Comment 1 by bugdroid1@chromium.org, Jan 27 2017

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

commit c85710cea0a6cd68a730dc537d05c970015ef521
Author: tbarzic <tbarzic@chromium.org>
Date: Fri Jan 27 17:37:39 2017

Reset extension downloader when updating external extension cache.

Downloader is reset to force restarting any existing attempts for
extension download.

If downloader had previously started an extension
download, but hasn't yet finished, adding a new request for the
extension would just update existing request and wait for it to finish.
This can be problematic in case the previous extension download kept
failing - e.g. due to network problem. Downloader would retry
downloading the extension, but with an exponential back-off.
This means that request for an extension download when extension list
is updated might be delayed for a long time due to previous failures
(up to 17 minutes; download request allows up to 10 retries before
reporting download failure - final delay seems to be 17 minutes).

Resetting downloader when extension list update is requested will reset
the failure count, and thus reset the back-off value.

BUG= 685878 

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

[modify] https://crrev.com/c85710cea0a6cd68a730dc537d05c970015ef521/chrome/browser/chromeos/extensions/external_cache.cc