Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in

Issue 10948048: [net] Increase HostCache size to 1000 entries on CrOS. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
5 years, 1 month ago by szym
5 years ago
chromium-reviews,,, mmenke


[net] Increase HostCache size to 1000 entries on CrOS. The built-in async DNS client notoriously fills up the small (100-entry) cache. To even out the playfield for the field trial, drastically increase the size. This is a temporary measure before we can tune the size according to performance. BUG=114277 TEST=on CrOS, check HostCache size in chrome://net-internals/#dns Committed:

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -0 lines) Patch
M net/base/ View 1 chunk +7 lines, -0 lines 1 comment Download
Commit queue not available (can’t edit this change).


Total messages: 5 (0 generated)
5 years, 1 month ago (2012-09-19 22:56:45 UTC) #1
5 years ago (2012-09-20 10:03:11 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at
5 years ago (2012-09-20 10:03:23 UTC) #3
commit-bot: I haz the power
Change committed as 157742
5 years ago (2012-09-20 11:02:52 UTC) #4
jar (doing other things)
5 years ago (2012-09-20 17:24:34 UTC) #5
Drive by...
File net/base/ (right):
net/base/ static const size_t kMaxHostCacheEntries = 100;
nit: static on these data items seems like a waste.  I hope the compiler gets
rid of this variable... but static might possibly confuse the issue.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 81bcdb8aa