Closed
Bug 971703
Opened 10 years ago
Closed 10 years ago
Lazy load SSLExceptions
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 30
People
(Reporter: mfinkle, Assigned: mfinkle)
References
Details
Attachments
(1 file, 1 obsolete file)
5.43 KB,
patch
|
wesj
:
review+
|
Details | Diff | Splinter Review |
We currently load it in browser.xul (ouch!) but we only ever need it for times when a user wants to add an exception for a networking cert issue. This patch converts the code to a JSM since it has no browser.js or window dependencies. The patch also lazy loads the code.
Attachment #8374813 -
Flags: review?(wjohnston)
Assignee | ||
Comment 2•10 years ago
|
||
I missed an XMLHttpRequest() usage. I converted it to use non-window safe creation. Tested on https://jal.com and it works fine. No errors in the code.
Assignee: nobody → mark.finkle
Attachment #8374813 -
Attachment is obsolete: true
Attachment #8374813 -
Flags: review?(wjohnston)
Attachment #8374835 -
Flags: review?(wjohnston)
Updated•10 years ago
|
Attachment #8374835 -
Flags: review?(wjohnston) → review+
Assignee | ||
Comment 3•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/725892611403
https://hg.mozilla.org/mozilla-central/rev/725892611403
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 30
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•