mirrors/electron
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

only emit download-item done event once.

Browse source
This commit is contained in:
liusi 2017-02-14 14:39:33 +08:00
parent d3c967401d
commit d56c1909df
2 changed files with 7 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
atom/browser/api/atom_api_download_item.cc
View file

@ -58,7 +58,7 @@ std::map<uint32_t, v8::Global<v8::Object>> g_download_item_objects;
DownloadItem::DownloadItem(v8::Isolate* isolate, DownloadItem::DownloadItem(v8::Isolate* isolate,
content::DownloadItem* download_item) content::DownloadItem* download_item)
: download_item_(download_item) { : download_item_(download_item), done_emitted_(false) {
download_item_->AddObserver(this); download_item_->AddObserver(this);
Init(isolate); Init(isolate);
AttachAsUserData(download_item); AttachAsUserData(download_item);
@ -76,14 +76,16 @@ DownloadItem::~DownloadItem() {
} }
void DownloadItem::OnDownloadUpdated(content::DownloadItem* item) { void DownloadItem::OnDownloadUpdated(content::DownloadItem* item) {
if (download_item_->IsDone()) { if (!download_item_->IsDone()) {
Emit("updated", item->GetState());
} else if (!done_emitted_) {
Emit("done", item->GetState()); Emit("done", item->GetState());
done_emitted_ = true;
// Destroy the item once item is downloaded. // Destroy the item once item is downloaded.
base::ThreadTaskRunnerHandle::Get()->PostTask( base::ThreadTaskRunnerHandle::Get()->PostTask(
FROM_HERE, GetDestroyClosure()); FROM_HERE, GetDestroyClosure());
} else {
Emit("updated", item->GetState());
} }
} }

1
atom/browser/api/atom_api_download_item.h
View file

@ -59,6 +59,7 @@ class DownloadItem : public mate::TrackableObject<DownloadItem>,
private: private:
base::FilePath save_path_; base::FilePath save_path_;
content::DownloadItem* download_item_; content::DownloadItem* download_item_;
bool done_emitted_;
DISALLOW_COPY_AND_ASSIGN(DownloadItem); DISALLOW_COPY_AND_ASSIGN(DownloadItem);
}; };