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

Unified Diff: content/browser/background_fetch/background_fetch_job_data.h

Issue 2767373002: Implement GetJobResponse and merge JobData into DataManager. (Closed)
Patch Set: Removed typedef and added DISALLOW_COPY_AND_ASSIGN Created 3 years, 9 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_job_data.h
diff --git a/content/browser/background_fetch/background_fetch_job_data.h b/content/browser/background_fetch/background_fetch_job_data.h
deleted file mode 100644
index 13f76c774d7fbf0e922631de085dd65bee562e47..0000000000000000000000000000000000000000
--- a/content/browser/background_fetch/background_fetch_job_data.h
+++ /dev/null
@@ -1,64 +0,0 @@
-// Copyright 2017 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.
-
-#ifndef CONTENT_BROWSER_BACKGROUND_FETCH_BACKGROUND_FETCH_JOB_DATA_H_
-#define CONTENT_BROWSER_BACKGROUND_FETCH_BACKGROUND_FETCH_JOB_DATA_H_
-
-#include <string>
-#include <unordered_map>
-
-#include "base/macros.h"
-#include "content/browser/background_fetch/background_fetch_request_info.h"
-#include "content/common/content_export.h"
-#include "content/public/browser/download_interrupt_reasons.h"
-#include "content/public/browser/download_item.h"
-#include "url/origin.h"
-
-namespace content {
-
-// The BackgroundFetchJobData class provides the interface from the
-// JobController to the data storage in the DataManager. It has a reference
-// to the DataManager and invokes calls given the stored batch_guid.
-class CONTENT_EXPORT BackgroundFetchJobData {
- public:
- explicit BackgroundFetchJobData(BackgroundFetchRequestInfos& request_infos);
- ~BackgroundFetchJobData();
-
- // Called by the JobController to inform the JobData that the given fetch
- // has been updated. The JobData returns a boolean indicating whether there
- // are more requests to process.
- bool UpdateBackgroundFetchRequestState(
- const std::string& fetch_guid,
- DownloadItem::DownloadState state,
- DownloadInterruptReason interrupt_reason);
-
- // Called by the JobController to get a BackgroundFetchRequestInfo to
- // process.
- const BackgroundFetchRequestInfo& GetNextBackgroundFetchRequestInfo();
-
- // Indicates whether all requests have been handed to the JobController.
- bool HasRequestsRemaining() const;
-
- // Indicates whether all requests have been handed out and completed.
- bool IsComplete() const;
-
- // Inform the JobData of the GUID the DownloadManager assigned to this
- // request.
- void SetRequestDownloadGuid(const std::string& request_guid,
- const std::string& download_guid);
-
- private:
- BackgroundFetchRequestInfos& request_infos_;
-
- // Map from fetch_guid to index in request_infos_. Only currently
- // outstanding requests should be in this map.
- std::unordered_map<std::string, int> request_info_index_;
- size_t next_request_info_ = 0;
-
- DISALLOW_COPY_AND_ASSIGN(BackgroundFetchJobData);
-};
-
-} // namespace content
-
-#endif // CONTENT_BROWSER_BACKGROUND_FETCH_BACKGROUND_FETCH_JOB_DATA_H_

Powered by Google App Engine
This is Rietveld 408576698