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

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

Issue 2566453002: [OfflinePages] Rename PRERENDERING_* result terminology to LOADING_* (Closed)
Patch Set: Updated histogram enum Created 4 years 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
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_CORE_BACKGROUND_OFFLINER_H_ 5 #ifndef COMPONENTS_OFFLINE_PAGES_CORE_BACKGROUND_OFFLINER_H_
6 #define COMPONENTS_OFFLINE_PAGES_CORE_BACKGROUND_OFFLINER_H_ 6 #define COMPONENTS_OFFLINE_PAGES_CORE_BACKGROUND_OFFLINER_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/callback.h" 10 #include "base/callback.h"
(...skipping 12 matching lines...) Expand all
23 // Also update related switch code in RequestCoordinatorEventLogger. 23 // Also update related switch code in RequestCoordinatorEventLogger.
24 enum RequestStatus { 24 enum RequestStatus {
25 // No status determined/reported yet. Interim status, not sent in callback. 25 // No status determined/reported yet. Interim status, not sent in callback.
26 UNKNOWN = 0, 26 UNKNOWN = 0,
27 // Page loaded but not (yet) saved. Interim status, not sent in callback. 27 // Page loaded but not (yet) saved. Interim status, not sent in callback.
28 LOADED = 1, 28 LOADED = 1,
29 // Offline page snapshot saved. 29 // Offline page snapshot saved.
30 SAVED = 2, 30 SAVED = 2,
31 // RequestCoordinator canceled request. 31 // RequestCoordinator canceled request.
32 REQUEST_COORDINATOR_CANCELED = 3, 32 REQUEST_COORDINATOR_CANCELED = 3,
33 // Prerendering was canceled. 33 // Loading was canceled.
34 PRERENDERING_CANCELED = 4, 34 LOADING_CANCELED = 4,
35 // Prerendering failed to load page. 35 // Loader failed to load page.
36 PRERENDERING_FAILED = 5, 36 LOADING_FAILED = 5,
37 // Failed to save loaded page. 37 // Failed to save loaded page.
38 SAVE_FAILED = 6, 38 SAVE_FAILED = 6,
39 // Foreground transition canceled request. 39 // Foreground transition canceled request.
40 FOREGROUND_CANCELED = 7, 40 FOREGROUND_CANCELED = 7,
41 // RequestCoordinator canceled request attempt per time limit. 41 // RequestCoordinator canceled request attempt per time limit.
42 REQUEST_COORDINATOR_TIMED_OUT = 8, 42 REQUEST_COORDINATOR_TIMED_OUT = 8,
43 // The loader did not accept/start the request. 43 // The loader did not accept/start the request.
44 PRERENDERING_NOT_STARTED = 9, 44 LOADING_NOT_STARTED = 9,
45 // Prerendering failed with hard error so should not retry the request. 45 // Loader failed with hard error so should not retry the request.
46 PRERENDERING_FAILED_NO_RETRY = 10, 46 LOADING_FAILED_NO_RETRY = 10,
47 // Prerendering failed with some error that suggests we should not continue 47 // Loader failed with some error that suggests we should not continue
48 // processing another request at this time. 48 // processing another request at this time.
49 PRERENDERING_FAILED_NO_NEXT = 11, 49 LOADING_FAILED_NO_NEXT = 11,
50 // NOTE: insert new values above this line and update histogram enum too. 50 // NOTE: insert new values above this line and update histogram enum too.
51 STATUS_COUNT 51 STATUS_COUNT
52 }; 52 };
53 53
54 // Reports the completion status of a request. 54 // Reports the completion status of a request.
55 // TODO(dougarnett): consider passing back a request id instead of request. 55 // TODO(dougarnett): consider passing back a request id instead of request.
56 typedef base::Callback<void(const SavePageRequest&, RequestStatus)> 56 typedef base::Callback<void(const SavePageRequest&, RequestStatus)>
57 CompletionCallback; 57 CompletionCallback;
58 58
59 Offliner() {} 59 Offliner() {}
60 virtual ~Offliner() {} 60 virtual ~Offliner() {}
61 61
62 // Processes |request| to load and save an offline page. 62 // Processes |request| to load and save an offline page.
63 // Returns whether the request was accepted or not. |callback| is guaranteed 63 // Returns whether the request was accepted or not. |callback| is guaranteed
64 // to be called if the request was accepted and |Cancel()| is not called. 64 // to be called if the request was accepted and |Cancel()| is not called.
65 virtual bool LoadAndSave(const SavePageRequest& request, 65 virtual bool LoadAndSave(const SavePageRequest& request,
66 const CompletionCallback& callback) = 0; 66 const CompletionCallback& callback) = 0;
67 67
68 // Clears the currently processing request, if any, and skips running its 68 // Clears the currently processing request, if any, and skips running its
69 // CompletionCallback. 69 // CompletionCallback.
70 virtual void Cancel() = 0; 70 virtual void Cancel() = 0;
71 71
72 // TODO(dougarnett): add policy support methods. 72 // TODO(dougarnett): add policy support methods.
73 }; 73 };
74 74
75 } // namespace offline_pages 75 } // namespace offline_pages
76 76
77 #endif // COMPONENTS_OFFLINE_PAGES_CORE_BACKGROUND_OFFLINER_H_ 77 #endif // COMPONENTS_OFFLINE_PAGES_CORE_BACKGROUND_OFFLINER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698