Closed Bug 1326520 Opened 8 years ago Closed 7 years ago

Rename nsIURI.path to something less confusing

Categories

(Core :: Networking, defect)

50 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla57
Tracking Status
firefox57 --- fixed

People

(Reporter: emk, Assigned: emk)

References

(Depends on 1 open bug, Blocks 2 open bugs)

Details

(Whiteboard: [necko-backlog])

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1323683 +++

1. Add an alias of nsIURI.path (e.g. .pathQueryRef).
2. Remove in-tree use of .path.
3. Deprecate .path.
4. Remove .path completely (probably after Firefox 57).
Assignee: nobody → VYV03354
Status: NEW → ASSIGNED
Comment on attachment 8891800 [details]
Bug 1326520 - Rename nsIURI.path to pathQueryRef. .gosu

https://reviewboard.mozilla.org/r/162836/#review168144

Looks good. Do we have any (Mozilla-built) old style addons that we'll keep in 57? If so, we should make sure those are properly tested and/or updated.
Attachment #8891800 - Flags: review?(valentin.gosu) → review+
Comment on attachment 8891800 [details]
Bug 1326520 - Rename nsIURI.path to pathQueryRef. .gosu

https://reviewboard.mozilla.org/r/162836/#review168144

According to the recent dev-platform discussion, we don't have to care about out-of-tree legacy addons in a post-57 world.
https://groups.google.com/forum/#!topic/mozilla.dev.platform/TqoheEOdm8w
Pushed by VYV03354@nifty.ne.jp:
https://hg.mozilla.org/integration/autoland/rev/509b327a4a10
Rename nsIURI.path to pathQueryRef. r=valentin.gosu
https://hg.mozilla.org/mozilla-central/rev/509b327a4a10
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Depends on: 1386996
Depends on: 1387017
Depends on: 1387101
Is there a reason the labeling on the diagram at http://searchfox.org/mozilla-central/rev/4d8e389498a08668cce9ebf6232cc96be178c3e4/netwerk/base/nsIURI.idl#16-28 wasn't modified in this bug?
Flags: needinfo?(VYV03354)
No, I simply forgot to change the diagram.
Flags: needinfo?(VYV03354)
Depends on: 1420714
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: