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

Side by Side Diff: components/offline_pages/background/request_queue.cc

Issue 2473553004: Request Picker task (Closed)
Patch Set: merge with latest Created 4 years, 1 month 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 #include "components/offline_pages/background/request_queue.h" 5 #include "components/offline_pages/background/request_queue.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/location.h" 10 #include "base/location.h"
11 #include "base/threading/thread_task_runner_handle.h" 11 #include "base/threading/thread_task_runner_handle.h"
12 #include "components/offline_pages/background/change_requests_state_task.h" 12 #include "components/offline_pages/background/change_requests_state_task.h"
13 #include "components/offline_pages/background/mark_attempt_aborted_task.h" 13 #include "components/offline_pages/background/mark_attempt_aborted_task.h"
14 #include "components/offline_pages/background/mark_attempt_completed_task.h" 14 #include "components/offline_pages/background/mark_attempt_completed_task.h"
15 #include "components/offline_pages/background/mark_attempt_started_task.h" 15 #include "components/offline_pages/background/mark_attempt_started_task.h"
16 #include "components/offline_pages/background/pick_request_task.h"
17 #include "components/offline_pages/background/pick_request_task_factory.h"
16 #include "components/offline_pages/background/remove_requests_task.h" 18 #include "components/offline_pages/background/remove_requests_task.h"
17 #include "components/offline_pages/background/request_queue_store.h" 19 #include "components/offline_pages/background/request_queue_store.h"
18 #include "components/offline_pages/background/save_page_request.h" 20 #include "components/offline_pages/background/save_page_request.h"
19 21
20 namespace offline_pages { 22 namespace offline_pages {
21 23
22 namespace { 24 namespace {
23 // Completes the get requests call. 25 // Completes the get requests call.
24 void GetRequestsDone(const RequestQueue::GetRequestsCallback& callback, 26 void GetRequestsDone(const RequestQueue::GetRequestsCallback& callback,
25 bool success, 27 bool success,
26 std::vector<std::unique_ptr<SavePageRequest>> requests) { 28 std::vector<std::unique_ptr<SavePageRequest>> requests) {
27 RequestQueue::GetRequestsResult result = 29 GetRequestsResult result =
28 success ? RequestQueue::GetRequestsResult::SUCCESS 30 success ? GetRequestsResult::SUCCESS : GetRequestsResult::STORE_FAILURE;
29 : RequestQueue::GetRequestsResult::STORE_FAILURE;
30 // TODO(fgorski): Filter out expired requests based on policy. 31 // TODO(fgorski): Filter out expired requests based on policy.
31 // This may trigger the purging if necessary. 32 // This may trigger the purging if necessary.
32 // Also this may be turned into a method on the request queue or add a policy 33 // Also this may be turned into a method on the request queue or add a policy
33 // parameter in the process. 34 // parameter in the process.
34 callback.Run(result, std::move(requests)); 35 callback.Run(result, std::move(requests));
35 } 36 }
36 37
37 // Completes the add request call. 38 // Completes the add request call.
38 void AddRequestDone(const RequestQueue::AddRequestCallback& callback, 39 void AddRequestDone(const RequestQueue::AddRequestCallback& callback,
39 const SavePageRequest& request, 40 const SavePageRequest& request,
40 ItemActionStatus status) { 41 ItemActionStatus status) {
41 RequestQueue::AddRequestResult result; 42 AddRequestResult result;
42 switch (status) { 43 switch (status) {
43 case ItemActionStatus::SUCCESS: 44 case ItemActionStatus::SUCCESS:
44 result = RequestQueue::AddRequestResult::SUCCESS; 45 result = AddRequestResult::SUCCESS;
45 break; 46 break;
46 case ItemActionStatus::ALREADY_EXISTS: 47 case ItemActionStatus::ALREADY_EXISTS:
47 result = RequestQueue::AddRequestResult::ALREADY_EXISTS; 48 result = AddRequestResult::ALREADY_EXISTS;
48 break; 49 break;
49 case ItemActionStatus::STORE_ERROR: 50 case ItemActionStatus::STORE_ERROR:
50 result = RequestQueue::AddRequestResult::STORE_FAILURE; 51 result = AddRequestResult::STORE_FAILURE;
51 break; 52 break;
52 case ItemActionStatus::NOT_FOUND: 53 case ItemActionStatus::NOT_FOUND:
53 default: 54 default:
54 NOTREACHED(); 55 NOTREACHED();
55 return; 56 return;
56 } 57 }
57 callback.Run(result, request); 58 callback.Run(result, request);
58 } 59 }
59 60
60 } // namespace 61 } // namespace
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
107 108
108 void RequestQueue::MarkAttemptCompleted(int64_t request_id, 109 void RequestQueue::MarkAttemptCompleted(int64_t request_id,
109 const UpdateCallback& callback) { 110 const UpdateCallback& callback) {
110 std::unique_ptr<Task> task( 111 std::unique_ptr<Task> task(
111 new MarkAttemptCompletedTask(store_.get(), request_id, callback)); 112 new MarkAttemptCompletedTask(store_.get(), request_id, callback));
112 task_queue_.AddTask(std::move(task)); 113 task_queue_.AddTask(std::move(task));
113 } 114 }
114 115
115 void RequestQueue::PurgeRequests(const PurgeRequestsCallback& callback) {} 116 void RequestQueue::PurgeRequests(const PurgeRequestsCallback& callback) {}
116 117
118 void RequestQueue::PickNextRequest(
119 PickRequestTask::RequestPickedCallback picked_callback,
120 PickRequestTask::RequestNotPickedCallback not_picked_callback,
121 DeviceConditions& conditions,
122 std::set<int64_t>& disabled_requests) {
123 // Using the PickerContext, create a picker task.
124 std::unique_ptr<Task> task(picker_factory_->CreatePickerTask(
125 store_.get(), picked_callback, not_picked_callback, conditions,
126 disabled_requests));
127
128 // Queue up the picking task, it will call one of the callbacks when it
129 // completes.
130 task_queue_.AddTask(std::move(task));
131 }
132
117 } // namespace offline_pages 133 } // namespace offline_pages
OLDNEW
« no previous file with comments | « components/offline_pages/background/request_queue.h ('k') | components/offline_pages/background/request_queue_results.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698