Cleanup old TODO comments, removing ones that are no longer relevant, and adding bug numbers to one…

components/offline_pages/core/background/pick_request_task.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/pick_request_task.h"
 
 #include <unordered_set>
 
 #include "base/bind.h"
 #include "base/logging.h"
@@ skipping 66 matching lines @@
   const SavePageRequest* picked_request = nullptr;
 
   RequestCompareFunction comparator = nullptr;
 
   // Choose which comparison function to use based on policy.
   if (policy_->RetryCountIsMoreImportantThanRecency())
     comparator = &PickRequestTask::RetryCountFirstCompareFunction;
   else
     comparator = &PickRequestTask::RecencyFirstCompareFunction;
 
-  // TODO(petewil): Consider replacing this bool with a better named enum.
   bool non_user_requested_tasks_remaining = false;
   bool cleanup_needed = false;
 
   size_t available_request_count = 0;
   // Request ids which are available for picking.
   std::unordered_set<int64_t> available_request_ids;
 
   // Iterate through the requests, filter out unavailable requests and get other
   // information (if cleanup is needed and number of non-user-requested
   // requests).
@@ skipping 93 matching lines @@
   if (device_conditions_->GetBatteryPercentage() <
       policy_->BatteryPercentageRequired(request->user_requested())) {
     return false;
   }
 
   // If the request is paused, do not consider it.
   if (request->request_state() == SavePageRequest::RequestState::PAUSED)
     return false;
 
   // If this request is not active yet, return false.
-  // TODO(petewil): If the only reason we return nothing to do is that we have
-  // inactive requests, we still want to try again later after their activation
-  // time elapses, we shouldn't take ourselves completely off the scheduler.
+  // TODO(petewil): Remove the concept of activation time.  crbug.com/705103.
   if (request->activation_time() > base::Time::Now())
     return false;
 
   return true;
 }
 
 // Look at policies to decide which requests to prefer.
 bool PickRequestTask::IsNewRequestBetter(const SavePageRequest* oldRequest,
                                          const SavePageRequest* newRequest,
                                          RequestCompareFunction comparator) {
@@ skipping 68 matching lines @@
   int result = signum(difference.InMilliseconds());
 
   // Flip the direction of comparison if policy prefers fewer retries.
   if (policy_->ShouldPreferEarlierRequests())
     result *= -1;
 
   return result;
 }
 
 }  // namespace offline_pages