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

Side by Side Diff: components/offline_pages/background/offliner.h

Issue 1971923004: Add calls to the offliner and unit tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: CR feedback per DougArnett Created 4 years, 7 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
« no previous file with comments | « no previous file | components/offline_pages/background/request_coordinator.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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 #ifndef COMPONENTS_OFFLINE_PAGES_BACKGROUND_OFFLINER_H_ 5 #ifndef COMPONENTS_OFFLINE_PAGES_BACKGROUND_OFFLINER_H_
6 #define COMPONENTS_OFFLINE_PAGES_BACKGROUND_OFFLINER_H_ 6 #define COMPONENTS_OFFLINE_PAGES_BACKGROUND_OFFLINER_H_
7 7
8 #include "base/callback.h" 8 #include "base/callback.h"
9 9
10 namespace offline_pages { 10 namespace offline_pages {
11 11
12 class SavePageRequest; 12 class SavePageRequest;
13 13
14 // Interface of a class responsible for constructing an offline page given 14 // Interface of a class responsible for constructing an offline page given
15 // a request with a URL. 15 // a request with a URL.
16 class Offliner { 16 class Offliner {
17 public: 17 public:
18 // Completion status of processing an offline page request. 18 // Completion status of processing an offline page request.
19 enum CompletionStatus { 19 enum CompletionStatus {
20 SAVED = 0, 20 SAVED = 0,
21 CANCELED = 1, 21 CANCELED = 1,
22 FAILED_CONSIDER_RETRY = 2, 22 FAILED_CONSIDER_RETRY = 2,
23 FAILED_DO_NOT_RETRY = 3, 23 FAILED_DO_NOT_RETRY = 3,
24 UNKNOWN = 4,
24 }; 25 };
25 26
26 // Reports the completion status of a request. 27 // Reports the completion status of a request.
27 // TODO(dougarnett): consider passing back a request id instead of request. 28 // TODO(dougarnett): consider passing back a request id instead of request.
28 typedef base::Callback<void(const SavePageRequest&, CompletionStatus)> 29 typedef base::Callback<void(const SavePageRequest&, CompletionStatus)>
29 CompletionCallback; 30 CompletionCallback;
30 31
31 Offliner() {} 32 Offliner() {}
32 virtual ~Offliner() {} 33 virtual ~Offliner() {}
33 34
34 // Processes |request| to load and save an offline page. 35 // Processes |request| to load and save an offline page.
35 // Returns whether the request was accepted or not. 36 // Returns whether the request was accepted or not.
36 virtual bool LoadAndSave( 37 virtual bool LoadAndSave(
37 const SavePageRequest& request, 38 const SavePageRequest& request,
38 const CompletionCallback& callback) = 0; 39 const CompletionCallback& callback) = 0;
39 40
40 // Clears the currently processing request, if any. 41 // Clears the currently processing request, if any.
41 virtual void Cancel() = 0; 42 virtual void Cancel() = 0;
42 43
43 // TODO(dougarnett): add policy support methods. 44 // TODO(dougarnett): add policy support methods.
44 }; 45 };
45 46
46 } // namespace offline_pages 47 } // namespace offline_pages
47 48
48 #endif // COMPONENTS_OFFLINE_PAGES_BACKGROUND_OFFLINER_H_ 49 #endif // COMPONENTS_OFFLINE_PAGES_BACKGROUND_OFFLINER_H_
OLDNEW
« no previous file with comments | « no previous file | components/offline_pages/background/request_coordinator.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698