abfc440394
The original change makes Prefs.deleteBranch() calls properly notify observers that the prefs have been deleted. I believe that makes the deleteBranch() call in this test end up in [[ https://searchfox.org/mozilla-central/rev/bd4d1cd1ca3037e6dc8d4081a4303824880b1b45/toolkit/components/doh/DoHController.sys.mjs#618-620 | this code ]], which calls reset() on the DoHController. This means that as the test was originally written, this reset can be going on while the test code is [[ https://searchfox.org/mozilla-central/rev/bd4d1cd1ca3037e6dc8d4081a4303824880b1b45/toolkit/components/doh/test/browser/head.js#340 | calling _uninit() ]], which is now unnecessary. (and was leading to exceptions since the observer may have already been removed) Instead, the test can just wait until the reset is finished and then continue on. Differential Revision: https://phabricator.services.mozilla.com/D249498