Closed Bug 328140 Opened 18 years ago Closed 18 years ago

Integrate 0-filling patch into storage system

Categories

(Toolkit :: Storage, defect)

x86
All
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: brettw, Assigned: brettw)

Details

(Keywords: fixed1.8.1, privacy)

Attachments

(1 file)

See: http://www.sqlite.org/cvstrac/chngview?cn=3107

This will ensure that deleted rows are not exposed in the history file, potentially voilating user expectations after clearing history. We can then remove vacuuming, which is slow.
Attached patch PatchSplinter Review
This makes deletes from the DB actually clear the information from the file. This way we don't have to do the VERY expensive VACUUM operation to ensure that people's data gets deleted when they clear their history (or it expires).
Attachment #212806 - Flags: first-review?(vladimir)
Comment on attachment 212806 [details] [diff] [review]
Patch

r=me
Attachment #212806 - Flags: first-review?(vladimir) → first-review+
On branch and trunk.
Status: NEW → RESOLVED
Closed: 18 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Keywords: privacy
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: