1531254 - Port bug 1531119 to Thunderbird - Remove editMenuCommands.inc.xul and make editMenuOverlay.js in charge of constructing the relevant commandset DOM

Status: RESOLVED FIXED

(Thunderbird :: General, task)

Description

[Magnus Melin [:mkmelin]]

+++ This bug was initially created as a clone of Bug #1531119 +++

editMenuCommands.inc.xul is going away and editMenuOverlay.js is instead taking care of the functionality.

Comment 1

[Magnus Melin [:mkmelin]]

Attachment: bug1531254_editMenuCommands.inc_removal.patch

This seems to do it.

Comment 2

[Geoff Lankow (:darktrojan)]

Comment on attachment 9047296 [details] [diff] [review]
bug1531254_editMenuCommands.inc_removal.patch

Review of attachment 9047296 [details] [diff] [review]:
-----------------------------------------------------------------

I'm a little surprised that's the only place, but okay.

Comment 3

[Ryan Lee Sipes]

Is this how xul is replaced by moving the functionality to the relevant .js file?

Comment 4

[Magnus Melin [:mkmelin]]

This is one example yes. Bug 1489447 tracks some other prime examples.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/2ab8c7d1987f
Port bug 1531119 - Remove editMenuCommands.inc.xul and make editMenuOverlay.js in charge of constructing the relevant commandset DOM. r=darktrojan

Comment 6

[Jorg K (CEST = GMT+2)]

It would be good if you could let me know BEFORE it goes bang :-( - I worked out the same patch.

Comment 7

[Magnus Melin [:mkmelin]]

Don't know if you're referring to me or Geoff, but this patch was prepared weeks ago, and also clearly linked from the m-c bug.

Comment 8

[Jorg K (CEST = GMT+2)]

Primarily you, but also Geoff for not talking to me on IRC when the bustage happened and I was looking into where it came from at 12 AM midnight :-(

It may be linked from the M-C bug, but I'm not following that.

So please in the future, adhere to this process:

If you know of incoming bustage and you're pro-actively preparing a fix, please CC me on the bug or better even, NI me on the bug with these words: "Incoming bustage, please watch bug NNNNNN and when it lands, please land the patch here".

This should really go without saying since you know very well that I do most the landings and coordinate these things overall. I can't coordinate them if I don't know about them.

Given that even M-C developers give (some) advanced warning, I expect my C-C colleagues to inform me 100% in advance.

I enclose the bugmail to prove that I got CC'ed to the bug after my comment #6:

https://bugzilla.mozilla.org/show_bug.cgi?id=1531254

Jorg K (GMT+1) <...> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #6 from Jorg K (GMT+1) <[e-mail]> 2019-03-22 00:16:59 CET ---
It would be good if you could let me know **BEFORE** it goes bang  - I
worked out the same patch.

Comment 9

[Geoff Lankow (:darktrojan)]

You weren't on IRC at the time. If you had been I would've left you to it because I had much better things to be doing at the time.

Comment 10

[Jorg K (CEST = GMT+2)]

That's possible. Yesterday I had switched the "auto-connect" off to avoid noise on the channel. That said, there would have been ample opportunity to CC me on the bug.

Comment 11

[Magnus Melin [:mkmelin]]

I'll try to remember to cc you. I'd recommend checking bugs linked from the main one though.

Comment 12

[Jorg K (CEST = GMT+2)]

Which one is the "main" one? Am I to follow all de-XBL activities given that we have a team of five people doing that work?

And with 200+ bugmails a day something can slip through the cracks. It's much easier for you to draw attention to a bustage prevention bug like this one.