Closed
Bug 1064212
Opened 10 years ago
Closed 10 years ago
Add fromWindow() to LoadContextInfo.jsm
Categories
(Toolkit :: General, defect)
Toolkit
General
Tracking
()
RESOLVED
FIXED
mozilla35
People
(Reporter: mayhemer, Assigned: mayhemer)
Details
Attachments
(1 file, 1 obsolete file)
1.71 KB,
patch
|
Dolske
:
review+
|
Details | Diff | Splinter Review |
Since usually consumers of the new cache have |window| in hands, there should be a shortcut to get the load context info directly from it rather then to first get nsILoadContext and use that to get nsILoadContextInfo.
Assignee | ||
Comment 1•10 years ago
|
||
- added short cut to get nsILoadContextInfo (derived interface used in the networking code when nsILoadContext cannot be provided or is not enough) directly from window objects
Attachment #8485662 -
Flags: review?(dolske)
Assignee | ||
Comment 2•10 years ago
|
||
- added shortcut to get nsILoadContextInfo (derived interface used in the networking code when nsILoadContext cannot be provided or is not enough) directly from window objects - fixed typo in LoadContextInfo.fromLoadContext: s/isPrivate/usePrivateBrowsing/, _loadContext is nsILoadContext
Attachment #8485662 -
Attachment is obsolete: true
Attachment #8485662 -
Flags: review?(dolske)
Attachment #8485667 -
Flags: review?(dolske)
Updated•10 years ago
|
Attachment #8485667 -
Flags: review?(dolske) → review+
Assignee | ||
Comment 3•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/e31f6893fd19
https://hg.mozilla.org/mozilla-central/rev/e31f6893fd19
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla35
You need to log in
before you can comment on or make changes to this bug.
Description
•