Hook up the Background Loader to StartDownload for offline pages.

Hook up the Background Loader to StartDownload for offline pages.
If the page in the tab is not ready to be captured right from the tab, we will post a background loader
request and exit. The page will be loaded when conditions permit.

BUG=630817
Committed: https://crrev.com/14ba6e1ac8051f481d2bc42b2750d0cb2b5968bd
Cr-Commit-Position: refs/heads/master@{#414526}

[Dmitry Titov, 2016-08-25 01:25:51]

dimich@chromium.org changed reviewers:
+ petewil@chromium.org

[Pete Williamson, 2016-08-25 01:33:54]

LGTM with nit

https://codereview.chromium.org/2274373002/diff/1/chrome/browser/android/offl...
File
chrome/browser/android/offline_pages/downloads/offline_page_download_bridge.cc
(right):

https://codereview.chromium.org/2274373002/diff/1/chrome/browser/android/offl...
chrome/browser/android/offline_pages/downloads/offline_page_download_bridge.cc:160:
// If the page is not loaded enough to be captured. Submit a background loader
Grammar nit - ". Submit" -> ", submit"  (this reads better as one sentence than
two)

[Dmitry Titov, 2016-08-25 02:01:59]

https://codereview.chromium.org/2274373002/diff/1/chrome/browser/android/offl...
File
chrome/browser/android/offline_pages/downloads/offline_page_download_bridge.cc
(right):

https://codereview.chromium.org/2274373002/diff/1/chrome/browser/android/offl...
chrome/browser/android/offline_pages/downloads/offline_page_download_bridge.cc:160:
// If the page is not loaded enough to be captured. Submit a background loader
On 2016/08/25 01:33:53, Pete Williamson wrote:
> Grammar nit - ". Submit" -> ", submit"  (this reads better as one sentence
than
> two)

Done.

[Dmitry Titov, 2016-08-25 19:40:47]

The CQ bit was checked by dimich@chromium.org

[Dmitry Titov, 2016-08-25 19:40:48]

The patchset sent to the CQ was uploaded after l-g-t-m from petewil@chromium.org
Link to the patchset: https://codereview.chromium.org/2274373002/#ps20001
(title: "nit fix")

[commit-bot: I haz the power, 2016-08-25 19:41:38]

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-08-25 20:27:38]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-25 20:30:59]

Description was changed from

==========
Hook up the Background Loader to StartDownload for offline pages.
If the page in the tab is not ready to be captured right from the tab, we will
post a background loader
request and exit. The page will be loaded when conditions permit.

BUG=630817
==========

to

==========
Hook up the Background Loader to StartDownload for offline pages.
If the page in the tab is not ready to be captured right from the tab, we will
post a background loader
request and exit. The page will be loaded when conditions permit.

BUG=630817

Committed: https://crrev.com/14ba6e1ac8051f481d2bc42b2750d0cb2b5968bd
Cr-Commit-Position: refs/heads/master@{#414526}
==========

[commit-bot: I haz the power, 2016-08-25 20:31:00]

Patchset 2 (id:??) landed as
https://crrev.com/14ba6e1ac8051f481d2bc42b2750d0cb2b5968bd
Cr-Commit-Position: refs/heads/master@{#414526}