GitLab

The "last_survey_check_time" preference value associated with a survey
trigger limits the rate at which contact is attempted to be made with
the HaTS service to once per 24hrs.

Previously the creation of a HaTS Next dialog, which immediately
attempts to contact the HaTS service, did not set this value. This
could result in an individual client making unlimited requests to the
HaTS service.

This CL adjusts logic such that this value is set on creation of a
HaTS Next dialog, and is consulted earlier in attempting to show a
survey. Specifically as part of the CanShowSurvey check, rather than
CheckSurveyStatusAndMaybeShow.

Bug: 1110888
Change-Id: I289245893ed0ff7c49afeb5b560b970df02a5416
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2640433Reviewed-by: Peter Marshall <petermarshall@chromium.org>
Commit-Queue: Theodore Olsauskas-Warren <sauski@google.com>
Cr-Commit-Position: refs/heads/master@{#845563}