[Offline Pages] Converts MarkAttemptCompleted to use TaskQueue

components/offline_pages/background/request_coordinator.h

Index: components/offline_pages/background/request_coordinator.h
diff --git a/components/offline_pages/background/request_coordinator.h b/components/offline_pages/background/request_coordinator.h
index 263128f7edded1671f78c11e0b2cd76e7eb5b9f4..a39411d29826b9b41cd7decd42b74a46df18f5b7 100644
--- a/components/offline_pages/background/request_coordinator.h
+++ b/components/offline_pages/background/request_coordinator.h
@@ -7,6 +7,7 @@
 
 #include <memory>
 #include <set>
+#include <string>
 #include <vector>
 
 #include "base/callback.h"
@@ -241,9 +242,11 @@ class RequestCoordinator : public KeyedService,
   void AddRequestResultCallback(RequestQueue::AddRequestResult result,
                                 const SavePageRequest& request);
 
-  // Receives the result of update and delete requests to the request queue.
-  void UpdateRequestCallback(const ClientId& client_id,
-                             RequestQueue::UpdateRequestResult result);
+  // Receives the result of mark attempt completed requests.
+  void MarkAttemptCompletedDoneCallback(

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

I thought this callback was used for lots of things, this name change might not
be a good idea.

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

No, this was last usage.

+      int64_t request_id,
+      const ClientId& client_id,
+      std::unique_ptr<UpdateRequestsResult> result);
 
   void UpdateMultipleRequestsCallback(
       std::unique_ptr<UpdateRequestsResult> result);