New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 827143 link

Starred by 4 users

Issue metadata

Status: Assigned
Owner:
Last visit > 30 days ago
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

1.1% regression in system_health.memory_desktop at 546159:546298

Project Member Reported by kraynov@chromium.org, Mar 29 2018

Issue description

See the link to graphs below.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Mar 29 2018

All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=827143

(For debugging:) Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?sid=d4601ede1ee7b4b4bd7949c3bbac6ae98aaf1a2a193e1d32e56926a7de0e35a3


Bot(s) for this bug's original alert(s):

chromium-rel-win8-dual
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Mar 29 2018

Cc: robpercival@chromium.org jwd@chromium.org
Owner: robpercival@chromium.org
Status: Assigned (was: Untriaged)
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/15d38f2b440000

Reland "Add test coverage of field trial for Certificate Transparency log auditing" by robpercival@chromium.org
https://chromium.googlesource.com/chromium/src/+/12795821a0a35b9ca4a6145d9c60da691b356daa

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions
Components: Internals>Network>CertTrans
I'm not sure why Windows would be seeing a significant increase in memory usage with the CertificateTransparencyLogAuditing feature enabled, since it doesn't do much on that platform. At the time that this bug was created, the only effect of enabling this feature on Windows was that a lookup would be performed in the net::HostCache for each host that provided Signed Certificate Timestamps. This shouldn't affect memory usage. 

We've since made some changes that should reduce memory usage when the feature is disabled (https://crrev.com/c/1009913). The only thing I can suggest would be to remove Windows from the field trial config for the time being; the feature isn't planned for launch on Windows anyway.

Does anyone have recommendations for investigating this? The docs weren't particularly helpful for tracking down the cause of this diff.

Sign in to add a comment