API to provide status of save page reqeusts to API.

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"
 
 namespace offline_pages {
 
 RequestQueueInMemoryStore::RequestQueueInMemoryStore() {}
 
 RequestQueueInMemoryStore::~RequestQueueInMemoryStore() {}
 
 void RequestQueueInMemoryStore::GetRequests(
     const GetRequestsCallback& callback) {
   std::vector<SavePageRequest> result_requests;
-  for (const auto& id_request_pair : requests_)
+  for (const auto& id_request_pair : requests_) {

[fgorski, 2016/08/02 04:18:12]

nit: do you need the braces here?

[Pete Williamson, 2016/08/03 00:24:32]

Done.

     result_requests.push_back(id_request_pair.second);
+  }
   base::ThreadTaskRunnerHandle::Get()->PostTask(
       FROM_HERE, base::Bind(callback, true, result_requests));
 }
 
 void RequestQueueInMemoryStore::AddOrUpdateRequest(
     const SavePageRequest& request,
     const UpdateCallback& callback) {
   RequestsMap::iterator iter = requests_.find(request.request_id());
   if (iter != requests_.end())
     requests_.erase(iter);
@@ skipping 21 matching lines @@
       FROM_HERE, base::Bind(callback, true, count));
 }
 
 void RequestQueueInMemoryStore::Reset(const ResetCallback& callback) {
   requests_.clear();
   base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE,
                                                 base::Bind(callback, true));
 }
 
 }  // namespace offline_pages