Confirmed users, Bureaucrats and Sysops emeriti
998
edits
Electrolysis/Meetings/2016-02-25: Difference between revisions
Electrolysis/Meetings/2016-02-25 (view source)
Revision as of 16:28, 25 February 2016
, 25 February 2016→mconley: Add mconley stuff.
(jimm) |
(→mconley: Add mconley stuff.) |
||
| Line 17: | Line 17: | ||
** FINALLY landed this. smaug, mccr8 and ting helped huge, and we found some leaks along the way. | ** FINALLY landed this. smaug, mccr8 and ting helped huge, and we found some leaks along the way. | ||
** (The short story is that my patch fixed half of a leak, which caused many more things to be added to the cycle-collector graph, but those things were kept alive too long so the graph got larger and larger, and then on shutdown when analyzing the cc logs, we'd take too long and miss a native event, which would cause us to leak nsBaseAppShell.) | ** (The short story is that my patch fixed half of a leak, which caused many more things to be added to the cycle-collector graph, but those things were kept alive too long so the graph got larger and larger, and then on shutdown when analyzing the cc logs, we'd take too long and miss a native event, which would cause us to leak nsBaseAppShell.) | ||
* '''Perf P2''' {{bug|1174770}} - tpaint regressions (3%-33%) in e10s mode compared to non-e10s mode | |||
** Found that we're sending two sync IPC messages, one after the other, accounting for > 50% of the regression. Filed {{bug|1251032}} | |||
** We're also sending two sync IPC messages for IME. Filed {{bug|1251063}} | |||
* Bug 666365 - Figure out what TabChild::SetVisibility needs to do | |||
** I have a needinfo on this, and it's taking me a while to sort this out. I _think_ we're doing the right thing by just having the TabChild be "visible" by default, but I haven't fully verified this yet. | |||