Closed
Bug 776764
Opened 12 years ago
Closed 12 years ago
Allow permissionDB parent directory to not be the default profile directory
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
People
(Reporter: gwagner, Assigned: gwagner)
References
Details
Attachments
(1 file, 2 obsolete files)
2.72 KB,
patch
|
fabrice
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → anygregor
Assignee | ||
Comment 1•12 years ago
|
||
Assignee | ||
Comment 2•12 years ago
|
||
Attachment #645131 -
Attachment is obsolete: true
Assignee | ||
Comment 3•12 years ago
|
||
Also update the directoryProvider
Attachment #645154 -
Attachment is obsolete: true
Attachment #645159 -
Flags: review?(bent.mozilla)
Assignee | ||
Updated•12 years ago
|
blocking-basecamp: --- → ?
Comment 4•12 years ago
|
||
Comment on attachment 645159 [details] [diff] [review] patch Review of attachment 645159 [details] [diff] [review]: ----------------------------------------------------------------- r=me with the nits addressed. ::: b2g/components/DirectoryProvider.js @@ +21,5 @@ > QueryInterface: XPCOMUtils.generateQI([Ci.nsIDirectoryServiceProvider]), > > getFile: function dp_getFile(prop, persistent) { > #ifdef MOZ_WIDGET_GONK > + let localProps = ["cachePDir", "webappsDir", "PrefD", "indexedDBPDir", "permisionDBDir"]; Ditto: 'permissionDBPDir ::: extensions/cookie/nsPermissionManager.cpp @@ +306,5 @@ > + nsresult rv = NS_GetSpecialDirectory(NS_APP_PERMISSION_PARENT_DIR, getter_AddRefs(permissionsFile)); > + if (NS_FAILED(rv)) { > + rv = NS_GetSpecialDirectory(NS_APP_USER_PROFILE_50_DIR, getter_AddRefs(permissionsFile)); > + } > + NS_ENSURE_SUCCESS(rv, nsnull); why nsnull and not NS_ENSURE_SUCCESS(rv, NS_ERROR_UNEXPECTED) ? ::: xpcom/io/nsAppDirectoryServiceDefs.h @@ +88,5 @@ > #define NS_APP_STORAGE_50_FILE "UStor" // sqlite database used as mozStorage profile db > > #define NS_APP_INDEXEDDB_PARENT_DIR "indexedDBPDir" > + > +#define NS_APP_PERMISSION_PARENT_DIR "permisionDBDir" I would use permissionDBPDir ('P' for parent, and also you have a typo with a missing 's')
Attachment #645159 -
Flags: review+
Assignee | ||
Comment 5•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/d9419716db6f
Updated•12 years ago
|
blocking-basecamp: ? → +
Assignee | ||
Updated•12 years ago
|
Attachment #645159 -
Flags: review?(bent.mozilla)
Comment 6•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/d9419716db6f
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
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
•