Electrolysis/Meetings/2014-08-14

« previous week | index | next week »

jimm

• PTO last week, family in town
• M1 update
  • Yay, I'm the last straggler! I suck! (I disagree) (Also disagree) :)
  • bug 691601 (e10s support for gFormSubmitObserver) - not blocked, just waiting on mstange. ni'd him in the bug this morning.
  • bug 942707 (pdf viewer doesn't work with e10s enabled) - yury has a rev of his extension working with all of my changes. He ran into a problem with findbar, which we confirmed was caused by cpow asserts that landed after I posted my last patch for review. With billm's help we have a fix. Final patch posted for review.
• non-M2 e10s related bugs I've been pulled into this week
  • tracking bug 1052362
  • Most of the pressing work here is either under review or fixed. bug 874437 should fix pretty much everything left outstanding.
• bug 903022 (P1, Save pages to disk) - no real progress on this this week, but I have a good wip with a few remaining issues to clean up.
• bug 1046512 (P1, Marionette breaks in e10s enabled Nightly) - looking for someone to pick this up. (Ally?)
• Everything else I have is P2 or less
• Had a good confab with johns on plugins this week, the first major piece should land in ~2 weeks. After which we should be able to break up remaining work.

bjacob

• PTO last week
• Many gfx things came up:
  • bug 1018278 - Nightly is unusable with direct2d enabled
    • Did the crash analysis, https://bugzilla.mozilla.org/show_bug.cgi?id=1018278#c22
  • bug 1051886 - we almost hid WebGL mochitests due to intermittent bug 777574!!
    • Fixed bug 777574 by skipping the right subset of bad test pages on affected platforms
  • reviewed :handyman's dual-GPU blacklisting patch
• Cocoa plugin focus (bug 1044182): discussed with people, we think this is too nontrivial to be just hacked away and needs to be discussed when Steven Michaud is back, also with Markus Stange and Benoit Girard at least, and possibly needs to be done by a widget/cocoa specialist.
• Printing (bug 927188) turning out to be very hard due to browser/chrome and docshell/document code being too intertwined, not only at the level of toolkit/component/printing but also in layout/printing (e.g. layout/printing code calls directly into widget code). Currently talking with :smaug who is knowledgeable there. Depending on that conversation, might either do it myself or drag in more people for a conversation.

mconley

• Confirmed that bug 1043256, which has a patch that is reviewed and prepped for landing, will fix the icloud.com and work.com logging in issues.
• M2: Bug 1050447 - Address bar lost focus after opening a new tab using Electrolysis
  • I have a patch that is up for review.
• M2: Bug 961362 - [e10s] HTML5 video's fullscreen feature zooms video to fill tab, not screen
  • I have a patch that I will request review on once my try push comes back green.
• M2: Bug 1041917 - [e10s] Bugzilla is not load when I file a bug from "mozilla crash reports"
  • Turns out this is related to other M2 (bug 1047603), but I have a patch for it that works around the problem (changing location from about: page to other page in same browser causes us to reload as a remote browser)
• M2: Bug 1047603 - [e10s] Non-remote tabs in e10s windows do not handle target="_blank" or window.open links properly.
  • How big of a problem is this? How often do we think we'll be opening new tabs or popup windows from non-remote (chrome:// or about:) pages?
  • If it's not a big deal, I can probably fix the original bookmarks sidebar bug by making the bookmarks sidebar content browser as a remote browser.
• Bug 1053981 - [e10s] Wrong <select> dropdown widget on Windows and Linux
  • Spun out of bug 1049285
  • We're drawing the wrong form widgets for Windows. Actually, we're drawing the wrong form widgets for all platforms. Current popups in e10s do not match what we currently ship with non-e10s. This has a number of usability issues. Can I assume we just want to ship what we already ship with non-e10s?
    • Will investigate what it would take to ship what we already ship on non-e10s (though previous attempts sound like it was hard)

allison

• Spellchecker suggest/replace context menu landed Bug 1030451.
  • Now blassey & I sound more intelligent in bug comments. Isn't technology amazing?
  • should also fix the ubuntu bug, reporter having trouble reproducing
• Started work on the social e10s activation bug, 915547
  • this turns out to be the middle of the project that will be social api
  • shane was very helpful
  • 1053974 [e10s] update MozSocialAPI.jsm
  • 1053973 [e10s] update Social.jsm OpenGraphBuilder
  • 1053971 [e10s] update Social.jsm panel resizing code
  • 1053969 [e10s] fix/update social API tests
  • 1053965 [e10s] fix activation from about:home
  • 1053967 [e10s] fix social api share
  • 1053963 [e10s] pass "ActivateSocialFeature" from content process to parent
  • 915547 Get social activation working with e10s

alex_tz

• sent patch for bug 948194 (the one with the decoded images that were not discarded)
• still working on the cmd+e bug (it appears that there is more to this feature than I thought)
• went back to working on the project (phase 1 will hopefully be done soon)