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

Unified Diff: content/browser/background_fetch/background_fetch_request_info.cc

Issue 2796953003: Populate the response blob for finished Background Fetches (Closed)
Patch Set: Populate the response blob for finished Background Fetches Created 3 years, 8 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/background_fetch/background_fetch_request_info.cc
diff --git a/content/browser/background_fetch/background_fetch_request_info.cc b/content/browser/background_fetch/background_fetch_request_info.cc
index f2deb2375d92fc20dbd840fb69538857b08c049d..0047606157d8318f4f9ca80ec5645a235d9576a3 100644
--- a/content/browser/background_fetch/background_fetch_request_info.cc
+++ b/content/browser/background_fetch/background_fetch_request_info.cc
@@ -17,13 +17,47 @@ BackgroundFetchRequestInfo::BackgroundFetchRequestInfo(
BackgroundFetchRequestInfo::~BackgroundFetchRequestInfo() {}
-bool BackgroundFetchRequestInfo::IsComplete() const {
- return (state_ == DownloadItem::DownloadState::COMPLETE ||
- state_ == DownloadItem::DownloadState::CANCELLED);
+void BackgroundFetchRequestInfo::PopulateDownloadState(
+ DownloadItem* download_item,
+ DownloadInterruptReason download_interrupt_reason) {
+ DCHECK(!download_state_populated_);
+
+ download_guid_ = download_item->GetGuid();
+ download_state_ = download_item->GetState();
+
+ download_state_populated_ = true;
+}
+
+void BackgroundFetchRequestInfo::PopulateResponseFromDownloadItem(
+ DownloadItem* download_item) {
+ DCHECK(!response_data_populated_);
+
+ url_chain_ = download_item->GetUrlChain();
+ file_path_ = download_item->GetTargetFilePath();
+ file_size_ = download_item->GetReceivedBytes();
+ response_time_ = download_item->GetEndTime();
+
+ response_data_populated_ = true;
+}
+
+const std::vector<GURL>& BackgroundFetchRequestInfo::GetURLChain() const {
+ DCHECK(response_data_populated_);
+ return url_chain_;
+}
+
+const base::FilePath& BackgroundFetchRequestInfo::GetFilePath() const {
+ DCHECK(response_data_populated_);
+ return file_path_;
+}
+
+int64_t BackgroundFetchRequestInfo::GetFileSize() const {
+ DCHECK(response_data_populated_);
+ return file_size_;
}
-const GURL& BackgroundFetchRequestInfo::GetURL() const {
- return fetch_request_.url;
+const base::Time& BackgroundFetchRequestInfo::GetResponseTime() const {
+ DCHECK(response_data_populated_);
+ return response_time_;
}
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698