Issue metadata
Sign in to add a comment
|
40.5% regression in system_health.memory_desktop at 537577:537618 |
||||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Feb 22 2018
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/12dea75f840000
,
Feb 22 2018
😿 Pinpoint job stopped with an error. https://pinpoint-dot-chromeperf.appspot.com/job/12dea75f840000
,
Feb 22 2018
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/15d1546f840000
,
Feb 22 2018
😿 Pinpoint job stopped with an error. https://chromeperf.appspot.com/job/15d1546f840000
,
Mar 16 2018
Assigning to reviewer yhirano, since tyoshino is leaving the project: r537580, "" caused a pretty significant memory regression on this test case. The job had an error, but it bisected to this CL. Can you take a look?
,
Mar 16 2018
Issue 814701 has been merged into this issue.
,
Mar 16 2018
,
Mar 16 2018
Issue 814699 has been merged into this issue.
,
Mar 16 2018
The change was a security fix and we expected that it would introduce some regressions.
,
Mar 16 2018
cc-ing test owner erikchen FYI
,
Mar 16 2018
Please clarify what you mean by "we expected that it would introduce some regressions.". We're observing a ~50MB regression in cc memory usage. It's possible that your CL is triggering some type of non-deterministic loading in WPR, causing more resources [images] to be loaded, and thus more memory usage. It's also possible that your CL has unintended [and unnecessary] side-effects that greatly increase memory usage. Given the magnitude of the regression, the perf team would feel a lot better if you could spend some time investigating and/or explaining the magnitude of the regression.
,
Mar 27 2018
As explained in issue 799477 and issue 809350, reusing a resource that was fetched by another origin leads to a security issue. The CL stopped that, which means with the CL MemoryCache is less used. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Feb 22 2018