Reduce CPU usage of download shelf UI. (both MD and pre-MD shelves)

chrome/browser/ui/views/download/download_item_view_md.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/ui/views/download/download_item_view_md.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "base/bind.h"
@@ skipping 181 matching lines @@
   // ExperienceSampling: If the user took no action to remove the warning
   // before it disappeared, then the user effectively dismissed the download
   // without keeping it.
   if (sampling_event_.get())
     sampling_event_->CreateUserDecisionEvent(ExperienceSamplingEvent::kIgnore);
 }
 
 // Progress animation handlers.
 
 void DownloadItemViewMd::StartDownloadProgress() {
-  if (progress_timer_.IsRunning())
+  if (progress_start_time_ != base::TimeTicks())
     return;
   progress_start_time_ = base::TimeTicks::Now();
-  progress_timer_.Start(
-      FROM_HERE,
-      base::TimeDelta::FromMilliseconds(DownloadShelf::kProgressRateMs),
-      base::Bind(&DownloadItemViewMd::SchedulePaint, base::Unretained(this)));
 }
 
 void DownloadItemViewMd::StopDownloadProgress() {
-  if (!progress_timer_.IsRunning())
+  if (progress_start_time_ == base::TimeTicks())
     return;
   previous_progress_elapsed_ += base::TimeTicks::Now() - progress_start_time_;
   progress_start_time_ = base::TimeTicks();
-  progress_timer_.Stop();
 }
 
 // static
 SkColor DownloadItemViewMd::GetTextColorForThemeProvider(
     const ui::ThemeProvider* theme) {
   return theme ? theme->GetColor(ThemeProperties::COLOR_BOOKMARK_TEXT)
                : gfx::kPlaceholderColor;
 }
 
 void DownloadItemViewMd::OnExtractIconComplete(gfx::Image* icon_bitmap) {
@@ skipping 20 matching lines @@
     ShowWarningDialog();
     // Force the shelf to layout again as our size has changed.
     shelf_->Layout();
     SchedulePaint();
   } else {
     base::string16 status_text = model_.GetStatusText();
     switch (download()->GetState()) {
       case DownloadItem::IN_PROGRESS:
         download()->IsPaused() ? StopDownloadProgress()
                                : StartDownloadProgress();
+        SchedulePaint();
         LoadIconIfItemPathChanged();
         break;
       case DownloadItem::INTERRUPTED:
         StopDownloadProgress();
         complete_animation_.reset(new gfx::SlideAnimation(this));
         complete_animation_->SetSlideDuration(kInterruptedAnimationDurationMs);
         complete_animation_->SetTweenType(gfx::Tween::LINEAR);
         complete_animation_->Show();
         SchedulePaint();
         LoadIcon();
@@ skipping 23 matching lines @@
     status_text_ = status_text;
   }
 
   base::string16 new_tip = model_.GetTooltipText(font_list_, kTooltipMaxWidth);
   if (new_tip != tooltip_text_) {
     tooltip_text_ = new_tip;
     TooltipTextChanged();
   }
 
   UpdateAccessibleName();
-
-  // We use the parent's (DownloadShelfView's) SchedulePaint, since there
-  // are spaces between each DownloadItemViewMd that the parent is responsible
-  // for painting.
-  shelf_->SchedulePaint();
 }
 
 void DownloadItemViewMd::OnDownloadDestroyed(DownloadItem* download) {
   shelf_->RemoveDownloadView(this);  // This will delete us!
 }
 
 void DownloadItemViewMd::OnDownloadOpened(DownloadItem* download) {
   disabled_while_opening_ = true;
   SetEnabled(false);
   base::MessageLoop::current()->task_runner()->PostDelayedTask(
@@ skipping 757 matching lines @@
   }
 }
 
 SkColor DownloadItemViewMd::GetTextColor() {
   return GetTextColorForThemeProvider(GetThemeProvider());
 }
 
 SkColor DownloadItemViewMd::GetDimmedTextColor() {
   return SkColorSetA(GetTextColor(), 0xC7);
 }