Closed
Bug 1546913
Opened 5 years ago
Closed 5 years ago
Remove Callsides of nsIChannel::GetLoadinfo()
Categories
(Core :: DOM: Security, enhancement, P2)
Tracking
()
RESOLVED
FIXED
mozilla68
| Tracking | Status | |
|---|---|---|
| firefox68 | --- | fixed |
People
(Reporter: sstreich, Assigned: sstreich)
Details
(Whiteboard: [domsecurity-active])
Attachments
(1 file)
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.14; rv:66.0) Gecko/20100101 Firefox/66.0
|
Assignee | |
Comment 1•5 years ago
|
||
Currently we're doing this:
nsCOMPtr<nsILoadInfo> loadInfo;
rv = channel->GetLoadInfo(getter_AddRefs(loadInfo));
NS_ENSURE_SUCCESS(rv, rv);
NS_ENSURE_STATE(loadInfo);
while at the same time, we could simplify this with
nsCOMPtr<nsILoadInfo> loadInfo = channel->LoadInfo();
This would also be a preperation to resolve Bug 152955
Type: defect → enhancement
|
|
Assignee | |
Comment 2•5 years ago
|
||
Sorry i did mean Bug 1529554
|
|
Assignee | |
Comment 3•5 years ago
|
||
|
||
Updated•5 years ago
|
Assignee: nobody → streich.mobile
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Priority: -- → P2
Whiteboard: [domsecurity-active]
|
|
Assignee | |
Updated•5 years ago
|
Keywords: checkin-needed
Pushed by opoprus@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/510c23dc30c6
Remove GetLoadInfo callsites r=ckerschb
Keywords: checkin-needed
|
||
Comment 5•5 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox68:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in
before you can comment on or make changes to this bug.
Description
•