When closing the download shelf, hide the buttons of the dangerous download

warning. Also fix the shelf closing animation.

BUG=13632
TEST=Download an exe (e.g. limewire from downloads.com), so that a "dangerous
download" message appears on the shelf. Then close the shelf. The "Yes" and "No"
buttons should be hidden, and the whole download shelf should slide down with
all items on it in place.

Original review url: http://codereview.chromium.org/159261


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21506 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 2 insertions, 3 deletions

diff --git a/chrome/browser/views/download_shelf_view.cc b/chrome/browser/views/download_shelf_view.cc
index dcd7c01..47f0a74 100644
--- a/chrome/browser/views/download_shelf_view.cc
+++ b/chrome/browser/views/download_shelf_view.cc
@@ -199,7 +199,7 @@ void DownloadShelfView::Layout() {
   // Let our base class layout our child views
   views::View::Layout();
 
-  // If there is not enought room to show the first download item, show the
+  // If there is not enough room to show the first download item, show the
   // "Show all downloads" link to the left to make it more visible that there is
   // something to see.
   bool show_link_only = !CanFitFirstDownloadItem();
@@ -297,8 +297,7 @@ bool DownloadShelfView::IsShowing() const {
 }
 
 bool DownloadShelfView::IsClosing() const {
-  // TODO(estade): This is never called. For now just return false.
-  return false;
+  return shelf_animation_->IsClosing();
 }
 
 void DownloadShelfView::Show() {