Make JS callers of ios.newChannel call ios.newChannel2 in b2g/

RESOLVED FIXED in mozilla37

Status

()

Core
DOM: Security
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: ckerschb, Assigned: ckerschb)

Tracking

unspecified
mozilla37
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Comment hidden (empty)
(Assignee)

Updated

4 years ago
Blocks: 1087720
(Assignee)

Comment 1

4 years ago
Created attachment 8535770 [details] [diff] [review]
js_2_b2g.patch

Jonas, would you like to see the change that Steve suggested to happen? See https://bugzilla.mozilla.org/show_bug.cgi?id=1087739#c3 or is it not worth doing? If you agree with Steve, then I could perform that textual replacement in all of the JS callsite patches! I don't have a strong feeling about it!
Attachment #8535770 - Flags: review?(jonas)
(Assignee)

Updated

4 years ago
Assignee: nobody → mozilla
Status: NEW → ASSIGNED
Comment on attachment 8535770 [details] [diff] [review]
js_2_b2g.patch

Review of attachment 8535770 [details] [diff] [review]:
-----------------------------------------------------------------

Add a aLoadInfo argument and pass that through instead.
Attachment #8535770 - Flags: review?(jonas) → review-
(Assignee)

Updated

4 years ago
Blocks: 1099587
(Assignee)

Comment 3

4 years ago
Created attachment 8536314 [details] [diff] [review]
js_2_b2g.patch

In Bug 1111025 we are adding newChannelFromURIWithLoadInfo() which we then can use for this patch.
Attachment #8535770 - Attachment is obsolete: true
Attachment #8536314 - Flags: review?(jonas)
https://hg.mozilla.org/mozilla-central/rev/da727b086b56
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.