[Offline pages] Removing obsolete TODOs as part of PE fixit

components/offline_pages/core/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/core/background/request_queue.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/location.h"
@@ skipping 13 matching lines @@
 
 namespace offline_pages {
 
 namespace {
 // Completes the get requests call.
 void GetRequestsDone(const RequestQueue::GetRequestsCallback& callback,
                      bool success,
                      std::vector<std::unique_ptr<SavePageRequest>> requests) {
   GetRequestsResult result =
       success ? GetRequestsResult::SUCCESS : 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, std::move(requests));
 }
 
 // Completes the add request call.
 void AddRequestDone(const RequestQueue::AddRequestCallback& callback,
                     const SavePageRequest& request,
                     ItemActionStatus status) {
   AddRequestResult result;
   switch (status) {
     case ItemActionStatus::SUCCESS:
@@ skipping 23 matching lines @@
 RequestQueue::~RequestQueue() {}
 
 void RequestQueue::GetRequests(const GetRequestsCallback& callback) {
   std::unique_ptr<Task> task(new GetRequestsTask(
       store_.get(), base::Bind(&GetRequestsDone, callback)));
   task_queue_.AddTask(std::move(task));
 }
 
 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.
   std::unique_ptr<AddRequestTask> task(new AddRequestTask(
       store_.get(), request, base::Bind(&AddRequestDone, callback, request)));
   task_queue_.AddTask(std::move(task));
 }
 
 void RequestQueue::RemoveRequests(const std::vector<int64_t>& request_ids,
                                   const UpdateCallback& callback) {
   std::unique_ptr<Task> task(
       new RemoveRequestsTask(store_.get(), request_ids, callback));
   task_queue_.AddTask(std::move(task));
@@ skipping 69 matching lines @@
                                weak_ptr_factory_.GetWeakPtr())));
   task_queue_.AddTask(std::move(task));
 }
 
 void RequestQueue::InitializeStoreDone(bool success) {
   // TODO(fgorski): Result can be ignored for now. Report UMA in future.
   // No need to pass the result up to RequestCoordinator.
 }
 
 }  // namespace offline_pages