Makes the download shelf auto-close after the user opens all downloads

chrome/browser/download/download_item.cc

 // Copyright (c) 2010 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_item.h"
 
 #include "app/l10n_util.h"
 #include "base/file_util.h"
 #include "base/logging.h"
 #include "base/timer.h"
@@ skipping 45 matching lines @@
       safety_state_(SAFE),
       auto_opened_(false),
       original_name_(info.original_name),
       render_process_id_(-1),
       request_id_(-1),
       save_as_(false),
       is_otr_(false),
       is_extension_install_(info.is_extension_install),
       name_finalized_(false),
       is_temporary_(false),
-      need_final_rename_(false) {
+      need_final_rename_(false),
+      opened_(false) {
   if (state_ == IN_PROGRESS)
     state_ = CANCELLED;
   Init(false /* don't start progress timer */);
 }
 
 // Constructing for a regular download:
 DownloadItem::DownloadItem(DownloadManager* download_manager,
                            const DownloadCreateInfo& info,
                            bool is_otr)
     : id_(info.download_id),
@@ skipping 15 matching lines @@
       safety_state_(info.is_dangerous ? DANGEROUS : SAFE),
       auto_opened_(false),
       original_name_(info.original_name),
       render_process_id_(info.child_id),
       request_id_(info.request_id),
       save_as_(info.prompt_user_for_save_location),
       is_otr_(is_otr),
       is_extension_install_(info.is_extension_install),
       name_finalized_(false),
       is_temporary_(!info.save_info.file_path.empty()),
-      need_final_rename_(false) {
+      need_final_rename_(false),
+      opened_(false) {
   Init(true /* start progress timer */);
 }
 
 // Constructing for the "Save Page As..." feature:
 DownloadItem::DownloadItem(DownloadManager* download_manager,
                            const FilePath& path,
                            const GURL& url,
                            bool is_otr)
     : id_(1),
       full_path_(path),
@@ skipping 14 matching lines @@
       safety_state_(SAFE),
       auto_opened_(false),
       original_name_(FilePath()),
       render_process_id_(-1),
       request_id_(-1),
       save_as_(false),
       is_otr_(is_otr),
       is_extension_install_(false),
       name_finalized_(false),
       is_temporary_(false),
-      need_final_rename_(false) {
+      need_final_rename_(false),
+      opened_(false) {
   Init(true /* start progress timer */);
 }
 
 DownloadItem::~DownloadItem() {
   state_ = REMOVING;
   UpdateObservers();
 }
 
 void DownloadItem::AddObserver(Observer* observer) {
   observers_.AddObserver(observer);
@@ skipping 25 matching lines @@
 }
 
 void DownloadItem::OpenFilesBasedOnExtension(bool open) {
   return download_manager_->OpenFilesBasedOnExtension(full_path(), open);
 }
 
 void DownloadItem::OpenDownload() {
   if (state() == DownloadItem::IN_PROGRESS) {
     open_when_complete_ = !open_when_complete_;
   } else if (state() == DownloadItem::COMPLETE) {
-    FOR_EACH_OBSERVER(Observer, observers_, OnDownloadOpened(this));
     download_manager_->OpenDownload(this, NULL);
   }
 }
 
+void DownloadItem::Opened() {
+  opened_ = true;
+  FOR_EACH_OBSERVER(Observer, observers_, OnDownloadOpened(this));
+}
+
 void DownloadItem::ShowDownloadInShell() {
   download_manager_->ShowDownloadInShell(this);
 }
 
 void DownloadItem::DangerousDownloadValidated() {
   download_manager_->DangerousDownloadValidated(this);
 }
 
 void DownloadItem::UpdateSize(int64 bytes_so_far) {
   received_bytes_ = bytes_so_far;
@@ skipping 144 matching lines @@
     return name;
   }
   return original_name_;
 }
 
 void DownloadItem::Init(bool start_timer) {
   file_name_ = full_path_.BaseName();
   if (start_timer)
     StartProgressTimer();
 }