Open Bug 876430 Opened 10 years ago Updated 9 years ago

Update Dataman for changes in DOMStorage and localStorage

Categories

(SeaMonkey :: Passwords & Permissions, defect)

Product:
SeaMonkey
Component:
Passwords & Permissions
Platform:
All
Windows 7
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

People

(Reporter: philip.chee, Unassigned)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

Patch v1.1 WIP
5.60 KB, patch
kairo
: feedback+
Details | Diff | Splinter Review 
Relevant bugs:
* Bug 600307 - Rewrite and cleanup DOMStorage code
  (localStorage and sessionStorage implementation overhaul)
* Bug 767944 - Implement a manager for centralized quota and storage handling
Attached patch Patch v1.0 WIP (obsolete) — — Splinter Review 
This is what I have so far. IndexedDB and AppCache show non zero bytes. However localstorage is always 0 bytes so I'm probably doing something wrong.
Attachment #754502 - Flags: feedback?(kairo)
Attached patch Patch v1.1 WIP — — Splinter Review 
Darn, wrong author field in the git patch again, fixes.
Attachment #754502 - Attachment is obsolete: true
Attachment #754502 - Flags: feedback?(kairo)
Attachment #754798 - Flags: feedback?(kairo)
Attachment #754798 - Attachment is patch: true
Attachment #754798 - Attachment mime type: text/x-patch → text/plain
Comment on attachment 754798 [details] [diff] [review]
Patch v1.1 WIP

Sorry, I completely don't understand why this has to be done this way, but I realize I can't get myself to put enough priority on finding out. So, whatever. Better something works than nothing.
Attachment #754798 - Flags: feedback?(kairo) → feedback+
Blocks: 903865
So, either of the bugs mentioned in comment #0 broke the bug 588415 implementation?
When visiting a site using localStorage I don't see anything showing up in the Data Manager.
You need to log in before you can comment on or make changes to this bug.