Closed Bug 1190478 Opened 5 years ago Closed 5 years ago
Event .data until we ship message encryption .
40 bytes, text/x-review-board-request
Martin, This is a less invasive change than hiding the PushMessageData interface, but if you prefer the latter, let me know.
Bug 1190478 - Hide PushEvent.data until we ship message encryption. r?mt,jst
Comment on attachment 8642528 [details] MozReview Request: Bug 1190478 - Hide PushEvent.data until we ship message encryption. r?mt,jst https://reviewboard.mozilla.org/r/14725/#review13315 ::: dom/workers/ServiceWorkerEvents.h:193 (Diff revision 1) > + // nsRefPtr<PushMessageData> mData; I don't think that I would have removed this, or the cycle collection stuff. Removing the attributes from the WebIDL and then making the tests pass should be enough.
url: https://hg.mozilla.org/integration/mozilla-inbound/rev/14003c5e401ddc28b3d4c5e9da785b35facc80b0 changeset: 14003c5e401ddc28b3d4c5e9da785b35facc80b0 user: Nikhil Marathe <firstname.lastname@example.org> date: Mon Aug 03 11:04:41 2015 -0700 description: Bug 1190478 - Hide PushEvent.data until we ship message encryption. r=mt,jst
You need to log in before you can comment on or make changes to this bug.