Closed
Bug 459822
Opened 16 years ago
Closed 16 years ago
OfflineAppAllowed should not be on the IO service
Categories
(Core :: Networking, defect, P1)
Core
Networking
Tracking
()
RESOLVED
FIXED
People
(Reporter: Biesinger, Assigned: dcamp)
References
Details
(Keywords: fixed1.9.1)
Attachments
(1 file)
19.54 KB,
patch
|
Biesinger
:
review+
Biesinger
:
superreview+
|
Details | Diff | Splinter Review |
http://hg.mozilla.org/mozilla-central/rev/3e41c4b388c3 (for bug 442806) added OfflineAppAllowed to the IO service. That change was never reviewed. Per discussion with bz this function should probably be somewhere in Docshell instead. Also, the IDL method should start with a lowercase letter, and the implementation shouldn't use NS_ENSURE_SUCCESS when getting a preference value.
Flags: blocking1.9.1?
Comment 1•16 years ago
|
||
To be precise, the new implementation was never reviewed. It makes some changes from the original implementation that seem to me to be wrong (e.g. returning error on pref not being set as opposed to just treating it as false).
Assignee | ||
Comment 2•16 years ago
|
||
Patch moves offlineAppAllowed to nsIOfflineCacheUpdateService, probably a better dumping ground for offline-related stuff than the docshell. While I was at it, changed the primary offlineAppAllowed() to take a principal rather than a URI. There's an offlineAppAllowedForURI() variant, currently used by the docshell.
Attachment #343139 -
Flags: superreview?(cbiesinger)
Attachment #343139 -
Flags: review?(cbiesinger)
Reporter | ||
Comment 3•16 years ago
|
||
Comment on attachment 343139 [details] [diff] [review] v1 looks good. is it too late to fix the inconsistency in the pref name? (dashes vs underscores)
Attachment #343139 -
Flags: superreview?(cbiesinger)
Attachment #343139 -
Flags: superreview+
Attachment #343139 -
Flags: review?(cbiesinger)
Attachment #343139 -
Flags: review+
Assignee | ||
Comment 4•16 years ago
|
||
(In reply to comment #3) > (From update of attachment 343139 [details] [diff] [review]) > looks good. is it too late to fix the inconsistency in the pref name? (dashes > vs underscores) It's an undocumented pref that isn't included by default in any of the prefs files, so I think it should be fine to change.
Assignee | ||
Comment 5•16 years ago
|
||
Landed as http://hg.mozilla.org/mozilla-central/rev/d50b1e07f56e
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Comment 6•15 years ago
|
||
This landed prior to 1.9.1 branching.
You need to log in
before you can comment on or make changes to this bug.
Description
•