Fixes to enable indication of bytes downloaded for Offline Pages in Download Home.

chrome/browser/android/offline_pages/prerendering_offliner.cc

Index: chrome/browser/android/offline_pages/prerendering_offliner.cc
diff --git a/chrome/browser/android/offline_pages/prerendering_offliner.cc b/chrome/browser/android/offline_pages/prerendering_offliner.cc
index 5ceaf65abb73b74ad687d0f6a7122477991bb41d..95257a9a421768fbd831d87216ec6ed94084fed1 100644
--- a/chrome/browser/android/offline_pages/prerendering_offliner.cc
+++ b/chrome/browser/android/offline_pages/prerendering_offliner.cc
@@ -18,9 +18,6 @@
 #include "content/public/browser/web_contents.h"
 
 namespace offline_pages {
-namespace {
-const char kDownloadUIAdapterKey[] = "download-ui-adapter";
-}
 
 PrerenderingOffliner::PrerenderingOffliner(
     content::BrowserContext* browser_context,
@@ -39,10 +36,8 @@ void PrerenderingOffliner::OnNetworkProgress(const SavePageRequest& request,
                                              int64_t bytes) {
   if (!pending_request_)
     return;
-
-  DownloadUIAdapter* ui_adapter = static_cast<DownloadUIAdapter*>(
-      offline_page_model_->GetUserData(kDownloadUIAdapterKey));
-
+  DownloadUIAdapter* ui_adapter =
+      DownloadUIAdapter::FromOfflinePageModel(offline_page_model_);

[Pete Williamson, 2017/02/27 16:13:56]

Should we be making a similar change to the background loader offliner?

[fgorski, 2017/02/27 16:22:55]

Yes, if similar implementation is in place it causes trouble.

[Dmitry Titov, 2017/02/27 20:02:17]

If you mean adding the ability for Background Loader to report network bytes,
this is not yet implemented and Cathy is looking into this. It will be using a
different mechanism because there is no prerenderer.

   if (!ui_adapter)
     return;