Revert 121031 - Eliminate DownloadProgressUpdated.

chrome/browser/download/download_status_updater.cc

 // Copyright (c) 2012 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/download/download_status_updater.h"
 
 #include <vector>
 
 #include "base/logging.h"
 #include "base/stl_util.h"
-#include "chrome/browser/download/download_util.h"
 
 DownloadStatusUpdater::DownloadStatusUpdater() {
 }
 
 DownloadStatusUpdater::~DownloadStatusUpdater() {
   for (std::set<content::DownloadManager*>::iterator it = managers_.begin();
        it != managers_.end(); ++it)
     (*it)->RemoveObserver(this);
 
   for (std::set<content::DownloadItem*>::iterator it = items_.begin();
@@ skipping 30 matching lines @@
 
 // Methods inherited from content::DownloadManager::Observer.
 void DownloadStatusUpdater::ModelChanged(content::DownloadManager* manager) {
   std::vector<content::DownloadItem*> downloads;
   manager->SearchDownloads(string16(), &downloads);
 
   for (std::vector<content::DownloadItem*>::iterator it = downloads.begin();
        it != downloads.end(); ++it) {
     UpdateItem(*it);
   }
-
-  UpdateAppIconDownloadProgress();
 }
 
 void DownloadStatusUpdater::ManagerGoingDown(
     content::DownloadManager* manager) {
   DCHECK(ContainsKey(managers_, manager));
   managers_.erase(manager);
   manager->RemoveObserver(this);
   // Item removal will be handled in response to DownloadItem REMOVING
   // notification (in the !IN_PROGRESS conditional branch in UpdateItem).
 }
 
 void DownloadStatusUpdater::SelectFileDialogDisplayed(
     content::DownloadManager* manager, int32 id) {
 }
 
 // Methods inherited from content::DownloadItem::Observer.
 void DownloadStatusUpdater::OnDownloadUpdated(
     content::DownloadItem* download) {
   UpdateItem(download);
-  UpdateAppIconDownloadProgress();
 }
 
 void DownloadStatusUpdater::OnDownloadOpened(content::DownloadItem* download) {
 }
 
-void DownloadStatusUpdater::UpdateAppIconDownloadProgress() {
-  float progress = 0;
-  int download_count = 0;
-  bool progress_known = GetProgress(&progress, &download_count);
-  download_util::UpdateAppIconDownloadProgress(download_count,
-                                               progress_known,
-                                               progress);
-}
-
 // React to a transition that a download associated with one of our
 // download managers has made.  Our goal is to have only IN_PROGRESS
 // items on our set list, as they're the only ones that have relevance
 // to GetProgress() return values.
 void DownloadStatusUpdater::UpdateItem(content::DownloadItem* download) {
   if (download->GetState() == content::DownloadItem::IN_PROGRESS) {
     if (!ContainsKey(items_, download)) {
       items_.insert(download);
       download->AddObserver(this);
     }
   } else {
     if (ContainsKey(items_, download)) {
       items_.erase(download);
       download->RemoveObserver(this);
     }
   }
 }