Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(124)

Unified Diff: components/offline_pages/background/request_coordinator_unittest.cc

Issue 2373933003: [Offline pages] Updating RequestQueue::RemoveRequests to use a TaskQueue (Closed)
Patch Set: Rebasing Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/offline_pages/background/request_coordinator_unittest.cc
diff --git a/components/offline_pages/background/request_coordinator_unittest.cc b/components/offline_pages/background/request_coordinator_unittest.cc
index f17f2993b0bdec2d0b1f6a46b9dd8138544276ee..e36c55ef8663e6348b47f66c64cc83e916261ab4 100644
--- a/components/offline_pages/background/request_coordinator_unittest.cc
+++ b/components/offline_pages/background/request_coordinator_unittest.cc
@@ -233,8 +233,7 @@ class RequestCoordinatorTest
std::vector<std::unique_ptr<SavePageRequest>> requests);
// Callback for removing requests.
- void RemoveRequestsDone(
- const RequestQueue::UpdateMultipleRequestResults& results);
+ void RemoveRequestsDone(const MultipleItemStatuses& results);
// Callback for getting request statuses.
void GetQueuedRequestsDone(
@@ -251,8 +250,7 @@ class RequestCoordinatorTest
return last_requests_;
}
- const RequestQueue::UpdateMultipleRequestResults& last_remove_results()
- const {
+ const MultipleItemStatuses& last_remove_results() const {
return last_remove_results_;
}
@@ -301,7 +299,7 @@ class RequestCoordinatorTest
private:
RequestQueue::GetRequestsResult last_get_requests_result_;
- RequestQueue::UpdateMultipleRequestResults last_remove_results_;
+ MultipleItemStatuses last_remove_results_;
std::vector<std::unique_ptr<SavePageRequest>> last_requests_;
scoped_refptr<base::TestMockTimeTaskRunner> task_runner_;
base::ThreadTaskRunnerHandle task_runner_handle_;
@@ -355,7 +353,7 @@ void RequestCoordinatorTest::GetRequestsDone(
}
void RequestCoordinatorTest::RemoveRequestsDone(
- const RequestQueue::UpdateMultipleRequestResults& results) {
+ const MultipleItemStatuses& results) {
last_remove_results_ = results;
waiter_.Signal();
}

Powered by Google App Engine
This is Rietveld 408576698