Closed Bug 1363057 Opened 8 years ago Closed 8 years ago

make nsIDOMLocation an empty interface

Categories

(Core :: DOM: Core & HTML, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: baku, Assigned: baku)

References

Details

Attachments

(1 file, 2 obsolete files)

This is a follow up of bug 1362003
Depends on: 1362003
Attached patch location4.patch (obsolete) — Splinter Review
Assignee: nobody → amarchesini
Attachment #8865475 - Flags: review?(bzbarsky)
Comment on attachment 8865475 [details] [diff] [review] location4.patch Michael, do you want to review this one?
Attachment #8865475 - Flags: review?(bzbarsky) → review?(michael)
Also, could use a followup to remove nsIDOMLocation altogether.
Attached patch location4.patch (obsolete) — Splinter Review
Attachment #8865475 - Attachment is obsolete: true
Attachment #8865475 - Flags: review?(michael)
Attachment #8865496 - Flags: review?(bzbarsky)
Attachment #8865496 - Flags: review?(bzbarsky) → review?(michael)
Attached patch location4.patchSplinter Review
Attachment #8865496 - Attachment is obsolete: true
Attachment #8865496 - Flags: review?(michael)
Attachment #8865510 - Flags: review?(bzbarsky)
Comment on attachment 8865510 [details] [diff] [review] location4.patch Review of attachment 8865510 [details] [diff] [review]: ----------------------------------------------------------------- r=me - I'm assuming that I was supposed to review this given that bz redirected it to me a couple of times. ::: media/webrtc/signaling/src/peerconnection/PeerConnectionImpl.cpp @@ +610,5 @@ > nsAutoCString locationCStr; > > if (RefPtr<Location> location = mWindow->Location()) { > nsAutoString locationAStr; > + res = location->ToString(locationAStr); Is this change related? Should this be in a separate patch?
Attachment #8865510 - Flags: review?(bzbarsky) → review+
Pushed by amarchesini@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/267ad90533e8 Make nsIDOMLocation an empty interface, r=mystor
Flags: needinfo?(amarchesini)
Pushed by amarchesini@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/ac941cae8f63 Make nsIDOMLocation an empty interface, r=mystor
Backout by kwierso@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/e25b794d33f0 Backed out changeset 267ad90533e8 for Windows build bustage in nsNativeAppSupportWin.cpp a=backout CLOSED TREE
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: