Closed
Bug 1087725
Opened 10 years ago
Closed 10 years ago
Make JS callers of ios.newChannel call ios.newChannel2 in b2g/
Categories
(Core :: DOM: Security, defect)
Core
DOM: Security
Tracking
()
RESOLVED
FIXED
mozilla37
People
(Reporter: ckerschb, Assigned: ckerschb)
References
Details
Attachments
(1 file, 1 obsolete file)
|
1.43 KB,
patch
|
sicking
:
review+
|
Details | Diff | Splinter Review |
No description provided.
|
Assignee | |
Comment 1•10 years ago
|
||
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•10 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 | |
Comment 3•10 years ago
|
||
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)
Attachment #8536314 -
Flags: review?(jonas) → review+
|
|
Assignee | |
Comment 4•10 years ago
|
||
Target Milestone: --- → mozilla37
|
||
Comment 5•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•