| Index: components/offline_pages/background/request_picker.cc
|
| diff --git a/components/offline_pages/background/request_picker.cc b/components/offline_pages/background/request_picker.cc
|
| index a199a372212959e7ebfb210e439e7a2f06ad8f54..e0b03d4aacf6ef240d9a8a37b91bac27d0a77391 100644
|
| --- a/components/offline_pages/background/request_picker.cc
|
| +++ b/components/offline_pages/background/request_picker.cc
|
| @@ -242,7 +242,6 @@ void RequestPicker::SplitRequests(
|
| std::vector<std::unique_ptr<SavePageRequest>> requests,
|
| std::vector<std::unique_ptr<SavePageRequest>>* valid_requests,
|
| std::vector<std::unique_ptr<SavePageRequest>>* expired_requests) {
|
| - std::vector<std::unique_ptr<SavePageRequest>>::iterator request;
|
| for (auto& request : requests) {
|
| if (base::Time::Now() - request->creation_time() >=
|
| base::TimeDelta::FromSeconds(kRequestExpirationTimeInSeconds)) {
|
| @@ -258,14 +257,13 @@ void RequestPicker::SplitRequests(
|
| void RequestPicker::OnRequestExpired(
|
| const RequestQueue::UpdateMultipleRequestResults& results,
|
| const std::vector<std::unique_ptr<SavePageRequest>> requests) {
|
| - std::vector<std::unique_ptr<SavePageRequest>>::const_iterator request;
|
| - for (request = requests.begin(); request != requests.end(); ++request) {
|
| + for (const auto& request : requests) {
|
| const RequestCoordinator::BackgroundSavePageResult result(
|
| RequestCoordinator::BackgroundSavePageResult::EXPIRED);
|
| event_logger_->RecordDroppedSavePageRequest(
|
| - request->get()->client_id().name_space, result,
|
| - request->get()->request_id());
|
| - notifier_->NotifyCompleted(*(request->get()), result);
|
| + request->client_id().name_space, result,
|
| + request->request_id());
|
| + notifier_->NotifyCompleted(*request, result);
|
| }
|
| }
|
|
|
|
|