Chromium Code Reviews
Help | Chromium Project | Sign in
(10)

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

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 7 months ago by szym
Modified:
1 year, 7 months ago
CC:
chromium-reviews_chromium.org, cbentzel+watch_chromium.org, darin-cc_chromium.org, mmenke
Visibility:
Public.

Description

[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: http://src.chromium.org/viewvc/chrome?view=rev&revision=157742

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -0 lines) Lint Patch
M net/base/host_cache.cc View 1 chunk +7 lines, -0 lines 1 comment ? errors Download
Commit:

Messages

Total messages: 5
szym
1 year, 7 months ago #1
cbentzel (out until April 27)
LGTM
1 year, 7 months ago #2
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/szym@chromium.org/10948048/1
1 year, 7 months ago #3
I haz the power (commit-bot)
Change committed as 157742
1 year, 7 months ago #4
jar
1 year, 7 months ago #5
Drive by...

https://chromiumcodereview.appspot.com/10948048/diff/1/net/base/host_cache.cc
File net/base/host_cache.cc (right):

https://chromiumcodereview.appspot.com/10948048/diff/1/net/base/host_cache.cc...
net/base/host_cache.cc:81: 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 1280:2d3e6564b7b6