Closed Bug 1530980 Opened 2 years ago Closed 2 years ago

Send custom "marionette-startup-requested" observer notification to start Marionette in Thunderbird

Categories

(Thunderbird :: General, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 67.0

People

(Reporter: whimboo, Assigned: darktrojan)

Details

Attachments

(3 files)

As of now Marionette listens for a couple of startup observer notifications, and has to wait until the startup recorder has finished recording startup scripts:

https://searchfox.org/mozilla-central/rev/dbddac86aadf1d4871fb350bbe66db43728a9f81/testing/marionette/components/marionette.js#449-456

For Fennec and GeckoView we recently got the marionette-startup-requested observer notification added (see bug 1496773), which allows us to explicitly specify at which stage during startup Marionette has to be enabled.

The same behavior we would like to see for Thunderbird.

Geoff, would you mind to implement that?

Flags: needinfo?(geoff)
Summary: Send custom "marionette-startup-requested" observer notification to start Marionette → Send custom "marionette-startup-requested" observer notification to start Marionette in Thunderbird

Let me just check I've understood correctly. You want something in Thunderbird to send marionette-startup-requested so that you don't have to listen for mail-startup-done?

Flags: needinfo?(geoff)

That is exactly correct. In that case the product can define and change the time during startup where to initialize Marionette.

Assignee: nobody → geoff
Status: NEW → ASSIGNED
Attachment #9049398 - Flags: review?(mkmelin+mozilla)
Attachment #9049398 - Flags: review?(mkmelin+mozilla) → review+
Keywords: checkin-needed
Version: 52 Branch → unspecified
Keywords: checkin-needed
Attached file log.log

This is what happens if I start without changing the m-c code.

So yes, we should land both patches close together if possible to allow an initialization for TB.

I'm about to land this on inbound, it'll need the c-c patch to land when it gets to central.

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ec939d4f81dc
Stop listening for mail-startup-done to start Marionette in Thunderbird; r=whimboo

Keywords: checkin-needed
Keywords: leave-open

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/1f0d963096cb
Send custom "marionette-startup-requested" observer notification to start Marionette in Thunderbird; r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 67.0

Note to self: next time, set checkin-needed after checking in the first part.

Thanks Geoff! Good to see this extra registered observer listener gone.

You need to log in before you can comment on or make changes to this bug.