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

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

Issue 2372043002: [Offline pages] Applying TaskQueue to RequestQueue::ChangeRequestsState (Closed)
Patch Set: Addressing CR feedback and rebasing Created 4 years, 2 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
OLDNEW
(Empty)
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
3 // found in the LICENSE file.
4
5 #include "components/offline_pages/background/change_requests_state_task.h"
6
7 #include "base/bind.h"
8
9 namespace offline_pages {
10
11 ChangeRequestsStateTask::ChangeRequestsStateTask(
12 RequestQueueStore* store,
13 const std::vector<int64_t>& request_ids,
14 const SavePageRequest::RequestState new_state,
15 const RequestQueueStore::UpdateCallback& callback)
16 : store_(store),
17 request_ids_(request_ids.begin(), request_ids.end()),
18 new_state_(new_state),
19 callback_(callback),
20 weak_ptr_factory_(this) {}
21
22 ChangeRequestsStateTask::~ChangeRequestsStateTask() {}
23
24 void ChangeRequestsStateTask::Run() {
25 if (request_ids_.empty()) {
26 CompleteEarly(ItemActionStatus::NOT_FOUND);
27 return;
28 }
29
30 store_->GetRequests(base::Bind(&ChangeRequestsStateTask::SelectItemsToUpdate,
31 weak_ptr_factory_.GetWeakPtr()));
32 }
33
34 void ChangeRequestsStateTask::SelectItemsToUpdate(
35 bool success,
36 std::vector<std::unique_ptr<SavePageRequest>> requests) {
37 if (!success) {
38 CompleteEarly(ItemActionStatus::STORE_ERROR);
39 return;
40 }
41
42 std::vector<SavePageRequest> items_to_update;
43 for (const auto& request : requests) {
44 // If this request is in our list, update it.
45 if (request_ids_.count(request->request_id()) > 0) {
46 request->set_request_state(new_state_);
47 items_to_update.push_back(*request);
48 // Items that are missing before the update will be marked as not found
49 // before the callback.
50 request_ids_.erase(request->request_id());
51 }
52 }
53
54 if (items_to_update.empty()) {
55 CompleteEarly(ItemActionStatus::NOT_FOUND);
56 return;
57 }
58
59 store_->UpdateRequests(items_to_update,
60 base::Bind(&ChangeRequestsStateTask::UpdateCompleted,
61 weak_ptr_factory_.GetWeakPtr()));
62 }
63
64 void ChangeRequestsStateTask::UpdateCompleted(
65 std::unique_ptr<UpdateRequestsResult> update_result) {
66 CompleteWithStatus(std::move(update_result), ItemActionStatus::NOT_FOUND);
67 }
68
69 void ChangeRequestsStateTask::CompleteEarly(ItemActionStatus status) {
70 // TODO(fgorski): store_->state() once implemented
71 std::unique_ptr<UpdateRequestsResult> result(
72 new UpdateRequestsResult(StoreState::LOADED));
73 CompleteWithStatus(std::move(result), status);
74 }
75
76 void ChangeRequestsStateTask::CompleteWithStatus(
77 std::unique_ptr<UpdateRequestsResult> result,
78 ItemActionStatus status) {
79 // Mark items as not found, if they are still in the request IDs set.
80 for (int64_t request_id : request_ids_)
81 result->item_statuses.push_back(std::make_pair(request_id, status));
82 callback_.Run(std::move(result));
83 Complete();
84 }
85
86 } // namespace offline_pages
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698