1147274 - Issue warning when using the old push notification system

Status: RESOLVED INVALID

(Core :: DOM: Push Subscriptions, defect, P5)

Comment 1

[Doug Turner (:dougt)]

Attachment: 0003-Move-old-push-API-to-pushffos.patch

Comment 2

[Doug Turner (:dougt)]

Comment on attachment 8582885 [details] [diff] [review]
0003-Move-old-push-API-to-pushffos.patch

wrong file.

Comment 3

[Doug Turner (:dougt)]

Attachment: 0001-Bug-1137245.patch

Comment 4

[Doug Turner (:dougt)]

Attachment: warn_when_using_old_push

Comment 5

[Nikhil Marathe [:nsm] (No longer reading bugmail, please needinfo?)]

Comment on attachment 8582888 [details] [diff] [review]
warn_when_using_old_push

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

::: dom/push/Push.js
@@ +106,5 @@
>    register: function() {
>      debug("register()");
> +
> +    dump("This is a deprecated API. Get ready for the Push API.  See http://www.w3.org/TR/push-api/.\n");
> +      

Use something like https://dxr.mozilla.org/mozilla-central/source/dom/media/PeerConnection.js#544 instead to report to the console.

Comment 6

[Jens Stutte [:jstutte]]

Since bug 1296579 this seems to be not needed any more.