Skip to content

Commit

Permalink
Cleanup old TODOs that got done or are no longer relevant.
Browse files Browse the repository at this point in the history
BUG=610521

Review-Url: https://codereview.chromium.org/2189443004
Cr-Commit-Position: refs/heads/master@{#408222}
  • Loading branch information
petewil authored and Commit bot committed Jul 27, 2016
1 parent eb96532 commit 3df75a1
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions components/offline_pages/background/request_coordinator.cc
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,6 @@ bool RequestCoordinator::SavePageLater(
static int64_t id = 0;

// Build a SavePageRequest.
// TODO(petewil): Use something like base::Clock to help in testing.
offline_pages::SavePageRequest request(
id++, url, client_id, base::Time::Now(), was_user_requested);

Expand All @@ -76,8 +75,7 @@ void RequestCoordinator::AddRequestResultCallback(
RequestQueue::AddRequestResult result,
const SavePageRequest& request) {

// Inform the scheduler that we have an outstanding task.
// TODO(petewil): Determine trigger conditions from policy.
// Inform the scheduler that we have an outstanding task..
scheduler_->Schedule(GetTriggerConditionsForUserRequest());
}

Expand Down Expand Up @@ -121,8 +119,6 @@ bool RequestCoordinator::StartProcessing(

is_canceled_ = false;
scheduler_callback_ = callback;
// TODO(petewil): Check existing conditions (should be passed down from
// BackgroundTask)

TryNextRequest();

Expand Down

0 comments on commit 3df75a1

Please sign in to comment.