Electrolysis/Meetings/2015-03-26

From MozillaWiki
Jump to: navigation, search

perf

<vladan> http://avih.github.io/blog/2015/03/19/firefox-e10s-performance-on-talos/ <vladan> http://nbviewer.ipython.org/urls/gist.githubusercontent.com/vitillo/cb6f1304316c1c1a2cbc/raw/9e1071d7ee66481215f235bb0f113af5e7fbc51b/e10s%20analysis.ipynb

== ejpbruel == (DevTools)

  • Will join meeting 15 minutes in.
  • Triage:
 * Propose tracking bug 1128988 for M6.
 * Propose upgrading bug 1029451 to M6.
 * Propose downgrading bug 1068400 to M7.
 * Propose upgrading bug 1030318 to M6.

mconley

  • Mined Holly test logs for unsafe CPOW usage warnings. Kwan has been a huge help here.
  • M6 bug 1110887 - With e10s, plugin crash submit UI is broken
    • I have the crash report UI showing and I can submit crash reports. WIP is up.
    • Q: When a plugin crashes, does it ever leave the DOM - even for a split second - before it's replaced by the crashed plugin UI? I depend on being able to search for plugins with a particular run ID at arbitrary times post-crash using nsIDOMWindowUtils::GetPlugins.
    • I need to fix up a bunch of tests, and write some new ones to exercise both cases of some non-determinism that plugin crashes in e10s introduces.
    • Should have a reviewable patch series tomorrow or Monday at the latest.
  • bug 1147156 - Re-enable e10s by default for Nightly users who may have disabled it
    • Should we do this after the next uplift? See commentary in that bug for context.
  • Did a bunch of reviews

Tracy

  • Reading List compatibility with e10s is in good shape.
  • Taking over Juans work digging into addons compat/arewee10syet
  • pre-triage
  • determining Q2 goals
  • on PTO tomorrow (3/27)

jimm

  • bug 1129040 (addon/plugin blocklist broken) - landed. Filed follow up on tests.
  • bug 1145208 (plugin browser-chrome tests) - wip, mostly done, pushing to try today looking for random orange.
  • bug 1122525 (Save image has bad filename in file picker) - need to update per review comments
  • bug 1132874 (content process aborts in PPluginWidget::Msg_ParentShutdown) - looking at this again next.

Mossop

  • Reviews, so many reviews
  • Bug 1144578: Intermittent e10s test fix
  • Next up, tab detach bug 1087969 but blocked on billm in bug 1132072

felipe

Bug 1129564 m6 - [e10s] anchor navigation from the awesomebar removes the lock icon - upcoming patch today

  • Bug 1079665 m6 - findbar focuscontent - fixed

Gabor

  • review queue is empty... just saying :)
  • pdf.js CPOW removal patch: landed (\o/)
  • Bug 982319 - Network monitor (gzip) - need info pending
  • Bug 1133761 - warnings for shims - working on a patch, have the
 warning code already, got a list of things to warn about, should go
 quickly from here on
  • Bug 1146298 - Dave had a nice alternative solution, made quite some
 security related investigation
  • domainPolicy patch is landed

mrbkap

  • Found problem with my remaining m5 patch, now waiting for build peer review.
  • Working on m6 bugs, right now, <datalist> element.

handyman

  • bug 1075670 - [e10s] event.screenX and event.screenY is wrong
    • Skeptical of the legitimacy of the crash stack. It appears to be truncated.
    • Attempting to land
  • Bug 1018639 - [e10s] Mouse cursor indicates a bidirectional resize
    • Windows-specific
    • One patch approved, one patch in review
  • Bug 1138678 - Crash in [ChildView keyDown:] when dismissing password field in chrome content
    • Patch approved and landed
  • Bug 1125325 - Web content is not rescaled when OS X moves browser window from HiDPI laptop display to external display.
    • Patch not approved

billm

  • Tab switching bug
  • Thinking more about addons

gw280

  • Looked at the VMware/RDP e10s rendering issues, and tracked them all down to the same root cause (D3D11 WARP)
  • This is now fixed, Bas quickly got a solution together and it has r+. Should land really soon.
  • Looking at the RSS feed bug - subscriber UI doesn't work currently in e10s.

notes

  • Add-on community coordinator: Jorge Villalobos

« previous week | index | next week »