Issue metadata
Sign in to add a comment
|
1.1% regression in rendering.mobile/thread_total_all_cpu_time_per_frame at 604870:604892 |
||||||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Nov 8
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/11c48807e40000
,
Nov 9
📍 Found significant differences after each of 2 commits. https://pinpoint-dot-chromeperf.appspot.com/job/11c48807e40000 ServiceWorker: Add the new class ServiceWorkerModuleTreeClient to by asamidoi@google.com https://chromium.googlesource.com/chromium/src/+/0d4913d31a7a1d3fad1657e6c74391fe4299a119 thread_total_all_cpu_time_per_frame: 98.85 → 96.42 (-2.433) Close the connection with Blink when USB device permission revoked. by donna.wu@intel.com https://chromium.googlesource.com/chromium/src/+/c702987b204456bcdee69daa35c0e7bd94007ee8 thread_total_all_cpu_time_per_frame: 96.77 → 98.04 (+1.267) Understanding performance regressions: http://g.co/ChromePerformanceRegressions Benchmark documentation link: https://bit.ly/rendering-benchmarks
,
Dec 11
It doesn't seems related to the CL directly. Extra clean-up action added to each frame for USB device permission revocation case but that's what we want to do. So won't fix this. Reilly@, what's your opinion?
,
Dec 11
The bot where this is alerting is for WebView, where the WebUSB API is not enabled so the code added in your patch should not have any effect on performance. Looking at the graph it seems like this regression occurred at an earlier change and that is being tracked as issue 901358 . |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Nov 8