[Offline pages] Applying TaskQueue to RequestQueue::ChangeRequestsState

components/offline_pages/background/request_queue_in_memory_store.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_queue_in_memory_store.h"
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "components/offline_pages/background/save_page_request.h"
@@ skipping 27 matching lines @@
   } else {
     status = ItemActionStatus::ALREADY_EXISTS;
   }
 
   base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE,
                                                 base::Bind(callback, status));
 }
 
 void RequestQueueInMemoryStore::UpdateRequests(
     const std::vector<SavePageRequest>& requests,
-    const UpdateCallback& callback) {
+    const RequestQueue::UpdateCallback& callback) {
   std::unique_ptr<UpdateRequestsResult> result(
       new UpdateRequestsResult(StoreState::LOADED));
 
   ItemActionStatus status;
   for (const auto& request : requests) {
     RequestsMap::iterator iter = requests_.find(request.request_id());
     if (iter != requests_.end()) {
       status = ItemActionStatus::SUCCESS;
       iter->second = request;
       result->updated_items.push_back(request);
@@ skipping 30 matching lines @@
       result = RequestQueue::UpdateRequestResult::REQUEST_DOES_NOT_EXIST;
     }
     results.push_back(std::make_pair(request_id, result));
   }
 
   base::ThreadTaskRunnerHandle::Get()->PostTask(
       FROM_HERE,
       base::Bind(callback, results, base::Passed(std::move(requests))));
 }
 
-void RequestQueueInMemoryStore::ChangeRequestsState(
-    const std::vector<int64_t>& request_ids,
-    const SavePageRequest::RequestState new_state,
-    const UpdateMultipleRequestsCallback& callback) {
-  RequestQueue::UpdateMultipleRequestResults results;
-  std::vector<std::unique_ptr<SavePageRequest>> requests;
-  RequestQueue::UpdateRequestResult result;
-  for (int64_t request_id : request_ids) {
-    auto pair = requests_.find(request_id);
-    // If we find this request id, modify it, and return the modified request in
-    // the request list.
-    if (pair != requests_.end()) {
-      pair->second.set_request_state(new_state);
-      std::unique_ptr<SavePageRequest> request(
-          new SavePageRequest(pair->second));
-      requests.push_back(std::move(request));
-      result = RequestQueue::UpdateRequestResult::SUCCESS;
-    } else {
-      result = RequestQueue::UpdateRequestResult::REQUEST_DOES_NOT_EXIST;;
-    }
-
-    results.push_back(std::make_pair(request_id, result));
-  }
-
-  base::ThreadTaskRunnerHandle::Get()->PostTask(
-      FROM_HERE,
-      base::Bind(callback, results, base::Passed(std::move(requests))));
-}
-
 void RequestQueueInMemoryStore::Reset(const ResetCallback& callback) {
   requests_.clear();
   base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE,
                                                 base::Bind(callback, true));
 }
 
 }  // namespace offline_pages