+++ This bug was initially created as a clone of Bug #1489447 +++ Remove broadcasters from editor/. AFAICT, the <broadcaster id="args" value=""/> is some ancient artefact which isn't used for anything anymore. I can't find any reference to it being used, and composition seems to work fine without it. Most other broadcasters here aren't used in Thunderbird. There is some usage in SeaMonkey.

Comment on attachment 9030201 [details] [diff] [review] bug1512936_editor_broadcaster.patch Review of attachment 9030201 [details] [diff] [review]: ----------------------------------------------------------------- ::: editor/ui/composer/content/editor.xul @@ -110,5 @@ > > <tooltip id="aHTMLTooltip" onpopupshowing="return FillInHTMLTooltipEditor(this);"/> > > - <broadcasterset id="editorBroadcasters"> > - <broadcaster id="Editor:Throbber" busy="false"/> Where is this one set/changed? @@ -111,5 @@ > <tooltip id="aHTMLTooltip" onpopupshowing="return FillInHTMLTooltipEditor(this);"/> > > - <broadcasterset id="editorBroadcasters"> > - <broadcaster id="Editor:Throbber" busy="false"/> > - <broadcaster id="Communicator:WorkMode"/> Seems only used in SM. Can we remove it like this? @@ -115,5 @@ > - <broadcaster id="Communicator:WorkMode"/> > - <broadcaster id="args" value="about:blank"/> > - </broadcasterset> > - > - <broadcasterset id="mainBroadcasterSet"/> Also used in SM.

(In reply to :aceman from comment #2) > Where is this one set/changed? Unused, AFAIKT. That's why I'm removing it. > Seems only used in SM. Can we remove it like this? I think so. They will have to catch up in the unlikely event they do release anything past 60. The alternative of leaving code that will be dead-but-used-in-suite around isn't very enticing.

Yes, but there is difference between visible compilation failure and silent non-working JS code that nobody notices. But OK, having frg on the bug should prevent this change going unnoticed for SM. You copy the busy="false" attribute to the element. Is that one used? Or you try to not touch that for now?

With editor it is a bit hard to tell sometimes if things are used or not. I think it's used at least here: https://searchfox.org/comm-central/source/suite/mailnews/compose/MsgComposeCommands.js#361 and I'm just setting it the way the broadcaster would have set it.

Comment on attachment 9030201 [details] [diff] [review] bug1512936_editor_broadcaster.patch Review of attachment 9030201 [details] [diff] [review]: ----------------------------------------------------------------- Inheriting the 'busy' attribute value via Editor:Throbber and then setting it directly on navigator-throbber instead of the broadcaster seems strange. So the new way seems better.

Thx

https://hg.mozilla.org/comm-central/rev/9a7bbff046a2174fbf177aad9233255e19675389 (PulseBot's gone AWOL, so you get me instead.)