1261009 - Remove the Data Store API

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Ben Francis [:benfrancis]]

Data Store access is registered during app installation from a mozApp manifest. As we are removing mozApps we also need to remove the Data Store API.

Comment 1

[Ben Francis [:benfrancis]]

https://developer.mozilla.org/en-US/docs/Mozilla/Firefox_OS/API/Data_Store_API

Comment 2

[Andrew Overholt [:overholt]]

How urgent is this work? Do you know who's going to do it, Ben?

Comment 3

[Andrea Marchesini [:baku]]

If it's about removing this API, I can do it. Do we need to do something before getting rid of its source code?

Comment 4

[Ben Francis [:benfrancis]]

This is part of the B2G Transition Project (meta bug 1252143) to deprecate mozApps and related APIs in favour of new web standards https://wiki.mozilla.org/B2G/Transition_Project

Fabrice has already removed:
* System Messages API (bug 1261013)
* Web Activities API (bug 1261001)

Still to do are:
* Data Store API (this bug)
* Inter-app Communication API (bug 1261011)
* Mobile ID API (bug 1252573) - ready to land

NOTE: These APIs are initially being removed on the *pine* branch which will be merged back into mozilla-central once the TV team have branched Firefox OS 2.6.

There are no further MoCo resources currently assigned to this project after May and we're doing as much as we can to clean up Gecko in that time, so any help is appreciated.

Comment 5

[Andrew Overholt [:overholt]]

("fixlater" just means "ideally within the next few months")

Comment 6

[Andrea Marchesini [:baku]]

Attachment: dataStore.patch

Maybe I forgot something, but so far it compiles. I'll post the treeherder result when ready.

Comment 14

[[:fabrice] Fabrice Desré]

Comment on attachment 8746410 [details] [diff] [review]
dataStore.patch

Review of attachment 8746410 [details] [diff] [review]:
-----------------------------------------------------------------

thanks for taking that!

Comment 15

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/83b0db55bdc9

Comment 16

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/83b0db55bdc9

Comment 17

[Chris Mills [:cmills]]

I've moved all the DataStore-related pages to the archive, and added a note to the Firefox 49 release ntoes to say it has been removed:

https://developer.mozilla.org/en-US/Firefox/Releases/49#Others