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

components/offline_pages/core/downloads/download_ui_adapter.cc

 // Copyright 2017 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 "components/offline_pages/core/downloads/download_ui_adapter.h"
 
 #include "base/bind.h"
 #include "base/guid.h"
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
@@ skipping 146 matching lines @@
   items_[guid] = base::MakeUnique<ItemInfo>(request, temporarily_hidden);
 
   if (state_ != State::LOADED)
     return;
 
   const DownloadUIItem& download_ui_item = *(items_[guid]->ui_item);
   for (Observer& observer : observers_)
     observer.ItemUpdated(download_ui_item);
 }
 
+void DownloadUIAdapter::OnNetworkProgress(const SavePageRequest& request,
+                                          int64_t received_bytes) {
+  for (auto& item : items_) {
+    if (item.second->is_request &&
+        item.second->offline_id == request.request_id()) {
+      if (received_bytes == item.second->ui_item->download_progress_bytes)
+        return;
+
+      item.second->ui_item->download_progress_bytes = received_bytes;
+      for (Observer& observer : observers_)
+        observer.ItemUpdated(*(item.second->ui_item));
+      return;
+    }
+  }
+}
+
 void DownloadUIAdapter::TemporaryHiddenStatusChanged(
     const ClientId& client_id) {
   bool hidden = delegate_->IsTemporarilyHiddenInUI(client_id);
 
   for (const auto& item : items_) {
     if (item.second->client_id == client_id) {
       if (item.second->temporarily_hidden == hidden)
         continue;
       item.second->temporarily_hidden = hidden;
       if (hidden) {
@@ skipping 42 matching lines @@
 int64_t DownloadUIAdapter::GetOfflineIdByGuid(const std::string& guid) const {
   if (deleting_item_ && deleting_item_->ui_item->guid == guid)
     return deleting_item_->offline_id;
 
   DownloadUIItems::const_iterator it = items_.find(guid);
   if (it != items_.end())
     return it->second->offline_id;
   return 0;
 }
 
-void DownloadUIAdapter::UpdateProgress(int64_t offline_id, int64_t bytes) {
-  for (auto& item : items_) {
-    if (item.second->is_request && item.second->offline_id == offline_id) {
-      if (bytes == item.second->ui_item->download_progress_bytes)
-        return;
-
-      item.second->ui_item->download_progress_bytes = bytes;
-      for (Observer& observer : observers_)
-        observer.ItemUpdated(*(item.second->ui_item));
-    }
-  }
-}
-
 // Note that several LoadCache calls may be issued before the async GetAllPages
 // comes back.
 void DownloadUIAdapter::LoadCache() {
   state_ = State::LOADING_PAGES;
   model_->GetAllPages(base::Bind(&DownloadUIAdapter::OnOfflinePagesLoaded,
                                  weak_ptr_factory_.GetWeakPtr()));
 }
 
 void DownloadUIAdapter::ClearCache() {
   // Once loaded, this class starts to observe the model. Only remove observer
@@ skipping 108 matching lines @@
 
   if (!deleting_item_->temporarily_hidden && state_ == State::LOADED) {
     for (Observer& observer : observers_)
       observer.ItemDeleted(guid);
   }
 
   deleting_item_.reset();
 }
 
 }  // namespace offline_pages