Closed
Bug 995430
Opened 11 years ago
Closed 10 years ago
send header when user is signed into a children's account
Categories
(Core :: Networking: HTTP, defect)
Tracking
()
RESOLVED
FIXED
mozilla32
People
(Reporter: dougt, Assigned: dougt)
References
Details
Attachments
(1 file, 2 obsolete files)
3.27 KB,
patch
|
mcmanus
:
review+
dougt
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Updated•11 years ago
|
Summary: For 'safe' browsing hit on user accounts that parental protected. → send header when user is signed into a children's account
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8417183 -
Flags: review?(mcmanus)
Assignee | ||
Comment 2•11 years ago
|
||
Didn't know about nsIParentalControlsService. Works in windows 7 & 8.1.
Request without preference set in a 'child' account:
https://www.dropbox.com/s/brqee0imcu74n4s/Screenshot%202014-05-04%2022.07.17.png
I'll create a follow up for a mac implementation.
Comment 3•11 years ago
|
||
Comment on attachment 8417183 [details] [diff] [review]
338b9b10a5b5
Review of attachment 8417183 [details] [diff] [review]:
-----------------------------------------------------------------
This conflicts with the pref in a confusing way. you might just delete the pref?
this doesn't do dynamic config, which is probably ok give that it is windows account driven, just add a comment.
Attachment #8417183 -
Flags: review?(mcmanus)
Assignee | ||
Comment 4•11 years ago
|
||
Attachment #8417183 -
Attachment is obsolete: true
Attachment #8418386 -
Flags: review?(mcmanus)
Comment 5•11 years ago
|
||
Comment on attachment 8418386 [details] [diff] [review]
patch v.2
Review of attachment 8418386 [details] [diff] [review]:
-----------------------------------------------------------------
if network.http.safeHint.enabled is set to any value (either true or false) that will override the parental controls service value. Is that what you want? I can see the argument going either way.
Assignee | ||
Comment 6•11 years ago
|
||
i think this is clearer to what I want. We should send the header: If the user decides to set the preference OR if the user is using a child account.
Attachment #8418386 -
Attachment is obsolete: true
Attachment #8418386 -
Flags: review?(mcmanus)
Attachment #8418788 -
Flags: review?(mcmanus)
Updated•11 years ago
|
Attachment #8418788 -
Flags: review?(mcmanus) → review+
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 8•10 years ago
|
||
Assignee | ||
Updated•10 years ago
|
Attachment #8418788 -
Flags: approval-mozilla-aurora?
Comment 9•10 years ago
|
||
This is a pretty trivial patch and we would like to use it for a time-sensitive campaign.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
Assignee | ||
Updated•10 years ago
|
Attachment #8418788 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 11•10 years ago
|
||
Updated•10 years ago
|
status-firefox31:
--- → fixed
status-firefox32:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•