Settings API breaks due to IndexedDB Prefix change

RESOLVED FIXED in mozilla16

Status

()

Core
DOM
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: qdot, Assigned: qdot)

Tracking

Trunk
mozilla16
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
As of http://hg.mozilla.org/mozilla-central/rev/8f307e676f32, nsIDBObjectStore.getAll() is now nsIDBObjectStore.mozGetAll(). This causes API breakage in dom/settings/SettingsManager.js and dom/settings/SettingsService.js, which in turn causes gaia to not be able to load apps.
(Assignee)

Comment 1

5 years ago
Created attachment 637994 [details] [diff] [review]
V1: Settings API breaks due to IndexedDB Prefix change
Attachment #637994 - Flags: review?(khuey)
Comment on attachment 637994 [details] [diff] [review]
V1: Settings API breaks due to IndexedDB Prefix change

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

mumble tests mumble
Attachment #637994 - Flags: review?(khuey) → review+
(Assignee)

Comment 3

5 years ago
http://hg.mozilla.org/mozilla-central/rev/a12b54b2f848
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Component: DOM: Other → DOM
OS: Linux → All
Hardware: x86_64 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
(In reply to Kyle Huey [:khuey] (khuey@mozilla.com) from comment #2)
> Comment on attachment 637994 [details] [diff] [review]
> V1: Settings API breaks due to IndexedDB Prefix change
> 
> Review of attachment 637994 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> mumble tests mumble

We have tests but they are not enabled on desktop. Maybe we should do that.
Yes please!
You need to log in before you can comment on or make changes to this bug.