Use openUnsharedDatabase in localStorage SQLite

RESOLVED FIXED in mozilla12

Status

()

defect
RESOLVED FIXED
8 years ago
3 months ago

People

(Reporter: mak, Assigned: vladan)

Tracking

({perf})

unspecified
mozilla12
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [Snappy:P1])

Attachments

(1 attachment, 1 obsolete attachment)

Reporter

Description

8 years ago
This call is cheaper for our case, since SQLite doesn't have to check the cache before using it.
Reporter

Comment 1

8 years ago
this is basically part of vladan's patch in bug 704933, so assigning to him, I'll later attach the partial patch.
Assignee: nobody → vdjeric
Reporter

Updated

8 years ago
Blocks: 711972
Assignee

Comment 2

8 years ago
2-line patch from bug 704933
Attachment #582940 - Flags: review?(mak77)
Assignee

Comment 3

8 years ago
Attachment #582940 - Attachment is obsolete: true
Attachment #582940 - Flags: review?(mak77)
Attachment #582947 - Flags: review?(mak77)
Reporter

Comment 4

8 years ago
Comment on attachment 582947 [details] [diff] [review]
Open DOM storage DB as unshared

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

I'll leave feedback on this patch, and let Honza give review, so he is aware of the change and I'm sure we are on the same boat (he'll likely have to review my next patches on top of this) :)
Btw, there should be no risk involved in doing this change.
Attachment #582947 - Flags: review?(mak77)
Attachment #582947 - Flags: review?(honzab.moz)
Attachment #582947 - Flags: feedback+

Updated

8 years ago
Whiteboard: [Snappy:P1]
Reporter

Updated

8 years ago
Blocks: 714964
Comment on attachment 582947 [details] [diff] [review]
Open DOM storage DB as unshared

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

r=honzab
Attachment #582947 - Flags: review?(honzab.moz) → review+
https://hg.mozilla.org/mozilla-central/rev/afa049904ea6
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.