[Download Home] Use fewer AsyncTasks

[Download Home] Use fewer AsyncTasks

Download updates cause a new AsyncTask to be created and executed,
resulting in a potential to overrun the Executor's queue size.
While it will result in less up-to-date values for free space,
settle for just having one AsyncTask that can be in play at once
by explicitly checking if another AsyncTask is already running.

Moreover, catch any exceptions that may fire if we still overflow
the queue with our single task.

BUG=669493
Committed: https://crrev.com/731bb5908b6f2c782efc2fd1ef05a3ec72d1d822
Cr-Commit-Position: refs/heads/master@{#435745}

[gone, 2016-11-29 19:54:08]

dfalcantara@chromium.org changed reviewers:
+ twellington@chromium.org

[gone, 2016-12-01 18:22:17]

dfalcantara@chromium.org changed reviewers:
+ tedchoc@chromium.org
- twellington@chromium.org

[gone, 2016-12-01 18:22:18]

Swapping out; Ted can you take a look?

[Ted C, 2016-12-01 18:32:42]

lgtm

[gone, 2016-12-01 18:33:13]

The CQ bit was checked by dfalcantara@chromium.org

[commit-bot: I haz the power, 2016-12-01 18:33:31]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-01 20:06:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-01 20:06:09]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[gone, 2016-12-01 22:10:13]

The CQ bit was checked by dfalcantara@chromium.org

[gone, 2016-12-01 22:10:13]

The patchset sent to the CQ was uploaded after l-g-t-m from tedchoc@chromium.org
Link to the patchset: https://codereview.chromium.org/2540813002/#ps40001
(title: "Remove accidental thing")

[commit-bot: I haz the power, 2016-12-01 22:11:02]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-01 22:46:20]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1480630213273520,
"parent_rev": "d51137677b723c6fd59b11e6722a90c0bcf3355a", "commit_rev":
"3306e66574fe11137906514b4e2367b97aeea6ce"}

[commit-bot: I haz the power, 2016-12-01 22:46:45]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-12-01 22:49:04]

Description was changed from

==========
[Download Home] Use fewer AsyncTasks

Download updates cause a new AsyncTask to be created and executed,
resulting in a potential to overrun the Executor's queue size.
While it will result in less up-to-date values for free space,
settle for just having one AsyncTask that can be in play at once
by explicitly checking if another AsyncTask is already running.

Moreover, catch any exceptions that may fire if we still overflow
the queue with our single task.

BUG=669493
==========

to

==========
[Download Home] Use fewer AsyncTasks

Download updates cause a new AsyncTask to be created and executed,
resulting in a potential to overrun the Executor's queue size.
While it will result in less up-to-date values for free space,
settle for just having one AsyncTask that can be in play at once
by explicitly checking if another AsyncTask is already running.

Moreover, catch any exceptions that may fire if we still overflow
the queue with our single task.

BUG=669493

Committed: https://crrev.com/731bb5908b6f2c782efc2fd1ef05a3ec72d1d822
Cr-Commit-Position: refs/heads/master@{#435745}
==========

[commit-bot: I haz the power, 2016-12-01 22:49:06]

Patchset 3 (id:??) landed as
https://crrev.com/731bb5908b6f2c782efc2fd1ef05a3ec72d1d822
Cr-Commit-Position: refs/heads/master@{#435745}