JSONFile.jsm, a new module to help with serializing JSON to disk

David Teller dteller at mozilla.com
Wed Oct 12 09:18:11 UTC 2016


I understand. But since removing I/O from the main thread used to be my
responsibility, and since I have measurements that show just how bad
main thread I/O can be, I'm still pretty touchy about this :)

Filed bugs.

Cheers,
 David

On 12/10/16 05:03, Matthew N. wrote:
> Hi David,
> 
> Can you file Toolkit::General bugs blocking 1304322 for the first two
> issues please? Note that this module is just a refactoring of
> LoginStore.jsm from a few years ago to make it more generic so these
> issues already existed before.
> 
> Regarding ensureDataReady, code review and developer judgement can be
> used to judge whether that method should be used by an individual
> consumer. Perhaps we could have warnings in the JSDoc comment to make it
> more clear that it should be avoided but in the original code
> (LoginStore.jsm) it was needed.
> 
> Thanks,
> Matthew
> 



More information about the firefox-dev mailing list