API to provide status of save page reqeusts to API.

components/offline_pages/background/request_queue.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.h"
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "components/offline_pages/background/request_queue_store.h"
 #include "components/offline_pages/background/save_page_request.h"
 
 namespace offline_pages {
 
 namespace {
 // Completes the get requests call.
 void GetRequestsDone(const RequestQueue::GetRequestsCallback& callback,
                      bool success,
                      const std::vector<SavePageRequest>& requests) {
+  DVLOG(0) << "@@@@@@ " << __func__;

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

nit: do you need to keep these?

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

Nope, leftover from debugging.  I must have missed them on my pass to clean up
for code review.

   RequestQueue::GetRequestsResult result =
       success ? RequestQueue::GetRequestsResult::SUCCESS
               : RequestQueue::GetRequestsResult::STORE_FAILURE;
   // TODO(fgorski): Filter out expired requests based on policy.
   // This may trigger the purging if necessary.
   // Also this may be turned into a method on the request queue or add a policy
   // parameter in the process.
   callback.Run(result, requests);
 }
 
@@ skipping 31 matching lines @@
 }
 
 }  // namespace
 
 RequestQueue::RequestQueue(std::unique_ptr<RequestQueueStore> store)
     : store_(std::move(store)), weak_ptr_factory_(this) {}
 
 RequestQueue::~RequestQueue() {}
 
 void RequestQueue::GetRequests(const GetRequestsCallback& callback) {
+  DVLOG(0) << "@@@@@@ " << __func__;
   store_->GetRequests(base::Bind(&GetRequestsDone, callback));
 }
 
 void RequestQueue::AddRequest(const SavePageRequest& request,
                               const AddRequestCallback& callback) {
   // TODO(fgorski): check that request makes sense.
   // TODO(fgorski): check that request does not violate policy.
   store_->AddOrUpdateRequest(request,
                              base::Bind(&AddRequestDone, callback, request));
 }
@@ skipping 50 matching lines @@
 
 void RequestQueue::RemoveRequest(int64_t request_id,
                                  const UpdateRequestCallback& callback) {
   std::vector<int64_t> request_ids{request_id};
   store_->RemoveRequests(request_ids, base::Bind(RemoveRequestDone, callback));
 }
 
 void RequestQueue::PurgeRequests(const PurgeRequestsCallback& callback) {}
 
 }  // namespace offline_pages