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
3 // found in the LICENSE file.
5 #include "content/browser/download/download_manager_impl.h"
10 #include "base/callback.h"
11 #include "base/debug/alias.h"
12 #include "base/i18n/case_conversion.h"
13 #include "base/logging.h"
14 #include "base/message_loop/message_loop.h"
15 #include "base/stl_util.h"
16 #include "base/strings/stringprintf.h"
17 #include "base/strings/sys_string_conversions.h"
18 #include "base/supports_user_data.h"
19 #include "base/synchronization/lock.h"
20 #include "build/build_config.h"
21 #include "content/browser/byte_stream.h"
22 #include "content/browser/download/download_create_info.h"
23 #include "content/browser/download/download_file_factory.h"
24 #include "content/browser/download/download_item_factory.h"
25 #include "content/browser/download/download_item_impl.h"
26 #include "content/browser/download/download_stats.h"
27 #include "content/browser/loader/resource_dispatcher_host_impl.h"
28 #include "content/browser/renderer_host/render_view_host_impl.h"
29 #include "content/browser/web_contents/web_contents_impl.h"
30 #include "content/public/browser/browser_context.h"
31 #include "content/public/browser/browser_thread.h"
32 #include "content/public/browser/content_browser_client.h"
33 #include "content/public/browser/download_interrupt_reasons.h"
34 #include "content/public/browser/download_manager_delegate.h"
35 #include "content/public/browser/download_url_parameters.h"
36 #include "content/public/browser/notification_service.h"
37 #include "content/public/browser/notification_types.h"
38 #include "content/public/browser/render_process_host.h"
39 #include "content/public/browser/resource_context.h"
40 #include "content/public/browser/web_contents_delegate.h"
41 #include "content/public/common/referrer.h"
42 #include "net/base/elements_upload_data_stream.h"
43 #include "net/base/load_flags.h"
44 #include "net/base/request_priority.h"
45 #include "net/base/upload_bytes_element_reader.h"
46 #include "net/url_request/url_request_context.h"
51 void BeginDownload(scoped_ptr
<DownloadUrlParameters
> params
,
53 DCHECK_CURRENTLY_ON(BrowserThread::IO
);
54 // ResourceDispatcherHost{Base} is-not-a URLRequest::Delegate, and
55 // DownloadUrlParameters can-not include resource_dispatcher_host_impl.h, so
56 // we must down cast. RDHI is the only subclass of RDH as of 2012 May 4.
57 scoped_ptr
<net::URLRequest
> request(
58 params
->resource_context()->GetRequestContext()->CreateRequest(
59 params
->url(), net::DEFAULT_PRIORITY
, NULL
));
60 request
->set_method(params
->method());
61 if (!params
->post_body().empty()) {
62 const std::string
& body
= params
->post_body();
63 scoped_ptr
<net::UploadElementReader
> reader(
64 net::UploadOwnedBytesElementReader::CreateWithString(body
));
66 net::ElementsUploadDataStream::CreateWithReader(reader
.Pass(), 0));
68 if (params
->post_id() >= 0) {
69 // The POST in this case does not have an actual body, and only works
70 // when retrieving data from cache. This is done because we don't want
71 // to do a re-POST without user consent, and currently don't have a good
72 // plan on how to display the UI for that.
73 DCHECK(params
->prefer_cache());
74 DCHECK_EQ("POST", params
->method());
75 ScopedVector
<net::UploadElementReader
> element_readers
;
76 request
->set_upload(make_scoped_ptr(
77 new net::ElementsUploadDataStream(element_readers
.Pass(),
81 // If we're not at the beginning of the file, retrieve only the remaining
83 bool has_last_modified
= !params
->last_modified().empty();
84 bool has_etag
= !params
->etag().empty();
86 // If we've asked for a range, we want to make sure that we only
87 // get that range if our current copy of the information is good.
88 // We shouldn't be asked to continue if we don't have a verifier.
89 DCHECK(params
->offset() == 0 || has_etag
|| has_last_modified
);
91 if (params
->offset() > 0) {
92 request
->SetExtraRequestHeaderByName(
94 base::StringPrintf("bytes=%" PRId64
"-", params
->offset()),
97 if (has_last_modified
) {
98 request
->SetExtraRequestHeaderByName("If-Unmodified-Since",
99 params
->last_modified(),
103 request
->SetExtraRequestHeaderByName("If-Match", params
->etag(), true);
107 for (DownloadUrlParameters::RequestHeadersType::const_iterator iter
108 = params
->request_headers_begin();
109 iter
!= params
->request_headers_end();
111 request
->SetExtraRequestHeaderByName(
112 iter
->first
, iter
->second
, false /*overwrite*/);
115 scoped_ptr
<DownloadSaveInfo
> save_info(new DownloadSaveInfo());
116 save_info
->file_path
= params
->file_path();
117 save_info
->suggested_name
= params
->suggested_name();
118 save_info
->offset
= params
->offset();
119 save_info
->hash_state
= params
->hash_state();
120 save_info
->prompt_for_save_location
= params
->prompt();
121 save_info
->file
= params
->GetFile();
123 ResourceDispatcherHost::Get()->BeginDownload(
126 params
->content_initiated(),
127 params
->resource_context(),
128 params
->render_process_host_id(),
129 params
->render_view_host_routing_id(),
130 params
->prefer_cache(),
131 params
->do_not_prompt_for_login(),
137 class MapValueIteratorAdapter
{
139 explicit MapValueIteratorAdapter(
140 base::hash_map
<int64
, DownloadItem
*>::const_iterator iter
)
143 ~MapValueIteratorAdapter() {}
145 DownloadItem
* operator*() { return iter_
->second
; }
147 MapValueIteratorAdapter
& operator++() {
152 bool operator!=(const MapValueIteratorAdapter
& that
) const {
153 return iter_
!= that
.iter_
;
157 base::hash_map
<int64
, DownloadItem
*>::const_iterator iter_
;
158 // Allow copy and assign.
161 class DownloadItemFactoryImpl
: public DownloadItemFactory
{
163 DownloadItemFactoryImpl() {}
164 ~DownloadItemFactoryImpl() override
{}
166 DownloadItemImpl
* CreatePersistedItem(
167 DownloadItemImplDelegate
* delegate
,
169 const base::FilePath
& current_path
,
170 const base::FilePath
& target_path
,
171 const std::vector
<GURL
>& url_chain
,
172 const GURL
& referrer_url
,
173 const std::string
& mime_type
,
174 const std::string
& original_mime_type
,
175 const base::Time
& start_time
,
176 const base::Time
& end_time
,
177 const std::string
& etag
,
178 const std::string
& last_modified
,
179 int64 received_bytes
,
181 DownloadItem::DownloadState state
,
182 DownloadDangerType danger_type
,
183 DownloadInterruptReason interrupt_reason
,
185 const net::BoundNetLog
& bound_net_log
) override
{
186 return new DownloadItemImpl(
208 DownloadItemImpl
* CreateActiveItem(
209 DownloadItemImplDelegate
* delegate
,
211 const DownloadCreateInfo
& info
,
212 const net::BoundNetLog
& bound_net_log
) override
{
213 return new DownloadItemImpl(delegate
, download_id
, info
, bound_net_log
);
216 DownloadItemImpl
* CreateSavePageItem(
217 DownloadItemImplDelegate
* delegate
,
219 const base::FilePath
& path
,
221 const std::string
& mime_type
,
222 scoped_ptr
<DownloadRequestHandleInterface
> request_handle
,
223 const net::BoundNetLog
& bound_net_log
) override
{
224 return new DownloadItemImpl(delegate
, download_id
, path
, url
,
225 mime_type
, request_handle
.Pass(),
232 DownloadManagerImpl::DownloadManagerImpl(
233 net::NetLog
* net_log
,
234 BrowserContext
* browser_context
)
235 : item_factory_(new DownloadItemFactoryImpl()),
236 file_factory_(new DownloadFileFactory()),
238 shutdown_needed_(true),
239 browser_context_(browser_context
),
242 weak_factory_(this) {
243 DCHECK(browser_context
);
246 DownloadManagerImpl::~DownloadManagerImpl() {
247 DCHECK(!shutdown_needed_
);
250 DownloadItemImpl
* DownloadManagerImpl::CreateActiveItem(
251 uint32 id
, const DownloadCreateInfo
& info
) {
252 DCHECK_CURRENTLY_ON(BrowserThread::UI
);
253 DCHECK(!ContainsKey(downloads_
, id
));
254 net::BoundNetLog bound_net_log
=
255 net::BoundNetLog::Make(net_log_
, net::NetLog::SOURCE_DOWNLOAD
);
256 DownloadItemImpl
* download
=
257 item_factory_
->CreateActiveItem(this, id
, info
, bound_net_log
);
258 downloads_
[id
] = download
;
262 void DownloadManagerImpl::GetNextId(const DownloadIdCallback
& callback
) {
263 DCHECK_CURRENTLY_ON(BrowserThread::UI
);
265 delegate_
->GetNextId(callback
);
268 static uint32 next_id
= content::DownloadItem::kInvalidId
+ 1;
269 callback
.Run(next_id
++);
272 void DownloadManagerImpl::DetermineDownloadTarget(
273 DownloadItemImpl
* item
, const DownloadTargetCallback
& callback
) {
274 // Note that this next call relies on
275 // DownloadItemImplDelegate::DownloadTargetCallback and
276 // DownloadManagerDelegate::DownloadTargetCallback having the same
277 // type. If the types ever diverge, gasket code will need to
279 if (!delegate_
|| !delegate_
->DetermineDownloadTarget(item
, callback
)) {
280 base::FilePath target_path
= item
->GetForcedFilePath();
281 // TODO(asanka): Determine a useful path if |target_path| is empty.
282 callback
.Run(target_path
,
283 DownloadItem::TARGET_DISPOSITION_OVERWRITE
,
284 DOWNLOAD_DANGER_TYPE_NOT_DANGEROUS
,
289 bool DownloadManagerImpl::ShouldCompleteDownload(
290 DownloadItemImpl
* item
, const base::Closure
& complete_callback
) {
292 delegate_
->ShouldCompleteDownload(item
, complete_callback
)) {
295 // Otherwise, the delegate has accepted responsibility to run the
296 // callback when the download is ready for completion.
300 bool DownloadManagerImpl::ShouldOpenFileBasedOnExtension(
301 const base::FilePath
& path
) {
305 return delegate_
->ShouldOpenFileBasedOnExtension(path
);
308 bool DownloadManagerImpl::ShouldOpenDownload(
309 DownloadItemImpl
* item
, const ShouldOpenDownloadCallback
& callback
) {
313 // Relies on DownloadItemImplDelegate::ShouldOpenDownloadCallback and
314 // DownloadManagerDelegate::DownloadOpenDelayedCallback "just happening"
315 // to have the same type :-}.
316 return delegate_
->ShouldOpenDownload(item
, callback
);
319 void DownloadManagerImpl::SetDelegate(DownloadManagerDelegate
* delegate
) {
320 delegate_
= delegate
;
323 DownloadManagerDelegate
* DownloadManagerImpl::GetDelegate() const {
327 void DownloadManagerImpl::Shutdown() {
328 DVLOG(20) << __FUNCTION__
<< "()"
329 << " shutdown_needed_ = " << shutdown_needed_
;
330 if (!shutdown_needed_
)
332 shutdown_needed_
= false;
334 FOR_EACH_OBSERVER(Observer
, observers_
, ManagerGoingDown(this));
335 // TODO(benjhayden): Consider clearing observers_.
337 // If there are in-progress downloads, cancel them. This also goes for
338 // dangerous downloads which will remain in history if they aren't explicitly
339 // accepted or discarded. Canceling will remove the intermediate download
341 for (DownloadMap::iterator it
= downloads_
.begin(); it
!= downloads_
.end();
343 DownloadItemImpl
* download
= it
->second
;
344 if (download
->GetState() == DownloadItem::IN_PROGRESS
)
345 download
->Cancel(false);
347 STLDeleteValues(&downloads_
);
349 // We'll have nothing more to report to the observers after this point.
353 delegate_
->Shutdown();
357 void DownloadManagerImpl::StartDownload(
358 scoped_ptr
<DownloadCreateInfo
> info
,
359 scoped_ptr
<ByteStreamReader
> stream
,
360 const DownloadUrlParameters::OnStartedCallback
& on_started
) {
361 DCHECK_CURRENTLY_ON(BrowserThread::UI
);
363 uint32 download_id
= info
->download_id
;
364 const bool new_download
= (download_id
== content::DownloadItem::kInvalidId
);
365 base::Callback
<void(uint32
)> got_id(base::Bind(
366 &DownloadManagerImpl::StartDownloadWithId
,
367 weak_factory_
.GetWeakPtr(),
368 base::Passed(info
.Pass()),
369 base::Passed(stream
.Pass()),
375 got_id
.Run(download_id
);
379 void DownloadManagerImpl::StartDownloadWithId(
380 scoped_ptr
<DownloadCreateInfo
> info
,
381 scoped_ptr
<ByteStreamReader
> stream
,
382 const DownloadUrlParameters::OnStartedCallback
& on_started
,
385 DCHECK_CURRENTLY_ON(BrowserThread::UI
);
386 DCHECK_NE(content::DownloadItem::kInvalidId
, id
);
387 DownloadItemImpl
* download
= NULL
;
389 download
= CreateActiveItem(id
, *info
);
391 DownloadMap::iterator item_iterator
= downloads_
.find(id
);
392 // Trying to resume an interrupted download.
393 if (item_iterator
== downloads_
.end() ||
394 (item_iterator
->second
->GetState() == DownloadItem::CANCELLED
)) {
395 // If the download is no longer known to the DownloadManager, then it was
396 // removed after it was resumed. Ignore. If the download is cancelled
397 // while resuming, then also ignore the request.
398 info
->request_handle
.CancelRequest();
399 if (!on_started
.is_null())
400 on_started
.Run(NULL
, DOWNLOAD_INTERRUPT_REASON_USER_CANCELED
);
403 download
= item_iterator
->second
;
404 DCHECK_EQ(DownloadItem::INTERRUPTED
, download
->GetState());
405 download
->MergeOriginInfoOnResume(*info
);
408 base::FilePath default_download_directory
;
410 base::FilePath website_save_directory
; // Unused
411 bool skip_dir_check
= false; // Unused
412 delegate_
->GetSaveDir(GetBrowserContext(), &website_save_directory
,
413 &default_download_directory
, &skip_dir_check
);
416 // Create the download file and start the download.
417 scoped_ptr
<DownloadFile
> download_file(
418 file_factory_
->CreateFile(
419 info
->save_info
.Pass(), default_download_directory
,
420 info
->url(), info
->referrer_url
,
421 delegate_
&& delegate_
->GenerateFileHash(),
422 stream
.Pass(), download
->GetBoundNetLog(),
423 download
->DestinationObserverAsWeakPtr()));
425 // Attach the client ID identifying the app to the AV system.
426 if (download_file
.get() && delegate_
) {
427 download_file
->SetClientGuid(
428 delegate_
->ApplicationClientIdForFileScanning());
431 scoped_ptr
<DownloadRequestHandleInterface
> req_handle(
432 new DownloadRequestHandle(info
->request_handle
));
433 download
->Start(download_file
.Pass(), req_handle
.Pass());
435 // For interrupted downloads, Start() will transition the state to
436 // IN_PROGRESS and consumers will be notified via OnDownloadUpdated().
437 // For new downloads, we notify here, rather than earlier, so that
438 // the download_file is bound to download and all the usual
439 // setters (e.g. Cancel) work.
441 FOR_EACH_OBSERVER(Observer
, observers_
, OnDownloadCreated(this, download
));
443 if (!on_started
.is_null())
444 on_started
.Run(download
, DOWNLOAD_INTERRUPT_REASON_NONE
);
447 void DownloadManagerImpl::CheckForHistoryFilesRemoval() {
448 DCHECK_CURRENTLY_ON(BrowserThread::UI
);
449 for (DownloadMap::iterator it
= downloads_
.begin();
450 it
!= downloads_
.end(); ++it
) {
451 DownloadItemImpl
* item
= it
->second
;
452 CheckForFileRemoval(item
);
456 void DownloadManagerImpl::CheckForFileRemoval(DownloadItemImpl
* download_item
) {
457 DCHECK_CURRENTLY_ON(BrowserThread::UI
);
458 if ((download_item
->GetState() == DownloadItem::COMPLETE
) &&
459 !download_item
->GetFileExternallyRemoved() &&
461 delegate_
->CheckForFileExistence(
463 base::Bind(&DownloadManagerImpl::OnFileExistenceChecked
,
464 weak_factory_
.GetWeakPtr(), download_item
->GetId()));
468 void DownloadManagerImpl::OnFileExistenceChecked(uint32 download_id
,
470 DCHECK_CURRENTLY_ON(BrowserThread::UI
);
471 if (!result
) { // File does not exist.
472 if (ContainsKey(downloads_
, download_id
))
473 downloads_
[download_id
]->OnDownloadedFileRemoved();
477 BrowserContext
* DownloadManagerImpl::GetBrowserContext() const {
478 return browser_context_
;
481 void DownloadManagerImpl::CreateSavePackageDownloadItem(
482 const base::FilePath
& main_file_path
,
483 const GURL
& page_url
,
484 const std::string
& mime_type
,
485 scoped_ptr
<DownloadRequestHandleInterface
> request_handle
,
486 const DownloadItemImplCreated
& item_created
) {
487 DCHECK_CURRENTLY_ON(BrowserThread::UI
);
488 GetNextId(base::Bind(
489 &DownloadManagerImpl::CreateSavePackageDownloadItemWithId
,
490 weak_factory_
.GetWeakPtr(),
494 base::Passed(request_handle
.Pass()),
498 void DownloadManagerImpl::CreateSavePackageDownloadItemWithId(
499 const base::FilePath
& main_file_path
,
500 const GURL
& page_url
,
501 const std::string
& mime_type
,
502 scoped_ptr
<DownloadRequestHandleInterface
> request_handle
,
503 const DownloadItemImplCreated
& item_created
,
505 DCHECK_CURRENTLY_ON(BrowserThread::UI
);
506 DCHECK_NE(content::DownloadItem::kInvalidId
, id
);
507 DCHECK(!ContainsKey(downloads_
, id
));
508 net::BoundNetLog bound_net_log
=
509 net::BoundNetLog::Make(net_log_
, net::NetLog::SOURCE_DOWNLOAD
);
510 DownloadItemImpl
* download_item
= item_factory_
->CreateSavePageItem(
516 request_handle
.Pass(),
518 downloads_
[download_item
->GetId()] = download_item
;
519 FOR_EACH_OBSERVER(Observer
, observers_
, OnDownloadCreated(
520 this, download_item
));
521 if (!item_created
.is_null())
522 item_created
.Run(download_item
);
525 void DownloadManagerImpl::OnSavePackageSuccessfullyFinished(
526 DownloadItem
* download_item
) {
527 FOR_EACH_OBSERVER(Observer
, observers_
,
528 OnSavePackageSuccessfullyFinished(this, download_item
));
531 // Resume a download of a specific URL. We send the request to the
532 // ResourceDispatcherHost, and let it send us responses like a regular
534 void DownloadManagerImpl::ResumeInterruptedDownload(
535 scoped_ptr
<content::DownloadUrlParameters
> params
,
537 RecordDownloadSource(INITIATED_BY_RESUMPTION
);
538 BrowserThread::PostTask(
541 base::Bind(&BeginDownload
, base::Passed(¶ms
), id
));
544 void DownloadManagerImpl::SetDownloadItemFactoryForTesting(
545 scoped_ptr
<DownloadItemFactory
> item_factory
) {
546 item_factory_
= item_factory
.Pass();
549 void DownloadManagerImpl::SetDownloadFileFactoryForTesting(
550 scoped_ptr
<DownloadFileFactory
> file_factory
) {
551 file_factory_
= file_factory
.Pass();
554 DownloadFileFactory
* DownloadManagerImpl::GetDownloadFileFactoryForTesting() {
555 return file_factory_
.get();
558 void DownloadManagerImpl::DownloadRemoved(DownloadItemImpl
* download
) {
562 uint32 download_id
= download
->GetId();
563 if (downloads_
.erase(download_id
) == 0)
568 int DownloadManagerImpl::RemoveDownloadsBetween(base::Time remove_begin
,
569 base::Time remove_end
) {
571 DownloadMap::const_iterator it
= downloads_
.begin();
572 while (it
!= downloads_
.end()) {
573 DownloadItemImpl
* download
= it
->second
;
575 // Increment done here to protect against invalidation below.
578 if (download
->GetStartTime() >= remove_begin
&&
579 (remove_end
.is_null() || download
->GetStartTime() < remove_end
) &&
580 (download
->GetState() != DownloadItem::IN_PROGRESS
)) {
581 // Erases the download from downloads_.
589 int DownloadManagerImpl::RemoveDownloads(base::Time remove_begin
) {
590 return RemoveDownloadsBetween(remove_begin
, base::Time());
593 int DownloadManagerImpl::RemoveAllDownloads() {
594 // The null times make the date range unbounded.
595 int num_deleted
= RemoveDownloadsBetween(base::Time(), base::Time());
596 RecordClearAllSize(num_deleted
);
600 void DownloadManagerImpl::DownloadUrl(
601 scoped_ptr
<DownloadUrlParameters
> params
) {
602 if (params
->post_id() >= 0) {
603 // Check this here so that the traceback is more useful.
604 DCHECK(params
->prefer_cache());
605 DCHECK_EQ("POST", params
->method());
607 BrowserThread::PostTask(BrowserThread::IO
, FROM_HERE
, base::Bind(
608 &BeginDownload
, base::Passed(¶ms
),
609 content::DownloadItem::kInvalidId
));
612 void DownloadManagerImpl::AddObserver(Observer
* observer
) {
613 observers_
.AddObserver(observer
);
616 void DownloadManagerImpl::RemoveObserver(Observer
* observer
) {
617 observers_
.RemoveObserver(observer
);
620 DownloadItem
* DownloadManagerImpl::CreateDownloadItem(
622 const base::FilePath
& current_path
,
623 const base::FilePath
& target_path
,
624 const std::vector
<GURL
>& url_chain
,
625 const GURL
& referrer_url
,
626 const std::string
& mime_type
,
627 const std::string
& original_mime_type
,
628 const base::Time
& start_time
,
629 const base::Time
& end_time
,
630 const std::string
& etag
,
631 const std::string
& last_modified
,
632 int64 received_bytes
,
634 DownloadItem::DownloadState state
,
635 DownloadDangerType danger_type
,
636 DownloadInterruptReason interrupt_reason
,
638 if (ContainsKey(downloads_
, id
)) {
642 DownloadItemImpl
* item
= item_factory_
->CreatePersistedItem(
661 net::BoundNetLog::Make(net_log_
, net::NetLog::SOURCE_DOWNLOAD
));
662 downloads_
[id
] = item
;
663 FOR_EACH_OBSERVER(Observer
, observers_
, OnDownloadCreated(this, item
));
664 DVLOG(20) << __FUNCTION__
<< "() download = " << item
->DebugString(true);
668 int DownloadManagerImpl::InProgressCount() const {
670 for (DownloadMap::const_iterator it
= downloads_
.begin();
671 it
!= downloads_
.end(); ++it
) {
672 if (it
->second
->GetState() == DownloadItem::IN_PROGRESS
)
678 int DownloadManagerImpl::NonMaliciousInProgressCount() const {
680 for (DownloadMap::const_iterator it
= downloads_
.begin();
681 it
!= downloads_
.end(); ++it
) {
682 if (it
->second
->GetState() == DownloadItem::IN_PROGRESS
&&
683 it
->second
->GetDangerType() != DOWNLOAD_DANGER_TYPE_DANGEROUS_URL
&&
684 it
->second
->GetDangerType() != DOWNLOAD_DANGER_TYPE_DANGEROUS_CONTENT
&&
685 it
->second
->GetDangerType() != DOWNLOAD_DANGER_TYPE_DANGEROUS_HOST
&&
686 it
->second
->GetDangerType() !=
687 DOWNLOAD_DANGER_TYPE_POTENTIALLY_UNWANTED
) {
694 DownloadItem
* DownloadManagerImpl::GetDownload(uint32 download_id
) {
695 return ContainsKey(downloads_
, download_id
) ? downloads_
[download_id
] : NULL
;
698 void DownloadManagerImpl::GetAllDownloads(DownloadVector
* downloads
) {
699 for (DownloadMap::iterator it
= downloads_
.begin();
700 it
!= downloads_
.end(); ++it
) {
701 downloads
->push_back(it
->second
);
705 void DownloadManagerImpl::OpenDownload(DownloadItemImpl
* download
) {
706 int num_unopened
= 0;
707 for (DownloadMap::iterator it
= downloads_
.begin();
708 it
!= downloads_
.end(); ++it
) {
709 DownloadItemImpl
* item
= it
->second
;
710 if ((item
->GetState() == DownloadItem::COMPLETE
) &&
714 RecordOpensOutstanding(num_unopened
);
717 delegate_
->OpenDownload(download
);
720 void DownloadManagerImpl::ShowDownloadInShell(DownloadItemImpl
* download
) {
722 delegate_
->ShowDownloadInShell(download
);
725 } // namespace content