Allow continuous request dispatch below the throttle rate

Allow continuous request dispatch below the throttle rate

ResourceDispatchThrottler will throttle requests when a give threshold
is reached within a given time window. However, it is overly restrictive
when the requests are continuous, as the sliding time window was keyed
off the most recently sent request.

Fix this by tracking the flush time, rather than the last sent request
time, allowing continuous request dispatch at or below the specified
throttling rate.

BUG=592037
Committed: https://crrev.com/46c01a93646af14072ce25fc59a9b59b29f097d7
Cr-Commit-Position: refs/heads/master@{#383740}

[jdduke (slow), 2016-03-25 16:57:50]

Description was changed from

==========
Allow continuous request dispatch below the throttle rate

ResourceDispatchThrottler will throttle requests when a give threshold
is reached within a given time window. However, it is overly restrictive
when the requests are continuous, as the sliding time window was keyed
off the most recently sent request.

Fix this by tracking the flush time, rather than the last sent request
time, allowing continuous request dispatch at or below the specified
throttling rate.

BUG=592037
==========

to

==========
Allow continuous request dispatch below the throttle rate

ResourceDispatchThrottler will throttle requests when a give threshold
is reached within a given time window. However, it is overly restrictive
when the requests are continuous, as the sliding time window was keyed
off the most recently sent request.

Fix this by tracking the flush time, rather than the last sent request
time, allowing continuous request dispatch at or below the specified
throttling rate.

BUG=592037
==========

[jdduke (slow), 2016-03-25 16:57:50]

jdduke@chromium.org changed reviewers:
+ alexclarke@chromium.org

[jdduke (slow), 2016-03-25 16:57:59]

Hey guys :)

[alex clarke (OOO till 29th), 2016-03-29 11:32:10]

lgtm

Thanks Jarred.  Sami tried this patch with the bug repro and it looked like it
fixed the problem.

[jdduke (slow), 2016-03-29 15:21:58]

The CQ bit was checked by jdduke@chromium.org

[commit-bot: I haz the power, 2016-03-29 15:22:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1834853003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1834853003/20001

[commit-bot: I haz the power, 2016-03-29 16:33:45]

Description was changed from

==========
Allow continuous request dispatch below the throttle rate

ResourceDispatchThrottler will throttle requests when a give threshold
is reached within a given time window. However, it is overly restrictive
when the requests are continuous, as the sliding time window was keyed
off the most recently sent request.

Fix this by tracking the flush time, rather than the last sent request
time, allowing continuous request dispatch at or below the specified
throttling rate.

BUG=592037
==========

to

==========
Allow continuous request dispatch below the throttle rate

ResourceDispatchThrottler will throttle requests when a give threshold
is reached within a given time window. However, it is overly restrictive
when the requests are continuous, as the sliding time window was keyed
off the most recently sent request.

Fix this by tracking the flush time, rather than the last sent request
time, allowing continuous request dispatch at or below the specified
throttling rate.

BUG=592037
==========

[commit-bot: I haz the power, 2016-03-29 16:33:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-29 16:35:09]

Description was changed from

==========
Allow continuous request dispatch below the throttle rate

ResourceDispatchThrottler will throttle requests when a give threshold
is reached within a given time window. However, it is overly restrictive
when the requests are continuous, as the sliding time window was keyed
off the most recently sent request.

Fix this by tracking the flush time, rather than the last sent request
time, allowing continuous request dispatch at or below the specified
throttling rate.

BUG=592037
==========

to

==========
Allow continuous request dispatch below the throttle rate

ResourceDispatchThrottler will throttle requests when a give threshold
is reached within a given time window. However, it is overly restrictive
when the requests are continuous, as the sliding time window was keyed
off the most recently sent request.

Fix this by tracking the flush time, rather than the last sent request
time, allowing continuous request dispatch at or below the specified
throttling rate.

BUG=592037

Committed: https://crrev.com/46c01a93646af14072ce25fc59a9b59b29f097d7
Cr-Commit-Position: refs/heads/master@{#383740}
==========

[commit-bot: I haz the power, 2016-03-29 16:35:09]

Patchset 2 (id:??) landed as
https://crrev.com/46c01a93646af14072ce25fc59a9b59b29f097d7
Cr-Commit-Position: refs/heads/master@{#383740}