980379 - AppsUtils.loadJSONAsync() should allow passing options to OS.File calls

Status: RESOLVED WONTFIX

(Core Graveyard :: DOM: Apps, defect)

Description

[Georg Fritzsche [:gfritzsche]]

AppsUtils.loadJSONAsync() should allow passing options to OS.File calls so e.g. compression can be used.

Comment 1

[Georg Fritzsche [:gfritzsche]]

Attachment: Simplify & allow passing options to OS.File.read()

fabrice, "hg log" suggests you as a reviewer.
This allows me to use |loadJSONAsync(path, {compression: "lz4"})|.

Comment 2

[Marco Castelluccio [:marco]]

We'll stop using OS.File for now because of some problems with b2g (see bug 961317 for details).

Comment 3

[Georg Fritzsche [:gfritzsche]]

Thanks for the heads-up. As it seems this is just B2G-test-specific for now i might just roll my own version if i land before bug 961317 is fixed.

Comment 4

[Marco Castelluccio [:marco]]

(In reply to Georg Fritzsche [:gfritzsche] from comment #3)
> Thanks for the heads-up. As it seems this is just B2G-test-specific for now
> i might just roll my own version if i land before bug 961317 is fixed.

I think in bug 981085 we're going to remove this function altogether.

Comment 5

[Georg Fritzsche [:gfritzsche]]

Alright, then let's not spend time on this.