Closed Bug 1149271 Opened 8 years ago Closed 7 years ago

figure out what the subscriptionid is.

Categories

(Core :: DOM: Push Notifications, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla40
Tracking Status
firefox40 --- fixed

People

(Reporter: dougt, Unassigned)

References

()

Details

Attachments

(1 file)

No description provided.
Blocks: 1153499
subscriptionid is removed from PushSubscription interface in current draft to be merged with endpoint property.
I'm not a DOM peer, and this has WebIDL changes.
Flags: needinfo?(jst)
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+
> 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
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Flags: needinfo?(jst)

Putting together a PR to update BCD to note this was apparently removed before it shipped.

You need to log in before you can comment on or make changes to this bug.