Chromium Code Reviews
Help | Chromium Project | Sign in

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

Can't Edit
Can't Publish+Mail
Start Review
3 years ago by szym
3 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
Project "None" does not have a commit queue.


Total messages: 5 (0 generated)
3 years ago (2012-09-19 22:56:45 UTC) #1
3 years ago (2012-09-20 10:03:11 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at
3 years ago (2012-09-20 10:03:23 UTC) #3
commit-bot: I haz the power
Change committed as 157742
3 years ago (2012-09-20 11:02:52 UTC) #4
jar (doing other things)
3 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 354b513