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..d4a9d3527a3a16e716d87e4e3b0f7435e1a38540 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,15 @@ class RequestQueueStore {
   virtual void RemoveRequests(const std::vector<int64_t>& request_ids,

[fgorski, 2016/08/02 05:01:22]

thought: should we rename this to  RemoveRequestByRequestId?
WDYT?

[Pete Williamson, 2016/08/02 23:27:17]

I think it is not being used, and I should just remove it in a follow on checkin
(there is already a TODO to do this.).  Since I'm removing it, it doesn't seem
worthwhile to rename it.

[fgorski, 2016/08/03 03:29:41]

Acknowledged.

                               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 be deleted, e.g. because it was missing. In case client_ids is
+  // empty, though this will return true, but the count of deleted pages will be
+  // 0.
+  virtual void RemoveRequestsByClientId(const std::vector<ClientId>& client_ids,
+                                        const RemoveCallback& callback) = 0;
+
   // Resets the store.
   virtual void Reset(const ResetCallback& callback) = 0;
 };