Граф коммитов

68 Коммитов

Автор SHA1 Сообщение Дата
Mathieu Leplatre 1cdc04dfda Bug 1541888 - Enable ESLinting in dom/push/ r=glasserc,lina
Enable ESLinting in dom/push/

Differential Revision: https://phabricator.services.mozilla.com/D26321

--HG--
extra : moz-landing-system : lando
2019-04-09 10:05:51 +00:00
Lina Cambridge 256003368d Bug 1524655 - Remove `dom.push.alwaysConnect` and connect unconditionally. r=jrconlin,pjenvey
This commit also fixes a race in `test_error_reporting.html`, where the
push service would initialize and attach its listeners after
`sessionstore-windows-restored`. Even though the test replaces the real
service with a mock, the former keeps listening for pref changes.
When the test calls `setupPrefs` to enable the push connection, the
real service tries to connect to the push server, which asserts in
automation because non-local connections aren't allowed.

We work around this by ensuring that `replacePushService` and
`restorePushService` always wait for the service to shut down before
replacing it with a mock, or restoring the real implementation.

Finally, this commit removes a test that's no longer relevant, since
we don't need to fetch all subscriptions at startup.

Differential Revision: https://phabricator.services.mozilla.com/D20085

--HG--
extra : moz-landing-system : lando
2019-03-14 22:37:51 +00:00
Dorel Luca f1c7ba91fa Backed out changeset 88ea72c345ab (bug 1524655) for mochitest failures in dom/push/test/test_error_reporting.html 2019-03-14 08:45:37 +02:00
Lina Cambridge 84764d6a93 Bug 1524655 - Remove `dom.push.alwaysConnect` and connect unconditionally. r=jrconlin,pjenvey
Also, remove a test that's no longer relevant, since we don't need to
fetch all subscriptions at startup.

Differential Revision: https://phabricator.services.mozilla.com/D20085

--HG--
extra : moz-landing-system : lando
2019-03-14 04:22:18 +00:00
Jonathan Kingston f6680698bd Bug 1525319 - Removing context from OnDataAvailable r=valentin
Differential Revision: https://phabricator.services.mozilla.com/D20881

--HG--
extra : moz-landing-system : lando
2019-02-27 23:42:27 +00:00
Jonathan Kingston dd4c731d8c Bug 1525319 - Changing js to remove context from onStartRequest and onStopRequest r=valentin
Differential Revision: https://phabricator.services.mozilla.com/D20771

--HG--
extra : moz-landing-system : lando
2019-02-27 23:41:54 +00:00
Dorel Luca 4a4dcd57fa Backed out 4 changesets (bug 1525319) for Android failures in dom/base/test/test_progress_events_for_gzip_data.html
Backed out changeset b73f033efb41 (bug 1525319)
Backed out changeset 1d318d5c6b98 (bug 1525319)
Backed out changeset 6d73418988d4 (bug 1525319)
Backed out changeset 84ca79bd2dc3 (bug 1525319)
2019-02-25 04:06:11 +02:00
Jonathan Kingston 63ab2bd8aa Bug 1525319 - Removing context from OnDataAvailable r=valentin
Differential Revision: https://phabricator.services.mozilla.com/D20881

--HG--
extra : moz-landing-system : lando
2019-02-24 20:27:55 +00:00
Jonathan Kingston 147e351e0b Bug 1525319 - Changing js to remove context from onStartRequest and onStopRequest r=valentin
Differential Revision: https://phabricator.services.mozilla.com/D20771

--HG--
extra : moz-landing-system : lando
2019-02-25 00:07:42 +00:00
Jonathan Kingston 3421b8fcff Bug 1520868 - Replacing AsyncOpen2 with AsyncOpen always r=valentin
Replacing js and text occurences of asyncOpen2
Replacing open2 with open

Differential Revision: https://phabricator.services.mozilla.com/D16885

--HG--
rename : layout/style/test/test_asyncopen2.html => layout/style/test/test_asyncopen.html
extra : moz-landing-system : lando
2019-02-12 16:08:25 +00:00
Kris Maglione e930b89c34 Bug 1514594: Part 3 - Change ChromeUtils.import API.
***
Bug 1514594: Part 3a - Change ChromeUtils.import to return an exports object; not pollute global. r=mccr8

This changes the behavior of ChromeUtils.import() to return an exports object,
rather than a module global, in all cases except when `null` is passed as a
second argument, and changes the default behavior not to pollute the global
scope with the module's exports. Thus, the following code written for the old
model:

  ChromeUtils.import("resource://gre/modules/Services.jsm");

is approximately the same as the following, in the new model:

  var {Services} = ChromeUtils.import("resource://gre/modules/Services.jsm");

Since the two behaviors are mutually incompatible, this patch will land with a
scripted rewrite to update all existing callers to use the new model rather
than the old.
***
Bug 1514594: Part 3b - Mass rewrite all JS code to use the new ChromeUtils.import API. rs=Gijs

This was done using the followng script:

https://bitbucket.org/kmaglione/m-c-rewrites/src/tip/processors/cu-import-exports.jsm
***
Bug 1514594: Part 3c - Update ESLint plugin for ChromeUtils.import API changes. r=Standard8

Differential Revision: https://phabricator.services.mozilla.com/D16747
***
Bug 1514594: Part 3d - Remove/fix hundreds of duplicate imports from sync tests. r=Gijs

Differential Revision: https://phabricator.services.mozilla.com/D16748
***
Bug 1514594: Part 3e - Remove no-op ChromeUtils.import() calls. r=Gijs

Differential Revision: https://phabricator.services.mozilla.com/D16749
***
Bug 1514594: Part 3f.1 - Cleanup various test corner cases after mass rewrite. r=Gijs
***
Bug 1514594: Part 3f.2 - Cleanup various non-test corner cases after mass rewrite. r=Gijs

Differential Revision: https://phabricator.services.mozilla.com/D16750

--HG--
extra : rebase_source : 359574ee3064c90f33bf36c2ebe3159a24cc8895
extra : histedit_source : b93c8f42808b1599f9122d7842d2c0b3e656a594%2C64a3a4e3359dc889e2ab2b49461bab9e27fc10a7
2019-01-17 10:18:31 -08:00
Kris Maglione 2fe0de01dc Bug 1519596: Part 1 - Remove several unnecessary/unused ChromeUtils.import() calls. r=Gijs
Differential Revision: https://phabricator.services.mozilla.com/D16377

--HG--
extra : rebase_source : 6c4311387d25de425806aeb6c4691e12c9fcb855
2019-01-11 16:59:23 -08:00
Ethan Glasser-Camp bad5e0b59e Bug 1440022: initial implementation r=lina
MozReview-Commit-ID: GMnGfpUSnox

--HG--
extra : rebase_source : 859c7b3d6c7ef70850b80b6a7356cef76ec66570
2018-04-11 10:53:20 -04:00
Bogdan Tara 3719c11a02 Backed out 4 changesets (bug 1440022) for nsSocketTransport crashes when trying to access the internet CLOSED TREE
Backed out changeset 9fa127e6df56 (bug 1440022)
Backed out changeset 4f84a6745a51 (bug 1440022)
Backed out changeset 00cf71bb3861 (bug 1440022)
Backed out changeset b7d5b0db60ad (bug 1440022)
2018-06-11 22:17:53 +03:00
Ethan Glasser-Camp 6b074ef615 Bug 1440022: initial implementation r=lina
MozReview-Commit-ID: GMnGfpUSnox

--HG--
extra : rebase_source : 66decefdde118f56b7cfba36a77b7f5b8a6e4922
2018-04-11 10:53:20 -04:00
Kris Maglione bf4d9f1be5 Bug 1456035: Part 5 - Convert manual QueryInterface to ChromeUtils.generateQI. r=mccr8
Manually-implemented QueryInterface functions don't benefit from the
MozQueryInterface optimizaions, and a lot of them are in hot code, and
implement a large number of interfaces.

MozReview-Commit-ID: 8OzglraowZt

--HG--
extra : rebase_source : 5fff3d9973a0ea976096339a63ce9ff628b68441
2018-04-23 12:58:34 -07:00
Florian Quèze c714053d73 Bug 1433175 - scripted patch to replace Components.classes[, Components.interfaces.nsI, Components.utils. and Components.results. with Cc, Ci, Cu and Cr, r=Mossop. 2018-02-28 18:51:33 +01:00
Florian Quèze 682b1ec3b2 Bug 1440284 - change this.EXPORTED_SYMBOLS back to var EXPORTED_SYMBOLS in JS modules, r=mccr8. 2018-02-23 20:50:01 +01:00
Andrew McCreight 5dec0e0beb Bug 1432992, part 1 - Remove definitions of Ci, Cr, Cc, and Cu. r=florian
This patch was autogenerated by my decomponents.py

It covers almost every file with the extension js, jsm, html, py,
xhtml, or xul.

It removes blank lines after removed lines, when the removed lines are
preceded by either blank lines or the start of a new block. The "start
of a new block" is defined fairly hackily: either the line starts with
//, ends with */, ends with {, <![CDATA[, """ or '''. The first two
cover comments, the third one covers JS, the fourth covers JS embedded
in XUL, and the final two cover JS embedded in Python. This also
applies if the removed line was the first line of the file.

It covers the pattern matching cases like "var {classes: Cc,
interfaces: Ci, utils: Cu, results: Cr} = Components;". It'll remove
the entire thing if they are all either Ci, Cr, Cc or Cu, or it will
remove the appropriate ones and leave the residue behind. If there's
only one behind, then it will turn it into a normal, non-pattern
matching variable definition. (For instance, "const { classes: Cc,
Constructor: CC, interfaces: Ci, utils: Cu } = Components" becomes
"const CC = Components.Constructor".)

MozReview-Commit-ID: DeSHcClQ7cG

--HG--
extra : rebase_source : d9c41878036c1ef7766ef5e91a7005025bc1d72b
2018-02-06 09:36:57 -08:00
Kris Maglione 918ed6c474 Bug 1431533: Part 5a - Auto-rewrite code to use ChromeUtils import methods. r=florian
This was done using the following script:
37e3803c7a/processors/chromeutils-import.jsm

MozReview-Commit-ID: 1Nc3XDu0wGl

--HG--
extra : source : 12fc4dee861c812fd2bd032c63ef17af61800c70
extra : intermediate-source : 34c999fa006bffe8705cf50c54708aa21a962e62
extra : histedit_source : b2be2c5e5d226e6c347312456a6ae339c1e634b0
2018-01-29 15:20:18 -08:00
Cosmin Sabou 9a65a40178 Backed out 3 changesets (bug 1431533) for Android mochitest failures on testEventDispatcher on a CLOSED TREE
Backed out changeset a1eca62826a1 (bug 1431533)
Backed out changeset 34c999fa006b (bug 1431533)
Backed out changeset e2674287e57f (bug 1431533)
2018-01-30 07:17:48 +02:00
Kris Maglione 6476f95b13 Bug 1431533: Part 5a - Auto-rewrite code to use ChromeUtils import methods. r=florian
This was done using the following script:
37e3803c7a/processors/chromeutils-import.jsm

MozReview-Commit-ID: 1Nc3XDu0wGl

--HG--
extra : source : 12fc4dee861c812fd2bd032c63ef17af61800c70
2018-01-29 15:20:18 -08:00
Brindusan Cristian af8879d1eb Backed out 2 changesets (bug 1431533) for ESlint failures on a CLOSED TREE
Backed out changeset 6e56f4c8843e (bug 1431533)
Backed out changeset 12fc4dee861c (bug 1431533)
2018-01-30 02:32:43 +02:00
Kris Maglione c276bb9375 Bug 1431533: Part 5a - Auto-rewrite code to use ChromeUtils import methods. r=florian
This was done using the following script:
37e3803c7a/processors/chromeutils-import.jsm

MozReview-Commit-ID: 1Nc3XDu0wGl

--HG--
extra : rebase_source : c004a023389f1f6bf3d2f3efe93c13d423b23ccd
2018-01-29 15:20:18 -08:00
Marco Castelluccio 51d8b32912 Bug 1357517 - Remove Preferences.jsm usage from some dom/push/* files. r=kitcambridge
--HG--
extra : rebase_source : 77dce879e3ceb5352ea7a65673dffd53c9b9276f
2017-07-28 23:13:57 +02:00
Florian Quèze 721c3ada5e Bug 1368456 - Remove imports of Promise.jsm from Firefox, r=mconley. 2017-06-23 11:25:18 +02:00
Kit Cambridge be7ad3ff16 Bug 1345001 - Remove expiring Push telemetry probes. r=liuche
We haven't found these probes useful for understanding the value of
Push, and many can be better determined from server metrics.

MozReview-Commit-ID: GM9K59OAbMT

--HG--
extra : rebase_source : b4478fe2aad5ce01ba5955406300973b3e0fb699
2017-03-06 19:06:32 -08:00
Florian Quèze fc6379e827 Bug 1329182 - remove trailing newURI null parameters in the rest of the tree, r=jaws. 2017-01-09 20:27:26 +01:00
Kit Cambridge fb5b58f401 Bug 1301469 - Handle exceptions thrown by `PushCrypto.decodeMsg` as decryption errors. r=mt
Previously, errors thrown by `decodeMsg` and `getCryptoParams` would
bubble up to the catch handler in `receivedPushMessage`, causing us to
report "ACK_NOT_DELIVERED" instead of "ACK_DECRYPTION_ERROR" in the ack
sent to the server.

MozReview-Commit-ID: FZFzYdebQGy

--HG--
extra : rebase_source : b21f15cc9ac12ff11496d4f63ee531c021bd7a29
2016-10-05 08:56:55 -07:00
Kit Cambridge 39cc699a48 Bug 1207744 - Track and re-send push unregister requests on reconnect. r=dragana
MozReview-Commit-ID: 2rFLm07n4EU

--HG--
extra : rebase_source : a93554e2320c0ff46e7b66d41f668ecaf75836d4
extra : source : f7e1ab1bd99c05c219fe75913f8f37ba39aec092
2016-06-08 06:13:15 -07:00
Kit Cambridge 9960b60842 Back out changeset f7e1ab1bd99c (bug 1207744) for X(3) failures.
MozReview-Commit-ID: 5mgyh20yXiI

--HG--
extra : rebase_source : d3adb0098eb1615fdd134cbb4f0afd8a151a5066
2016-06-07 20:34:54 -07:00
Kit Cambridge d8d829d3c8 Bug 1207744 - Track and re-send push unregister requests on reconnect. r=dragana
MozReview-Commit-ID: 2rFLm07n4EU

--HG--
extra : rebase_source : 813c59ee7a4ca0c56717343683a54df22f1697de
extra : histedit_source : 9288402c263848c381995c3c9ae52950f6d65467
2016-06-06 16:29:36 -07:00
Kit Cambridge 91fa002ac4 Bug 1267493 - Replace `isURIPotentiallyTrustworthy` usage in Push with a testing pref. r=dragana
MozReview-Commit-ID: LrjAyVeNMyI

--HG--
extra : rebase_source : 7b2f821a80f853986bb196d8fc5e1eae9d77fd48
2016-04-25 20:53:06 -07:00
Kit Cambridge 4f57dd25ce Bug 1266540 - Stub out Push error reporting for the GCM and H2 backends. r=wchen
MozReview-Commit-ID: 9TFH69IrQpP

--HG--
extra : rebase_source : 3dca34edb05cc3f386872b555bb3455b2fead317
2016-04-21 13:35:26 -07:00
Kit Cambridge 4f645f9bfe Bug 1247685 - Send subscription keys to the Push server. r=mt
MozReview-Commit-ID: 2i3UqgNGlEt

--HG--
extra : rebase_source : 9c30f34c7de7c9fa7f04db5f224bad3ac0e68904
2016-03-22 16:29:16 -07:00
Kit Cambridge d97d284c24 Bug 1246341 - Include status codes in "ack" and "unregister" requests. r=dragana
MozReview-Commit-ID: Gsh3FhTfvkX

--HG--
extra : rebase_source : 94b9afcf006a6201f01875df1aae20c8bb52fb76
2016-03-28 12:29:25 -07:00
Kit Cambridge bc0b536de1 Bug 1257821 - Support the new `aesgcm` content encoding scheme. r=mt
MozReview-Commit-ID: IPNXletzJRK

--HG--
extra : rebase_source : 3593624d3513a0ccc7422bfeefe5c85dca37541a
2016-03-18 09:01:50 -07:00
William Chen 2d5b3a7771 Bug 1246066 - Clear PushService timeout tasks on uninitialization. r=kitcambridge 2016-03-16 10:03:02 -07:00
Kit Cambridge 705744ff4b Bug 1243856 - Remove alarms from the Push H2 backend. r=dragana
MozReview-Commit-ID: LZJY0DtLE4E

--HG--
extra : rebase_source : 336c99819da6437a6a353946809f41b623bdc611
extra : source : a3a8b32866013bc64bf31c62a1fe9cf0a109cf33
2016-02-03 18:16:10 -08:00
Dragana Damjanovic 124d661902 Bug 1246632 - Fix ackMsgRecv in PushServiceHttp2.js. r=kitcambridge 2016-02-08 06:40:00 +01:00
Kit Cambridge efe23eadc2 Back out 2 changesets (bug 1243856) for intermittent xpcshell failures on a CLOSED TREE.
--HG--
extra : commitid : HOS6utihBe9
2016-02-05 22:39:23 -08:00
Kit Cambridge 7242c3549a Bug 1243856 - Remove alarms from the Push H2 backend. r=dragana
--HG--
extra : commitid : IdPNgHFGj1O
extra : rebase_source : c98f56b8c9e4683cf77ebd4269c834579d7651dd
2016-02-03 18:16:10 -08:00
Paolo Amadini 8b8676204b Bug 989960 - Unhandled rejections in DOM Promises should cause xpcshell tests to fail. r=Yoric
--HG--
extra : commitid : yWjnDGhk9j
extra : rebase_source : 24228660defd6971a621d52eee0b79be823c6926
extra : amend_source : ef8de24265fa9026b0f764a63453b816442f7232
extra : source : 3958782fe187026cbc3755ad6eae6920e67fa7d2
2016-02-03 12:58:03 +00:00
Wes Kocher 1b65a8ee2f Backed out changeset e8148cda7d85 (bug 989960) for xpcshell bustage
--HG--
extra : commitid : Bxwy5tzArNN
extra : rebase_source : a15513777534e54d05fda5037f24e15ac089b313
2016-02-01 08:25:19 -08:00
Paolo Amadini ecd744bdef Bug 989960 - Unhandled rejections in DOM Promises should cause xpcshell tests to fail. r=Yoric
--HG--
extra : commitid : 6aGIHyyYqSo
extra : rebase_source : 38fbf5e49d06970555a088c5ea3f8379e2064767
extra : source : 3958782fe187026cbc3755ad6eae6920e67fa7d2
2016-01-30 18:53:56 +00:00
Phil Ringnalda d42eb10039 Back out 3958782fe187 (bug 989960) for Android crashes in test_watchdog_default.js 2016-01-29 20:12:44 -08:00
Paolo Amadini aceee7aafc Bug 989960 - Unhandled rejections in DOM Promises should cause xpcshell tests to fail. r=Yoric
--HG--
extra : commitid : IkcxJUamV6B
extra : rebase_source : 07255457ec607c45037f832ca44a01f17a00da78
2016-01-29 13:19:53 +00:00
Kit Cambridge 0333c16142 Bug 1239558 - Exempt system Push subscriptions from quota and permissions checks. r=dragana
--HG--
extra : commitid : 6weiBkKPMVj
extra : rebase_source : b5d6479902725a37c5b212bc4b23dc8296f34c9a
2015-12-16 09:21:22 -08:00
Christoph Kerschbaumer 84b03c8a8e Bug 1232910 - Use channel.asyncOpen2 within dom/push/PushServiceHttp2.jsm. r=sicking r=dragana 2016-01-19 20:00:18 -08:00
Nick Alexander dcde606dc7 Bug 1240615 - Make PushService protocol init method return a Promise. r=kitcambridge
I'd like to make `PushService.init` fallible, but it's not obvious how
to act on failure.  This patch at least allows each service to block
_startService until service-specific work initialization is complete.

--HG--
extra : rebase_source : 641bb24429116b4a1db10343b04f83cc6331dc39
2015-11-18 15:55:13 -08:00