Closed Bug 1131281 Opened 10 years ago Closed 10 years ago

Bookmarks sqlite table

Categories

(Firefox for iOS :: Data Storage, defect)

All
iOS 8
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: wesj, Assigned: wesj)

References

Details

Attachments

(1 file)

47 bytes, text/x-github-pull-request
rnewman
: review+
nalexander
: feedback+
Details | Review
We've got some bookmarks code already in place on iOS, but we should have a locally stored database using sqlite as well. This is for that.
Blocks: 1127040
Attached file Pull request
This implements a sqlite datastore for bookmarks. I tried to keep from changing the existing code as much as I could. I don't love the existing code, but it was just to easy for this code to boil the ocean. I did change its ID field to a guid one, since most of our other data sources look that way. I then deleted all the old unused code.
Attachment #8561667 - Flags: review?(rnewman)
Assignee: nobody → wjohnston
Status: NEW → ASSIGNED
OS: Mac OS X → iOS 8
Hardware: x86 → All
Attachment #8561667 - Flags: review?(rnewman)
Attachment #8561667 - Flags: review+
Attachment #8561667 - Flags: feedback?(nalexander)
Comment on attachment 8561667 [details] [review] Pull request I have some questions and concerns but nothing major, and no need to block on me getting my work reviewed. Thanks, y'all!
Attachment #8561667 - Flags: feedback?(nalexander) → feedback+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: