Issue metadata
Sign in to add a comment
|
11.4%-13.6% regression in blink_perf.owp_storage at 563199:563225 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Jun 2 2018
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/14e1305e240000
,
Jun 6 2018
📍 Found a significant difference after 1 commit. https://pinpoint-dot-chromeperf.appspot.com/job/14e1305e240000 Remove PausableTimer from DOMWindowEventQueue by hajimehoshi@chromium.org https://chromium.googlesource.com/chromium/src/+/cb5d04f327ea4807d4db8dbf2750368673618361 27.73 → 32.13 (+4.4) Understanding performance regressions: http://g.co/ChromePerformanceRegressions
,
Jun 26 2018
,
Jun 26 2018
I understood the reason: Before the CL, events were queued to a hash set and all the queued events were dispatched when a timer was fired. After the CL, each event was queued as a task and dispatched one by one. I'd like to mark this issue 'wontfix' since this is a kind of WAI. +jsbell (an owner of indexeddb part), what do you think?
,
Jun 26 2018
Should we file a bug on general scheduling performance optimisations and block this on that? Agreed that no immediate action is necessary.
,
Jun 26 2018
Another similar issue is https://bugs.chromium.org/p/chromium/issues/detail?id=849045, but this is a little different thing since I'm still not sure the culprit of 849045.
,
Jun 26 2018
> Should we file a bug on general scheduling performance optimisations and block this on that? The number of similar problems are two including this as far as I know, and as I commented, the causes seem different. Then I'd like to leave them as they are.
,
Jul 30
|
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Jun 2 2018