Electrolysis/Meetings/2016-04-28: Difference between revisions

Revision as of 14:34, 28 April 2016

Continuing to work on bug 1262671. Should have complete set of patches to post by end of week.

Investigating checksumming IPC messages to detect message corruption
bug 1258317 REOPENED:: crash in mozilla::ipc::TransferHandleToProcess (PBackground related)
- Updated bug 1258317 with new findings from the annotations of new crashes
bug 1259480 NEW:: [e10s] FatalError crash in PContentChild::Read from PContentChild::SendReadPrefsArray
- Investigating dump and tracing code

bug 1253956 NEW:: [e10s] Windows debug browser/base/content/test/general application crashed [@ mozilla::LinkedList<js::WeakMapBase>::~LinkedList<js::WeakMapBase>()] (Assertion failure: isEmpty(), at LinkedList.h:332) on shutdown
- Running crash tests on windows 8

bug 1245068 NEW:: [e10s] Ctrl+mouse wheel to change zoom level doesn't work if web page is not focused
- Found the line that causes the issue, not sure how to fix it yet
bug 1190297 NEW:: Urlbar is cleared after invalid url was entered when electrolysis is enabled.
- Bug 1190297 can't reproduce, ni'd the reporter
bug 1220517 NEW:: Intermittent TEST-UNEXPECTED-FAIL | leakcheck | default process: 84 bytes leaked (Mutex, nsBaseAppShell)
- Responded to the mail from Andrew Overholt for bug 1220517
bug 1257869 NEW:: Window Resize of large page takes seconds to change viewport
- Checked live resize events for windows and gtk, also ni'd :jimm for whether it is still a P1?

bug 1264642 NEW:: Reduce the size of PBrowser::Msg_AsyncMessage IPC messages
- Discussed StructuredClone with baku
- Should have working patch by the end of the week.

Performed a variety of analysis on 47 and 48 for UI Smooooooooothness release criteria
- TL;DR: 48 is signed off, with the proviso that I prioritize the last tpaint regression fix in my work queue and fix it ASAP for 49.
- APZ is responsible for the CART and TART regressions. APZ team says this is the cost of APZ, and difference is not too bad. Trade-off is obvious.
bug 1261842 - Make initial browser remote sooner if we're defaulting to using remote tabs
- Blocked on bug 1267720 - Re-factor nsWindowWatcher::OpenWindowInternal to not be such a rat's nest
  - Hoping to have patches up early next week.
M9 bug 1263887 - [e10s] The dropdown selector is not working properly.
- Landed and bounced a bunch of times, and then finally stuck.
- Going to wait a few more days and then request uplift.
bug 1166351 - XBL parent bindings not properly attached - tabs suddenly do not switch or close properly
- This jerk bug has been around for a year and only now do we get a reproducible test case.
- Enn figured out what is going on.
- Waiting on info from bholley.
The usual reviews. Been on a few interview panels.

@@ Line 37: / Line 37: @@
 == mconley ==
-* TBD
+* Performed a variety of analysis on 47 and 48 for [https://wiki.mozilla.org/Electrolysis/Release_Criteria/UI_Smoothness UI Smooooooooothness] release criteria
+** TL;DR: 48 is signed off, with the proviso that I prioritize the last tpaint regression fix in my work queue and fix it ASAP for 49.
+** APZ is responsible for the CART and TART regressions. APZ team says this is the cost of APZ, and difference is not too bad. Trade-off is obvious.
+* {{bug|1261842}} - Make initial browser remote sooner if we're defaulting to using remote tabs
+** Blocked on {{bug|1267720}} - Re-factor nsWindowWatcher::OpenWindowInternal to not be such a rat's nest
+*** Hoping to have patches up early next week.
+* '''M9''' {{bug|1263887}} - [e10s] The dropdown selector is not working properly.
+** Landed and bounced a bunch of times, and then finally stuck.
+** Going to wait a few more days and then request uplift.
+* {{bug|1166351}} - XBL parent bindings not properly attached - tabs suddenly do not switch or close properly
+** This jerk bug has been around for a year and only now do we get a reproducible test case.
+** [https://bugzilla.mozilla.org/show_bug.cgi?id=1166351#c59 Enn figured out what is going on.]
+** Waiting on info from bholley.
+* The usual reviews. Been on a few interview panels.