Closed Bug 1508217 Opened 11 months ago Closed 11 months ago

remove broadcasters from mail/base

Categories

(Thunderbird :: General, task)

task
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 65.0

People

(Reporter: mkmelin, Assigned: mkmelin)

References

Details

Attachments

(1 file, 1 obsolete file)

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

Remove <broadcaster> from mail/base
Attachment #9026004 - Flags: review?(jorgk)
Comment on attachment 9026004 [details] [diff] [review]
bug1508217_de_broadcaster_otherActionsOpenIn.patch

Please rebase. A try run together with bug 1508209 would be great.
"try: -b do -p macosx64,linux64 -u all" should be sufficient.
Attachment #9026004 - Flags: review?(jorgk)
Just |hg pull -u| and you'll notice it ;-) - Plus you have two test failures, I'm not sure which of the three patches caused them:
TEST-UNEXPECTED-FAIL | /builds/worker/workspace/build/tests/mozmill/message-header/test-header-toolbar.js | test-header-toolbar.js::test_customize_header_toolbar_dialog_style
TEST-UNEXPECTED-FAIL | /builds/worker/workspace/build/tests/mozmill/message-header/test-header-toolbar.js | test-header-toolbar.js::test_customize_header_toolbar_change_button_style
Comment on attachment 9026363 [details] [diff] [review]
bug1508217_de_broadcaster_otherActionsOpenIn.patch

Looks OK to me (I started with and easy one).
Attachment #9026363 - Flags: review?(jorgk) → review+
Keywords: checkin-needed
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/72a2e97895e9
remove broadcasters from mail/base. r=jorgk
Status: ASSIGNED → RESOLVED
Closed: 11 months ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 65.0
Type: enhancement → task
You need to log in before you can comment on or make changes to this bug.