880741 - add indexedDB to frameworker API

Status: RESOLVED FIXED

(Firefox Graveyard :: SocialAPI, defect)

Description

[:Gavin Sharp [email: [e-mail]]]

IndexedDB is "better" than localStorage and is likely to be added to SharedWorkers, so we should let social providers use it in their workers.

Comment 1

[Shane Caraveo (:mixedpuppy)]

Attachment: indexeddb patch

Comment 2

[:Gavin Sharp [email: [e-mail]]]

Comment on attachment 759848 [details] [diff] [review]
indexeddb patch

thanks!

Comment 3

[Shane Caraveo (:mixedpuppy)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/7e181e566679

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/7e181e566679

Comment 5

[Shane Caraveo (:mixedpuppy)]

Comment on attachment 759848 [details] [diff] [review]
indexeddb patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 879658
User impact if declined: no user impact. the earlier we can get indexeddb into use, the earlier we can at least mark localstorage as deprecated for socialapi workers (if not blocked)
Testing completed (on m-c, etc.): on m-c
Risk to taking this patch (and alternatives if risky): low
String or IDL/UUID changes made by this patch: none

Comment 6

[Alex Keybl [:akeybl]]

Comment on attachment 759848 [details] [diff] [review]
indexeddb patch

Seems low risk, we won't track but will approve.

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/8bba8f847b9f

Comment 8

[u279076]

Deprioritizing for verification since it looks like this landed with tests in-testsuite. Please drop the [qa-] tag and add the verifyme keyword if there's testing required by QA here.