1228383 - Centralize TabStateFlusher.flush usage when closing tabs

Status: NEW

(Firefox :: Session Restore, defect)

Description

[Mike Conley (:mconley) (:⚙️)]

TabStateFlusher.flush is used to request an async flush of all messages from a browser. When the flushed messages come up to the parent, we handle them inside the SessionStore:update message handler in SessionStore.jsm.

It might be better to centralize the logic a bit more, and have the handling logic occur when the TabStateFlusher Promise resolves, much like we do with TabStateFlusher.flushWindow in the onClose handler in SessionStore.

See https://bugzilla.mozilla.org/show_bug.cgi?id=1171708#c52 for context.

Comment 1

[Mike Conley (:mconley) (:⚙️)]

On second thought, this might not be a good first bug.