Open Bug 1254928 Opened 8 years ago Updated 17 days ago

Remove or pref off non-standard IDB extensions

Categories

(Core :: Storage: IndexedDB, task, P3)

task

Tracking

()

People

(Reporter: Ms2ger, Unassigned)

References

(Depends on 2 open bugs, Blocks 1 open bug)

Details

(Keywords: addon-compat, dev-doc-needed, site-compat, Whiteboard: btpp-followup-2016-04-07)

I found:

* createMutableFile
* mozCreateFileHandle
* IDBFileRequest
* IDBFileHandle

and there's probably a bunch more I didn't find. Nobody else seems interested in supporting them, so we should not ship them in release.
* IDBMutableFile
I know at least one site that uses this stuff, www.peer5.com
Guy Paskar should know more.

The plan is to extend current MutableFile implementation to support the new FileSystem API and once that ships and sites adjust to it, we'll remove createMutableFile, etc. from IndexedDB.
(In reply to Jan Varga [:janv] from comment #2)
> I know at least one site that uses this stuff, www.peer5.com
> Guy Paskar should know more.

Does peer5.com do something different for other browsers?
 
> The plan is to extend current MutableFile implementation to support the new
> FileSystem API and once that ships and sites adjust to it, we'll remove
> createMutableFile, etc. from IndexedDB.

It's not clear we're going to implement the FileSystem API in the short term.
Flags: needinfo?(jvarga)
Whiteboard: btpp-followup-2016-04-07
(In reply to Andrew Overholt [:overholt] from comment #3)
> (In reply to Jan Varga [:janv] from comment #2)
> > I know at least one site that uses this stuff, www.peer5.com
> > Guy Paskar should know more.
> 
> Does peer5.com do something different for other browsers?

Yes, I think they use google's filesystem api.

>  
> > The plan is to extend current MutableFile implementation to support the new
> > FileSystem API and once that ships and sites adjust to it, we'll remove
> > createMutableFile, etc. from IndexedDB.
> 
> It's not clear we're going to implement the FileSystem API in the short term.

Yeah and the current FileHandle in IndexedDB doesn't do any big harm, so I think we should just keep it for now.
Flags: needinfo?(jvarga)
I think we should add use counter telemetry to measure how much these are used in the wild. Will file a bug.
Depends on: 1271457
See Also: → 1361917
Is the IDB MutableFile support going to be available to WebExtensions? Migrating an addon to a WebExtension, shall I consider these interfaces to be stable and available or deprecated?
The features in this bug are not in any web standard and are not implemented in any other browser. I would generally assume that such features will be removed eventually.
Priority: -- → P3
Type: defect → task
Depends on: 1354500
Depends on: 1577227
Depends on: 1500343

I'd like us to remove https://developer.mozilla.org/en-US/docs/Web/API/IndexedDB_API/Using_IndexedDB#locale-aware_sorting , which according to MDN is behind a flag anyway. Reason: Removing the dependency on collation key objects so that ICU4X collator MVP doesn't need to support collation key objects.

Depends on: 1730706
Severity: normal → S3
Flags: in-testsuite+
Flags: behind-pref+
Flags: in-testsuite+
Flags: behind-pref+
Depends on: 1878907
You need to log in before you can comment on or make changes to this bug.