Adds request queue removal capability to OfflinePageBridge.

components/offline_pages/background/request_coordinator_unittest.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/request_coordinator.h"
 
 #include <memory>
 #include <utility>
 #include <vector>
 
@@ skipping 196 matching lines @@
   }
 
   // Callback for Add requests.
   void AddRequestDone(RequestQueue::AddRequestResult result,
                       const SavePageRequest& request);
 
   // Callback for getting requests.
   void GetRequestsDone(RequestQueue::GetRequestsResult result,
                        const std::vector<SavePageRequest>& requests);
 
+  // Callback for removing requests.
+  void RemoveRequestsDone(const std::vector<SavePageRequest>& requests);
+
   // Callback for getting request statuses.
   void GetQueuedRequestsDone(const std::vector<SavePageRequest>& requests);
 
   void SendOfflinerDoneCallback(const SavePageRequest& request,
                                 Offliner::RequestStatus status);
 
   RequestQueue::GetRequestsResult last_get_requests_result() const {
     return last_get_requests_result_;
   }
 
@@ skipping 70 matching lines @@
   task_runner_->RunUntilIdle();
 }
 
 void RequestCoordinatorTest::GetRequestsDone(
     RequestQueue::GetRequestsResult result,
     const std::vector<SavePageRequest>& requests) {
   last_get_requests_result_ = result;
   last_requests_ = requests;
 }
 
+void RequestCoordinatorTest::RemoveRequestsDone(
+    const std::vector<SavePageRequest>& requests) {
+  last_requests_ = requests;
+  waiter_.Signal();
+}
+
 void RequestCoordinatorTest::GetQueuedRequestsDone(
     const std::vector<SavePageRequest>& requests) {
   last_requests_ = requests;
   waiter_.Signal();
 }
 
 void RequestCoordinatorTest::AddRequestDone(
     RequestQueue::AddRequestResult result,
     const SavePageRequest& request) {}
 
@@ skipping 431 matching lines @@
   observer().Clear();
 
   // Resume the request.
   coordinator()->ResumeRequests(request_ids);
   PumpLoop();
 
   EXPECT_TRUE(observer().changed_called());
   EXPECT_EQ(SavePageRequest::RequestState::AVAILABLE, observer().state());
 }
 
-TEST_F(RequestCoordinatorTest, ObserverdRemoveRequest) {
+TEST_F(RequestCoordinatorTest, RemoveRequest) {
   // Add a request to the queue.
   offline_pages::SavePageRequest request1(kRequestId1, kUrl1, kClientId1,
                                           base::Time::Now(), kUserRequested);
   coordinator()->queue()->AddRequest(
       request1, base::Bind(&RequestCoordinatorTest::AddRequestDone,
                            base::Unretained(this)));
   PumpLoop();
 
   // Remove the request.
   std::vector<int64_t> request_ids;
   request_ids.push_back(kRequestId1);
-  coordinator()->RemoveRequests(request_ids);
+  coordinator()->RemoveRequests(
+      request_ids, base::Bind(&RequestCoordinatorTest::RemoveRequestsDone,
+                              base::Unretained(this)));
+
+  PumpLoop();
+  WaitForCallback();
   PumpLoop();
 
   EXPECT_TRUE(observer().completed_called());
   EXPECT_EQ(RequestCoordinator::SavePageStatus::REMOVED,
             observer().last_status());
+  EXPECT_EQ(1UL, last_requests().size());
+  EXPECT_EQ(kRequestId1, last_requests().at(0).request_id());
 }
 
 }  // namespace offline_pages