Bug 1556261 Comment 74 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

(In reply to Geoff Lankow (:darktrojan) from comment #71)
> Weirdly, it looks like this is what broke mail/base/test/browser/browser_mailContext.js on Windows. I backed it out (on Try) and the error went away.

This is the code causing the test failure: https://searchfox.org/comm-central/rev/1471a7836a2abc68c373e51b739407ce151d84fd/mail/base/content/msgHdrView.js#223-227
Locally, it only happens when using the `--verify` tag, which fails when running `rootFolder.createSubfolder("test", null);` during a second pass.

I'm not sure if the issue is due to the `gFolderDBListener` not de-registering properly, or the fact that we're creating an identically named subfolder and that is making the folder db listener freak out.

I'll keep investigating.
(In reply to Geoff Lankow (:darktrojan) from comment #71)
> Weirdly, it looks like this is what broke mail/base/test/browser/browser_mailContext.js on Windows. I backed it out (on Try) and the error went away.

This is the code causing the test failure: https://searchfox.org/comm-central/rev/1471a7836a2abc68c373e51b739407ce151d84fd/mail/base/content/msgHdrView.js#223-227
Locally, it only happens when using the `--verify` tag, which fails when running `rootFolder.createSubfolder("test", null);` during a second pass.

I'm not sure if the issue is due to the `gFolderDBListener` not de-registering properly, or the fact that we're creating an identically named subfolder and that is making the folder db listener freak out.

I'll keep investigating.

EDIT: Try run to confirm that by removing the `initFolderDBListener` from the folder selection method fixes the test: https://treeherder.mozilla.org/jobs?repo=try-comm-central&revision=49c1b764dafd56ed031bfc63c915a9324e9c08cc

Back to Bug 1556261 Comment 74