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

commit 4f0be6ea9ee6b8f44a56673a26996a767fd4539c
Author: fgorski <fgorski@chromium.org>
Date: Fri Feb 10 01:22:02 2017

[Offline pages] Refactoring and code fix in BackgroundOffinerTask

This patch deals with 2 issues:
* We need to simplify the logic in BackgroundOffinerTask to reuse it
  when converting background work to JobService for O,
* It turns out that the waiter provided by ChromeBackgroundService
  was not released by the BackgroundOffinerTask in cases when device
  conditions were not met,
* Relevant test are updated: wait is added to make sure lock is
  released in all cases,
* Additionally tests are refactored to apply to new interface:
  boolean result of startBackgroundRequests was never used, hence it was
  removed and replaced with proper lock release.

BUG= 690322 ,683802

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

[modify] https://crrev.com/4f0be6ea9ee6b8f44a56673a26996a767fd4539c/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/BackgroundOfflinerTask.java
[modify] https://crrev.com/4f0be6ea9ee6b8f44a56673a26996a767fd4539c/chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/BackgroundOfflinerTaskTest.java

Dmitry, this bug causes us to take 4 minutes to do nothing, when triggering conditions are not met or we are on svelte devices... Please take a look and let's chat. I'd like to see it merged to M57 if RM allows it.

The bug is already fixed on trunk.

Your change meets the bar and is auto-approved for M57. Please go ahead and merge the CL to branch 2987 manually. Please contact milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), ketakid@(cros), govind@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

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

commit 4da2d1afd4c2ca9f1d40e91e66a012fb6ca7983d
Author: Filip Gorski <fgorski@chromium.org>
Date: Wed Feb 15 22:40:42 2017

[Offline pages] Refactoring and code fix in BackgroundOffinerTask

This patch deals with 2 issues:
* We need to simplify the logic in BackgroundOffinerTask to reuse it
  when converting background work to JobService for O,
* It turns out that the waiter provided by ChromeBackgroundService
  was not released by the BackgroundOffinerTask in cases when device
  conditions were not met,
* Relevant test are updated: wait is added to make sure lock is
  released in all cases,
* Additionally tests are refactored to apply to new interface:
  boolean result of startBackgroundRequests was never used, hence it was
  removed and replaced with proper lock release.

BUG= 690322 ,683802

Review-Url: https://codereview.chromium.org/2685873003
Cr-Commit-Position: refs/heads/master@{#449502}
(cherry picked from commit 4f0be6ea9ee6b8f44a56673a26996a767fd4539c)

Review-Url: https://codereview.chromium.org/2692853011 .
Cr-Commit-Position: refs/branch-heads/2987@{#533}
Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943}

[modify] https://crrev.com/4da2d1afd4c2ca9f1d40e91e66a012fb6ca7983d/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/BackgroundOfflinerTask.java
[modify] https://crrev.com/4da2d1afd4c2ca9f1d40e91e66a012fb6ca7983d/chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/BackgroundOfflinerTaskTest.java