Commit ec9e5529 authored by Michael Crouse's avatar Michael Crouse Committed by Commit Bot

Resolve DCHECK issues for fetching timer.

This looks to be the only place that the timer can be running and
ScheduleFetch can be called.


Bug: 1111815
Change-Id: Ic7e3bdacb610e48ac7934e2ec2712ef80c92edb3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2332298
Commit-Queue: Michael Crouse <mcrouse@chromium.org>
Reviewed-by: default avatarSophie Chang <sophiechang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#793748}
parent fee2db42
...@@ -565,7 +565,8 @@ void OptimizationGuideHintsManager::MaybeScheduleTopHostsHintsFetch() { ...@@ -565,7 +565,8 @@ void OptimizationGuideHintsManager::MaybeScheduleTopHostsHintsFetch() {
if (optimization_guide::switches::ShouldOverrideFetchHintsTimer()) { if (optimization_guide::switches::ShouldOverrideFetchHintsTimer()) {
SetLastHintsFetchAttemptTime(clock_->Now()); SetLastHintsFetchAttemptTime(clock_->Now());
FetchTopHostsHints(); FetchTopHostsHints();
} else { } else if (!top_hosts_hints_fetch_timer_.IsRunning()) {
// Only Schedule this is the time is not already running.
ScheduleTopHostsHintsFetch(); ScheduleTopHostsHintsFetch();
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment