[Offline Pages] Converts MarkAttemptCompleted to use TaskQueue

components/offline_pages/background/mark_attempt_completed_task.cc

 // 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/mark_attempt_started_task.h"
+#include "components/offline_pages/background/mark_attempt_completed_task.h"
+
+#include <utility>

[Pete Williamson, 2016/10/31 21:38:26]

I wonder if <utility> should be added to mark_attempt_started_task.h too.

[dougarnett, 2016/10/31 22:51:17]

Done.

 
 #include "base/bind.h"
 #include "base/time/time.h"
 
 namespace offline_pages {
 
-MarkAttemptStartedTask::MarkAttemptStartedTask(
+MarkAttemptCompletedTask::MarkAttemptCompletedTask(
     RequestQueueStore* store,
     int64_t request_id,
     const RequestQueueStore::UpdateCallback& callback)
     : UpdateRequestTask(store, request_id, callback) {}
 
-MarkAttemptStartedTask::~MarkAttemptStartedTask() {}
+MarkAttemptCompletedTask::~MarkAttemptCompletedTask() {}
 
-void MarkAttemptStartedTask::UpdateRequestImpl(
+void MarkAttemptCompletedTask::UpdateRequestImpl(
     std::unique_ptr<UpdateRequestsResult> read_result) {
   if (!ValidateReadResult(read_result.get())) {
     CompleteWithResult(std::move(read_result));
     return;
   }
 
   // It is perfectly fine to reuse the read_result->updated_items collection, as
   // it is owned by this callback and will be destroyed when out of scope.
-  read_result->updated_items[0].MarkAttemptStarted(base::Time::Now());
+  read_result->updated_items[0].MarkAttemptCompleted();
   store()->UpdateRequests(
       read_result->updated_items,
-      base::Bind(&MarkAttemptStartedTask::CompleteWithResult, GetWeakPtr()));
+      base::Bind(&MarkAttemptCompletedTask::CompleteWithResult, GetWeakPtr()));
 }
 
 }  // namespace offline_pages