Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(415)

Side by Side Diff: chrome/browser/plugin_installer.cc

Issue 10837125: Revert 149794 - DownloadItem::Observer::OnDownloadDestroyed() replaces DownloadItem::REMOVING (Closed) Base URL: svn://svn.chromium.org/chrome/branches/1228/src/
Patch Set: Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/browser/plugin_installer.h ('k') | chrome/browser/ui/cocoa/download/download_item_mac.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/plugin_installer.h" 5 #include "chrome/browser/plugin_installer.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/bind_helpers.h" 8 #include "base/bind_helpers.h"
9 #include "base/file_util.h" 9 #include "base/file_util.h"
10 #include "base/process.h" 10 #include "base/process.h"
(...skipping 132 matching lines...) Expand 10 before | Expand all | Expand 10 after
143 DCHECK_EQ(INSTALLER_STATE_DOWNLOADING, state_); 143 DCHECK_EQ(INSTALLER_STATE_DOWNLOADING, state_);
144 state_ = INSTALLER_STATE_IDLE; 144 state_ = INSTALLER_STATE_IDLE;
145 FOR_EACH_OBSERVER(PluginInstallerObserver, observers_, 145 FOR_EACH_OBSERVER(PluginInstallerObserver, observers_,
146 DownloadFinished()); 146 DownloadFinished());
147 break; 147 break;
148 } 148 }
149 case DownloadItem::CANCELLED: { 149 case DownloadItem::CANCELLED: {
150 DownloadCancelled(); 150 DownloadCancelled();
151 break; 151 break;
152 } 152 }
153 case DownloadItem::REMOVING: {
154 DCHECK_EQ(INSTALLER_STATE_DOWNLOADING, state_);
155 state_ = INSTALLER_STATE_IDLE;
156 break;
157 }
153 case DownloadItem::INTERRUPTED: { 158 case DownloadItem::INTERRUPTED: {
154 content::DownloadInterruptReason reason = download->GetLastReason(); 159 content::DownloadInterruptReason reason = download->GetLastReason();
155 DownloadError(content::InterruptReasonDebugString(reason)); 160 DownloadError(content::InterruptReasonDebugString(reason));
156 break; 161 break;
157 } 162 }
158 case DownloadItem::MAX_DOWNLOAD_STATE: { 163 case DownloadItem::MAX_DOWNLOAD_STATE: {
159 NOTREACHED(); 164 NOTREACHED();
160 return; 165 return;
161 } 166 }
162 } 167 }
163 download->RemoveObserver(this); 168 download->RemoveObserver(this);
164 } 169 }
165 170
166 void PluginInstaller::OnDownloadDestroyed(DownloadItem* download) { 171 void PluginInstaller::OnDownloadOpened(DownloadItem* download) {
167 DCHECK_EQ(INSTALLER_STATE_DOWNLOADING, state_);
168 state_ = INSTALLER_STATE_IDLE;
169 download->RemoveObserver(this);
170 } 172 }
171 173
172 void PluginInstaller::AddObserver(PluginInstallerObserver* observer) { 174 void PluginInstaller::AddObserver(PluginInstallerObserver* observer) {
173 observers_.AddObserver(observer); 175 observers_.AddObserver(observer);
174 } 176 }
175 177
176 void PluginInstaller::RemoveObserver(PluginInstallerObserver* observer) { 178 void PluginInstaller::RemoveObserver(PluginInstallerObserver* observer) {
177 observers_.RemoveObserver(observer); 179 observers_.RemoveObserver(observer);
178 if (observers_.size() == weak_observers_.size()) { 180 if (observers_.size() == weak_observers_.size()) {
179 FOR_EACH_OBSERVER(WeakPluginInstallerObserver, weak_observers_, 181 FOR_EACH_OBSERVER(WeakPluginInstallerObserver, weak_observers_,
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
243 DCHECK_EQ(INSTALLER_STATE_DOWNLOADING, state_); 245 DCHECK_EQ(INSTALLER_STATE_DOWNLOADING, state_);
244 state_ = INSTALLER_STATE_IDLE; 246 state_ = INSTALLER_STATE_IDLE;
245 FOR_EACH_OBSERVER(PluginInstallerObserver, observers_, DownloadError(msg)); 247 FOR_EACH_OBSERVER(PluginInstallerObserver, observers_, DownloadError(msg));
246 } 248 }
247 249
248 void PluginInstaller::DownloadCancelled() { 250 void PluginInstaller::DownloadCancelled() {
249 DCHECK_EQ(INSTALLER_STATE_DOWNLOADING, state_); 251 DCHECK_EQ(INSTALLER_STATE_DOWNLOADING, state_);
250 state_ = INSTALLER_STATE_IDLE; 252 state_ = INSTALLER_STATE_IDLE;
251 FOR_EACH_OBSERVER(PluginInstallerObserver, observers_, DownloadCancelled()); 253 FOR_EACH_OBSERVER(PluginInstallerObserver, observers_, DownloadCancelled());
252 } 254 }
OLDNEW
« no previous file with comments | « chrome/browser/plugin_installer.h ('k') | chrome/browser/ui/cocoa/download/download_item_mac.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698