Provide API in RequestCoordinator to remove results by client ID.

components/offline_pages/background/request_queue_in_memory_store.h

Index: components/offline_pages/background/request_queue_in_memory_store.h
diff --git a/components/offline_pages/background/request_queue_in_memory_store.h b/components/offline_pages/background/request_queue_in_memory_store.h
index 0a9b8bdb323337a9ddcfce877689717d01db4587..e5669a69858a12cf0470be2c53dced15f389766f 100644
--- a/components/offline_pages/background/request_queue_in_memory_store.h
+++ b/components/offline_pages/background/request_queue_in_memory_store.h
@@ -28,6 +28,8 @@ class RequestQueueInMemoryStore : public RequestQueueStore {
                           const UpdateCallback& callback) override;
   void RemoveRequests(const std::vector<int64_t>& request_ids,

[chili, 2016/08/01 19:34:40]

just out of curiosity - are there plans to expose 'remove requests by request
id' as well?

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

It is already exposed, RemoveReqeusts removes them by request id.
However, I am considering removing it in a future change, since we don't
actually use it.  It turns out that its utility is limited by the fact that the
request coordinator doesn't keep the request ids around, and its clients don't
use them either.

                       const RemoveCallback& callback) override;
+  void RemoveRequestsByClientId(const std::vector<ClientId>& client_ids,
+                                const RemoveCallback& callback) override;
   void Reset(const ResetCallback& callback) override;
 
  private: