Electrolysis/Meetings/2015-07-16

From MozillaWiki
Jump to: navigation, search

mconley

  • M8 - bug 1173219 - Make TabParent::RecvCreateWindow less crash prone
    • Uplifted to Aurora. jimm reports big drop-off on child aborts with this signature. \o/
  • M8 - bug 1114299 - [e10s] link with target="_blank" opens window with no tabs/toolbars/menus
    • Dealing with test failures.
  • M8 - bug 1146454 - Remove CPOW usage in printing
    • Fixed, landed.
  • M8 - bug 1158798 - [e10s] If mouse is plugged in, inline scrollbars are not used even if OS X is configured for them
    • Fixed, landed
  • M8 - bug 1181475 - [e10s] Shift + back/forward button opens page in a new window but renders blank
    • I know what's going on now. I have a proposed solution - waiting to hear back from ttaubert about it.
  • M8 - bug 1106321 - Printing Page Setup Settings Disobeyed Under e10s
    • I know what's happening here now - there are a number of Windows-specific things that we weren't serializing correctly.
    • I'm realizing now that if we end up kicking off printing from the parent process, this serialization work might end up being mostly throw-away. :/ Oh well.
  • Lots of reviewing.
  • Performance:
    • Latest comparison report: https://pastebin.mozilla.org/8839348
      • Includes damp, epic wins, and uses geometric mean now instead of standard average.
    • Bug 1166132 - Create new talos test to measure tab switching times
      • Helped shepherd this through.
      • We have tps reports coming in on the "g2" tests! (???)
    • Have started to tease profiles out of our talos machines running e10s. This has already yielded very useful information.
    • bug 1174776 - 8.5%-30% tp5o talos regression with e10s enabled on all three platforms
      • Partially caused by lack of consistent GC/CC in the content process between pageloads. BenWa has a patch for this which gives us a good chunk of win.
      • BenWa also noticed that we seem to be doing extra painting in the content process, and is investigating.

gabor

  • I was mainly busy with reviews this week.
  • Did some thinking about add-on strategy (see addons-notes)
  • Checked for missing bugs around the add-on API in general
  • Bug 1170484 - debugged the facebook related spelling checker bug a bit more

in the end came to he conclusion that I have to wait for khuey to finish up his work first

  • Bug 863512 - Fix drag and drop PageThumbs callers in browser - started reading

the code. At first glance it should just work, not sure what is wrong, Jim seems to have a stack that should show where the issue is, but I don't understand it yet.

ursula

  • Working away on Bug 1021654, patch is going through review process and I'm fixing tests
  • Encountering new issue with page thumbs not displaying for sites that you've visited - will investigate more


Enn

1132518 (m7) - document navigation with F6 is now done 1066663 (m8) - selecting text fails in some cases, now done 1176239 - tab navigation gets stuck, added a test, being reviewed 1182607 - select popup not being closed on click, added a patch and is being reviewed


jimm

  • bug 1080687 (m7, tabs crash on system shutdown) - fixed
  • bug 1174461 (m8, SendGetNativePluginPort caching) - landed but was backed out due to crashiness. will revisit later.
  • bug 1128454 (m8, SendLoadPlugin failures) - additional annotation patches landed. Hard to tell what causes this, likely we'll just avoid aborting when this happens to address. But I'd like to go for one more round of feedback.
  • made some improvements in the KillHard tracking reports to better coalesce signatures. Should update today.

mrbkap

  • Attached a patch for <datalist> regression affecting non-e10s.
  • Started looking at authenticated FTP. Looks fairly simple.
  • Lots of reviews!
  • Some PTO this and last week.
  • Working through needinfo queue.

jimicy

  • M8 Bug 653065 - Make the lightweight theme web installer ready for e10s.
   * r+ and merged in
  • M8 Bug 1040947 - Opening page info from a remote tab is sluggish
   * r+ before. But we found an existing error in the original code that happens in the new code too. Will fix that and it will be merged in
  • M8 Bug 1116478 - Still working on it.

gw280

  • Bug 1179478 (m8) - Tooltips in <select> patch written and r+, will land today
  • Bug 1168691 (m8) - LTR not working properlty in <select>. patch written and r+, will land today.
  • Bug 1157941 (m7) - FINALLY able to reproduce this bug. Had to get a loaner machine from releng. Have narrowed down the piece of code that's problematic, but haven't worked out a good solution for it yet.

blassey

bug 1160166 - Disabled plugin meta information often isn't available to content processes