figure out what the subscriptionid is.

RESOLVED FIXED in Firefox 40

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: dougt, Unassigned)

Tracking

unspecified
mozilla40
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(firefox40 fixed)

Details

(URL)

Attachments

(1 attachment)

Comment hidden (empty)
(Reporter)

Updated

4 years ago
Blocks: 1153499

Comment 1

4 years ago
subscriptionid is removed from PushSubscription interface in current draft to be merged with endpoint property.
(Reporter)

Comment 2

4 years ago
Created attachment 8597690 [details] [diff] [review]
0001-Bug-1149271-Remove-subscriptionId.-r-jdm.patch
Attachment #8597690 - Flags: review?(josh)

Comment 3

4 years ago
I'm not a DOM peer, and this has WebIDL changes.
(Reporter)

Updated

4 years ago
Flags: needinfo?(jst)
(Reporter)

Updated

4 years ago
Attachment #8597690 - Flags: review?(josh) → review?(amarchesini)
Comment on attachment 8597690 [details] [diff] [review]
0001-Bug-1149271-Remove-subscriptionId.-r-jdm.patch

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

Don't we have any test to update?
Attachment #8597690 - Flags: review?(amarchesini) → review+
(Reporter)

Comment 5

4 years ago
> Don't we have any test to update?

No.  we were hoping to remove this from the spec and didn't write tests for it.
https://hg.mozilla.org/mozilla-central/rev/a3d3da50bc0f
Status: NEW → RESOLVED
Last Resolved: 4 years ago
status-firefox40: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla40

Updated

4 years ago
Flags: needinfo?(jst)
You need to log in before you can comment on or make changes to this bug.