Fix the Download Notifications for Offline Pages to indicate bytes loaded.

chrome/browser/android/offline_pages/background_loader_offliner.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/android/offline_pages/background_loader_offliner.h"
 
 #include "base/metrics/histogram_macros.h"
 #include "base/sys_info.h"
 #include "chrome/browser/android/offline_pages/offline_page_mhtml_archiver.h"
 #include "chrome/browser/android/offline_pages/offliner_helper.h"
@@ skipping 21 matching lines @@
       save_state_(NONE),
       page_load_state_(SUCCESS),
       page_delay_ms_(kOfflinePageDelayMs),
       weak_ptr_factory_(this) {
   DCHECK(offline_page_model_);
   DCHECK(browser_context_);
 }
 
 BackgroundLoaderOffliner::~BackgroundLoaderOffliner() {}
 
-bool BackgroundLoaderOffliner::LoadAndSave(const SavePageRequest& request,
-                                           const CompletionCallback& callback) {
-  DCHECK(callback);
+// TODO(dimich): Invoke progress_callback as appropriate.
+bool BackgroundLoaderOffliner::LoadAndSave(
+    const SavePageRequest& request,
+    const CompletionCallback& completion_callback,
+    const ProgressCallback& progress_callback) {
+  DCHECK(completion_callback);
+  DCHECK(progress_callback);
 
   if (pending_request_) {
     DVLOG(1) << "Already have pending request";
     return false;
   }
 
   // Do not allow loading for custom tabs clients if 3rd party cookies blocked.
   // TODO(dewittj): Revise api to specify policy rather than hard code to
   // name_space.
   if (request.client_id().name_space == kCCTNamespace &&
@@ skipping 42 matching lines @@
   }
 
   if (!loader_)
     ResetState();
 
   // Invalidate ptrs for all delayed/saving tasks.
   weak_ptr_factory_.InvalidateWeakPtrs();
 
   // Track copy of pending request.
   pending_request_.reset(new SavePageRequest(request));
-  completion_callback_ = callback;
+  completion_callback_ = completion_callback;
 
   // Listen for app foreground/background change.
   app_listener_.reset(new base::android::ApplicationStatusListener(
       base::Bind(&BackgroundLoaderOffliner::OnApplicationStateChange,
                  weak_ptr_factory_.GetWeakPtr())));
 
   // Load page attempt.
   loader_.get()->LoadPage(request.url());
 
   return true;
@@ skipping 238 matching lines @@
 void BackgroundLoaderOffliner::HandleApplicationStateChangeCancel(
     const SavePageRequest& request,
     int64_t offline_id) {
   // If for some reason the request was reset during while waiting for callback
   // ignore the completion callback.
   if (pending_request_ && pending_request_->request_id() != offline_id)
     return;
   completion_callback_.Run(request, RequestStatus::FOREGROUND_CANCELED);
 }
 }  // namespace offline_pages