Closed
Bug 769785
Opened 12 years ago
Closed 12 years ago
Settings API breaks due to IndexedDB Prefix change
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla16
People
(Reporter: qdot, Assigned: qdot)
Details
Attachments
(1 file)
1.28 KB,
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
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•12 years ago
|
||
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•12 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/a12b54b2f848
Status: NEW → RESOLVED
Closed: 12 years ago
Component: DOM: Other → DOM
OS: Linux → All
Hardware: x86_64 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
Comment 4•12 years ago
|
||
(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!
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•