Bug 1649601 Comment 1 Edit History

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

Original comment by
on 
We need to remove the OS.File import in SearchService.jsm: https://searchfox.org/mozilla-central/rev/9c72508fcf2bba709a5b5b9eae9da35e0c707baa/toolkit/components/search/SearchService.jsm#19

We also need to update the usage of OS.File here: https://searchfox.org/mozilla-central/rev/9c72508fcf2bba709a5b5b9eae9da35e0c707baa/toolkit/components/search/SearchService.jsm#2610

This can be updated too use `AsyncShutdown.profileBeforeChange.addBlocker` instead, which we will also have to import.
Revision 1 by
on 
We need to remove the OS.File import in SearchService.jsm: https://searchfox.org/mozilla-central/rev/9c72508fcf2bba709a5b5b9eae9da35e0c707baa/toolkit/components/search/SearchService.jsm#19

We also need to update the usage of OS.File here: https://searchfox.org/mozilla-central/rev/9c72508fcf2bba709a5b5b9eae9da35e0c707baa/toolkit/components/search/SearchService.jsm#2610

This can be updated to instead use `AsyncShutdown.profileBeforeChange.addBlocker` instead, which we will also have to import.
Revision 2 by
on 
We need to remove the OS.File import in SearchService.jsm: https://searchfox.org/mozilla-central/rev/9c72508fcf2bba709a5b5b9eae9da35e0c707baa/toolkit/components/search/SearchService.jsm#19

We also need to update the usage of OS.File here: https://searchfox.org/mozilla-central/rev/9c72508fcf2bba709a5b5b9eae9da35e0c707baa/toolkit/components/search/SearchService.jsm#2610

This can be updated to instead use `AsyncShutdown.profileBeforeChange.addBlocker` instead, and we will need to import that module.

Back to Bug 1649601 Comment 1