HostCache: always evict stale entries before valid entries

HostCache: always evict stale entries before valid entries

Previously, HostCache would always evict the entry with the earliest
expiration time. Because entries can also become stale due to network
changes, this could potentially evict a valid entry when stale
entries are present in the cache.

BUG=692803
Review-Url: https://codereview.chromium.org/2701673003
Cr-Commit-Position: refs/heads/master@{#451395}
Committed: https://chromium.googlesource.com/chromium/src/+/9cdf01f7131fa57830221af1156b1a3ca1e11578

[mgersh, 2017-02-16 19:21:14]

mgersh@chromium.org changed reviewers:
+ juliatuttle@chromium.org

[mgersh, 2017-02-16 19:21:14]

PTAL

[Julia Tuttle, 2017-02-17 18:09:15]

On 2017/02/16 19:21:14, mgersh wrote:
> PTAL

lgtm!

[mgersh, 2017-02-17 18:13:30]

The CQ bit was checked by mgersh@chromium.org

[commit-bot: I haz the power, 2017-02-17 18:14:04]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-17 19:16:53]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-17 19:16:54]

Try jobs failed on following builders:
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[mgersh, 2017-02-17 19:43:49]

The CQ bit was checked by mgersh@chromium.org

[commit-bot: I haz the power, 2017-02-17 19:44:28]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-17 23:24:58]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1487360629915090, "parent_rev":
"9ac6e6466cc0df7e1a3ad4488c5c8bdc2db4da36", "commit_rev":
"9cdf01f7131fa57830221af1156b1a3ca1e11578"}

[commit-bot: I haz the power, 2017-02-17 23:25:39]

Description was changed from

==========
HostCache: always evict stale entries before valid entries

Previously, HostCache would always evict the entry with the earliest
expiration time. Because entries can also become stale due to network
changes, this could potentially evict a valid entry when stale
entries are present in the cache.

BUG=692803
==========

to

==========
HostCache: always evict stale entries before valid entries

Previously, HostCache would always evict the entry with the earliest
expiration time. Because entries can also become stale due to network
changes, this could potentially evict a valid entry when stale
entries are present in the cache.

BUG=692803

Review-Url: https://codereview.chromium.org/2701673003
Cr-Commit-Position: refs/heads/master@{#451395}
Committed:
https://chromium.googlesource.com/chromium/src/+/9cdf01f7131fa57830221af1156b...
==========

[commit-bot: I haz the power, 2017-02-17 23:25:40]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/9cdf01f7131fa57830221af1156b...