Add utility method to PrivateBrowsingUtils to obtain nsILoadContext from window

RESOLVED FIXED in Firefox 17

Status

()

defect
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: jdm, Assigned: jdm)

Tracking

unspecified
Firefox 19
x86_64
Windows 7
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite ?

Firefox Tracking Flags

(firefox17 fixed, firefox18 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

7 years ago
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.
Assignee

Updated

7 years ago
Blocks: PBnGen

Comment 1

7 years ago
Makes sense.
Assignee

Updated

7 years ago
Attachment #675169 - Flags: review?(ehsan)

Updated

7 years ago
Attachment #675169 - Flags: review?(ehsan) → review+

Comment 3

7 years ago
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: 7 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Target Milestone: --- → Firefox 19

Comment 7

7 years ago
(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.