Bug 1546309 Comment 52 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Part3 patch, with fixes for some eslint issues the try server caught.  Try run for this version: 
https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=8a259f601602694cc527641554aa243f32971b07

Requesting review of the mozmill changes.  (As jorgk says "first reviewer wins".)  

Unfortunately, the new appmenu works differently from the other menus and the previous appmenu, so I had to add some code for navigating it.  The new appmenu needed an async / event listener approach to testing it.  (More details in the comments at the top of this file: https://searchfox.org/mozilla-central/source/browser/components/customizableui/PanelMultiView.jsm)
Part3 patch, with fixes for some eslint issues the try server caught, and rebased on current comm-beta.  Try run for this version: 
https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=8a259f601602694cc527641554aa243f32971b07

Requesting review of the mozmill changes.  (As jorgk says "first reviewer wins".)  

Unfortunately, the new appmenu works differently from the other menus and the previous appmenu, so I had to add some code for navigating it.  The new appmenu needed an async / event listener approach to testing it.  (More details in the comments at the top of this file: https://searchfox.org/mozilla-central/source/browser/components/customizableui/PanelMultiView.jsm)

Back to Bug 1546309 Comment 52