Provide API in RequestCoordinator to remove results by client ID.

components/offline_pages/background/request_queue_store.h

Index: components/offline_pages/background/request_queue_store.h
diff --git a/components/offline_pages/background/request_queue_store.h b/components/offline_pages/background/request_queue_store.h
index 0fd34a32aed34abaa1a57108d63751e5797f384a..4974006583f740d6575ce8ef1629ab974311dd29 100644
--- a/components/offline_pages/background/request_queue_store.h
+++ b/components/offline_pages/background/request_queue_store.h
@@ -9,6 +9,7 @@
 #include <vector>
 
 #include "base/callback.h"
+#include "components/offline_pages/offline_page_item.h"
 
 namespace offline_pages {
 
@@ -51,6 +52,14 @@ class RequestQueueStore {
   virtual void RemoveRequests(const std::vector<int64_t>& request_ids,
                               const RemoveCallback& callback) = 0;
 
+  // Asynchronously removes requests from the store using their ClientIds.
+  // Result of the update, and a number of removed pages is passed in the
+  // callback.
+  // Result of remove should be false, when one of the provided items couldn't

[Dmitry Titov, 2016/08/01 19:37:31]

This is better reconciled with the comment from .cc file that says if the ids
are empty, it returns 'true' (since this is ambiguous w/o explicit mention).

[Pete Williamson, 2016/08/01 23:23:08]

Done.

+  // be deleted, e.g. because it was missing.
+  virtual void RemoveRequestsByClientId(const std::vector<ClientId>& client_ids,
+                                        const RemoveCallback& callback) = 0;
+
   // Resets the store.
   virtual void Reset(const ResetCallback& callback) = 0;
 };