1924728 - Disabling "Grouped by Sort" while a group header is selected may break the message pane

Status: RESOLVED FIXED

(Thunderbird :: Folder and Message Lists, defect)

Description

[Hartmut Welpmann [:welpy-cw]]

Steps to reproduce:

1. Select a folder that is grouped by sort, all groups collapsed.
2. Select a group header.
3. Change to threaded or unthreaded view.

Actual result:
The message pane doesn't work properly any more, the console shows

Uncaught : Component returned failure code: 0x80550018 [nsIMsgDBView.getURIForViewIndex]
    saveSelection chrome://messenger/content/about3Pane.js:5060
    onDestroyingView chrome://messenger/content/mailCommon.js:1033
    _applyViewChanges resource:///modules/DBViewWrapper.sys.mjs:1530
    set _viewFlags resource:///modules/DBViewWrapper.sys.mjs:1498
    set showThreaded resource:///modules/DBViewWrapper.sys.mjs:1891
    toggleThreaded chrome://messenger/content/about3Pane.js:6708
    init chrome://messenger/content/about3Pane.js:4352
    setColumn chrome://messenger/content/tree-view.mjs:2437
about3Pane.js:5060

Comment 1

[Hartmut Welpmann [:welpy-cw]]

Attachment: Bug 1924728 - Make isGroupedByHeaderAtIndex work properly when the view is already closed. r=#thunderbird-reviewers

Comment 2

[Hartmut Welpmann [:welpy-cw]]

Actually, this doesn't work quite right in 115 and 128 either, because in the case of the STR, the first message of a group is selected after changing the view. But the message pane remains functional.

Comment 3

[Hartmut Welpmann [:welpy-cw]]

Try run

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/9c9bc29de367
Make isGroupedByHeaderAtIndex work properly when the view is already closed. r=mkmelin