Issue metadata
Sign in to add a comment
|
4.9% regression in system_health.memory_mobile at 515123:515167 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Mar 22 2018
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/12e868fd440000
,
Mar 23 2018
📍 Found significant differences after each of 4 commits. https://pinpoint-dot-chromeperf.appspot.com/job/12e868fd440000 [Password Manager] Show confirmation bubble as manual fallback for generated cases by kolos@chromium.org https://chromium.googlesource.com/chromium/src/+/eb2186f3682000349de07278cc013da765ba364c [snapshot] Flip --lazy-handler-deserialization by jgruber@chromium.org https://chromium.googlesource.com/v8/v8/+/9664d017f7e219878d4b1a6c25be6c6f9b1b91bd Reland "Make TracingControllerBackend.StopTracing() return nonfatal exceptions" by charliea@chromium.org https://chromium.googlesource.com/catapult/+/013ddeb8a2fd8d246de026c043d915cf895c41c0 Update wprgo binaries to latest version by nednguyen@google.com https://chromium.googlesource.com/catapult/+/564e33d345ee080be36ad0e6b6107afbcd09df4d Understanding performance regressions: http://g.co/ChromePerformanceRegressions
,
Mar 30 2018
Ned, is it expected that the wprgo update would cause a memory increase? If so, can you mark WontFix?
,
Mar 30 2018
It's expected to contribute some of the memory increase, but I am not sure how much. What do we do about the other commits identified by pinpoint?
,
Mar 30 2018
I highly suspect that this is mostly caused by wprgo patch because I don't see memory regression on the desktop benchmark
,
Apr 5 2018
WontFix, since the wprgo change had expected memory impact. If you look at the pinpoint chart, the other CLs caused very small memory improvements, and pinpoint detects all changes. |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Mar 22 2018