[Offline pages] Updating RequestQueue::RemoveRequests to use a TaskQueue

components/offline_pages/background/remove_requests_task.cc

Index: components/offline_pages/background/remove_requests_task.cc
diff --git a/components/offline_pages/background/remove_requests_task.cc b/components/offline_pages/background/remove_requests_task.cc
new file mode 100644
index 0000000000000000000000000000000000000000..35c31c7164591a7a96ebeea01a64fa948b1931ca
--- /dev/null
+++ b/components/offline_pages/background/remove_requests_task.cc
@@ -0,0 +1,48 @@
+// Copyright 2016 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.
+
+#include "components/offline_pages/background/remove_requests_task.h"
+
+#include "base/bind.h"
+
+namespace offline_pages {
+
+RemoveRequestsTask::RemoveRequestsTask(
+    RequestQueueStore* store,
+    const std::vector<int64_t>& request_ids,
+    const RequestQueueStore::UpdateCallback& callback)

[Pete Williamson, 2016/09/30 18:03:53]

Let's apply the renames from the other (TaskQueue and Task) change to this
changelist too.

[fgorski, 2016/10/04 17:18:23]

Acknowledged.

+    : store_(store),
+      request_ids_(request_ids),
+      callback_(callback),
+      weak_ptr_factory_(this) {}
+
+RemoveRequestsTask::~RemoveRequestsTask() {}
+
+void RemoveRequestsTask::Run() {
+  if (request_ids_.empty()) {
+    CompleteEarly(ItemActionStatus::NOT_FOUND);
+    return;
+  }
+
+  store_->RemoveRequests(request_ids_,
+                         base::Bind(&RemoveRequestsTask::CompleteWithResult,
+                                    weak_ptr_factory_.GetWeakPtr()));
+}
+
+void RemoveRequestsTask::CompleteEarly(ItemActionStatus status) {
+  // TODO(fgorski): store_->state() once implemented
+  std::unique_ptr<UpdateRequestsResult> result(
+      new UpdateRequestsResult(StoreState::LOADED));
+  for (int64_t request_id : request_ids_)
+    result->item_statuses.push_back(std::make_pair(request_id, status));
+  CompleteWithResult(std::move(result));
+}
+
+void RemoveRequestsTask::CompleteWithResult(
+    std::unique_ptr<UpdateRequestsResult> result) {
+  callback_.Run(std::move(result));
+  Complete();
+}
+
+}  // namespace offline_pages