STR

1. composition with contacts side bar, ensure "All Address Books" is shown
2. select a contact from "All Address Books" view (contact originates from "Personal AB")
3. press DEL
4. repeat same STR for Mailing list from "All Address Books" view (mailing list originates from "Personal AB")

This works in current release version 68.5.0 (32-Bit),
but fails on Trunk 75.0a1 (2020-02-27) (64-bit), so it's a regression.

Alice, would you like to find the regression window? No obligations, just asking if.

Actual result

• deleting contact from "All ABs" view does nothing, 2 errors in console:

TypeError: can't access property "isMailList", selectedDir is null abCommon.js:429:11
    AbDelete chrome://messenger/content/addressbook/abCommon.js:429
    doCommand chrome://messenger/content/addressbook/abResultsPane.js:519
    goDoCommand chrome://global/content/globalOverlay.js:101
    oncommand chrome://messenger/content/messengercompose/messengercompose.xhtml:1

An error occurred executing the cmd_delete command: [Exception... "[JavaScript Error: "can't access property "isMailList", selectedDir is null" {file: "chrome://messenger/content/addressbook/abCommon.js" line: 429}]'[JavaScript Error: "can't access property "isMailList", selectedDir is null" {file: "chrome://messenger/content/addressbook/abCommon.js" line: 429}]' when calling method: [nsIController::doCommand]"  nsresult: "0x80570021 (NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS)"  location: "JS frame :: chrome://global/content/globalOverlay.js :: goDoCommand :: line 101"  data: yes] globalOverlay.js:104
    goDoCommand chrome://global/content/globalOverlay.js:104
    oncommand chrome://messenger/content/messengercompose/messengercompose.xhtml:1

• deleting a mailing list from "All Abs" view shows confirmation, but confirming does nothing, 2 Errors in console:

TypeError: can't access property "URI", selectedDir is null abCommon.js:495:7

An error occurred executing the cmd_delete command: [Exception... "[JavaScript Error: "can't access property "URI", selectedDir is null" {file: "chrome://messenger/content/addressbook/abCommon.js" line: 495}]'[JavaScript Error: "can't access property "URI", selectedDir is null" {file: "chrome://messenger/content/addressbook/abCommon.js" line: 495}]' when calling method: [nsIController::doCommand]"  nsresult: "0x80570021 (NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS)"  location: "JS frame :: chrome://global/content/globalOverlay.js :: goDoCommand :: line 101"  data: yes] globalOverlay.js:104
    goDoCommand chrome://global/content/globalOverlay.js:104
    oncommand chrome://messenger/content/addressbook/abContactsPanel.xhtml:1

Expected:

• delete selected entries (this bug)
• select and scroll into view next entry if existing, otherwise last entry (and focus something useful when there's no entry left)

REGRESSION WINDOW:
https://hg.mozilla.org/comm-central/pushloghtml?fromchange=306ac4b77c679440f825d487a9147544591db953&tochange=a04956310d2fe4df06c9ba41cb7bb239a09341d4
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=5bfe22b4bd0d64b9d105b66b3ed0fcee10d70e93&tochange=5bfe22b4bd0d64b9d105b66b3ed0fcee10d70e93

(In reply to Alice0775 White from comment #1)

REGRESSION WINDOW:
https://hg.mozilla.org/comm-central/pushloghtml?fromchange=306ac4b77c679440f825d487a9147544591db953&tochange=a04956310d2fe4df06c9ba41cb7bb239a09341d4
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=5bfe22b4bd0d64b9d105b66b3ed0fcee10d70e93&tochange=5bfe22b4bd0d64b9d105b66b3ed0fcee10d70e93

Thank you very much Alice, and so fast! You're the best! :-)

Thanks Geoff for picking this up for a fast and easy fix. Maybe this should have a test? I recall that this is not the first time that deletion fails in one corner or another (more so for the special animal of "All address books")...

You're right it needs a test. Nothing's a simple as just writing a test though, especially when it comes to figuring out why the test runs in one setup but not others.

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/7d7596b594c3
Test deletion of contacts from the composition sidebar. r=mkmelin