Closed
Bug 1441688
Opened 6 years ago
Closed 6 years ago
Remove nsIIPCSerializableURI.Deserialize
Categories
(Core :: Networking, enhancement, P2)
Core
Networking
Tracking
()
RESOLVED
FIXED
mozilla60
Tracking | Status | |
---|---|---|
firefox60 | --- | fixed |
People
(Reporter: valentin, Assigned: valentin)
References
Details
(Whiteboard: [necko-triaged])
Attachments
(1 file)
In order to make URI implementations immutable, nsIIPCSerializableURI.Deserialize needs to be removed. Deserialization already happens using mutators (see URIUtils.cpp::DeserializeURI)
Comment hidden (mozreview-request) |
Assignee | ||
Updated•6 years ago
|
Priority: P1 → P2
Comment 2•6 years ago
|
||
OK, so the only way to deserialize a URI is via [1], which forwards now to the (private) Deserialize() impl on respective uri classes, right? [1] https://searchfox.org/mozilla-central/rev/14d933246211b02f5be21d2e730a57cf087c6606/netwerk/base/nsIURIMutator.idl#57-66
Comment 3•6 years ago
|
||
mozreview-review |
Comment on attachment 8954560 [details] Bug 1441688 - Remove nsIIPCSerializableURI.Deserialize https://reviewboard.mozilla.org/r/223612/#review229870
Attachment #8954560 -
Flags: review?(honzab.moz) → review+
Assignee | ||
Comment 4•6 years ago
|
||
(In reply to Honza Bambas (:mayhemer) from comment #2) > OK, so the only way to deserialize a URI is via [1], which forwards now to > the (private) Deserialize() impl on respective uri classes, right? Yes, exactly.
Assignee | ||
Comment 5•6 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=478c2179340f97687b4f8d385fddf19c4dac8f89
Comment hidden (mozreview-request) |
Pushed by valentin.gosu@gmail.com: https://hg.mozilla.org/integration/autoland/rev/1e7b27d3a00f Remove nsIIPCSerializableURI.Deserialize r=mayhemer
Comment 8•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/1e7b27d3a00f
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
You need to log in
before you can comment on or make changes to this bug.
Description
•