Issue metadata
Sign in to add a comment
|
1.1% regression in system_health.memory_desktop at 546159:546298 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Mar 29 2018
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/15d38f2b440000
,
Mar 29 2018
📍 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
,
Mar 30 2018
,
Apr 24 2018
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 |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Mar 29 2018