Closed Bug 799314 Opened 8 years ago Closed 8 years ago

Add utility method to PrivateBrowsingUtils to obtain nsILoadContext from window

Categories

(Firefox :: Private Browsing, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 19
Tracking Status
firefox17 --- fixed
firefox18 --- fixed

People

(Reporter: jdm, Assigned: jdm)

References

Details

Attachments

(1 file)

Writing up docs for the bug 722872 fiasco, I realized that it sucks to be telling developers how to obtain an nsILoadContext. We should add a GetPrivacyContext method to PrivateBrowsingUtils and get it into FF17.
Blocks: PBnGen
Makes sense.
Attachment #675169 - Flags: review?(ehsan)
Attachment #675169 - Flags: review?(ehsan) → review+
Comment on attachment 675169 [details] [diff] [review]
Add utility method to get a privacy context from a window.

This is as low-risk as it can get, and it would be really really helpful for add-on developers if we can take this for Firefox 17.
Attachment #675169 - Flags: approval-mozilla-beta?
Attachment #675169 - Flags: approval-mozilla-aurora?
Comment on attachment 675169 [details] [diff] [review]
Add utility method to get a privacy context from a window.

given the very low risk, i'll pre-approve this even though it's on inbound for now - please go ahead with uplift once it's been merged to central.
Attachment #675169 - Flags: approval-mozilla-beta?
Attachment #675169 - Flags: approval-mozilla-beta+
Attachment #675169 - Flags: approval-mozilla-aurora?
Attachment #675169 - Flags: approval-mozilla-aurora+
https://hg.mozilla.org/mozilla-central/rev/fea03e42fb87

Should this have a test?
Assignee: nobody → josh
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Target Milestone: --- → Firefox 19
(In reply to Ryan VanderMeulen from comment #6)
> https://hg.mozilla.org/mozilla-central/rev/fea03e42fb87
> 
> Should this have a test?

isWindowPrivate is tested in a bunch of places, so it does, implicitly.
You need to log in before you can comment on or make changes to this bug.