[Offline pages] Add MarkAttemptAbortedTask to Request Queue

components/offline_pages/background/update_request_task.h

Index: components/offline_pages/background/update_request_task.h
diff --git a/components/offline_pages/background/update_request_task.h b/components/offline_pages/background/update_request_task.h
new file mode 100644
index 0000000000000000000000000000000000000000..a4ecee885e9d56f12365d6af7277f72e03500abb
--- /dev/null
+++ b/components/offline_pages/background/update_request_task.h
@@ -0,0 +1,63 @@
+// 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.
+
+#ifndef COMPONENTS_OFFLINE_PAGES_BACKGROUND_UPDATE_REQUEST_TASK_H_
+#define COMPONENTS_OFFLINE_PAGES_BACKGROUND_UPDATE_REQUEST_TASK_H_
+
+#include <stdint.h>
+
+#include <memory>
+
+#include "base/memory/weak_ptr.h"
+#include "components/offline_pages/background/request_queue_store.h"
+#include "components/offline_pages/core/task.h"
+
+namespace offline_pages {
+
+class UpdateRequestTask : public Task {

[dougarnett, 2016/10/19 21:23:10]

doc? base class for updating a request?

[fgorski, 2016/10/20 16:56:34]

Done.

+ public:
+  UpdateRequestTask(RequestQueueStore* store,
+                    int64_t request_id,
+                    const RequestQueueStore::UpdateCallback& callback);
+  ~UpdateRequestTask() override;
+
+  // TaskQueue::Task implementation.
+  void Run() override;
+
+ protected:
+  // Step 1. Reading the requests.
+  void ReadRequest();
+  // Step 2. Work is done in the implementation step.
+  virtual void UpdateRequestImpl(
+      std::unique_ptr<UpdateRequestsResult> result) = 0;
+  // Step 3. Completes once update is done.
+  void CompleteWithResult(std::unique_ptr<UpdateRequestsResult> result);
+
+  // Function to uniformly validate read request call for store errors and
+  // presence of the request.
+  bool ValidateReadResult(UpdateRequestsResult* result);
+
+  RequestQueueStore* store() const { return store_; }
+
+  int64_t request_id() const { return request_id_; }
+
+  base::WeakPtr<UpdateRequestTask> GetWeakPtr() {
+    return weak_ptr_factory_.GetWeakPtr();
+  }
+
+ private:
+  // Store that this task updates. Not owned.
+  RequestQueueStore* store_;
+  // Request ID of the request to be started.
+  int64_t request_id_;
+  // Callback to complete the task.
+  RequestQueueStore::UpdateCallback callback_;
+
+  base::WeakPtrFactory<UpdateRequestTask> weak_ptr_factory_;
+  DISALLOW_COPY_AND_ASSIGN(UpdateRequestTask);
+};
+
+}  // namespace offline_pages
+
+#endif  // COMPONENTS_OFFLINE_PAGES_BACKGROUND_UPDATE_REQUEST_TASK_H_