Commit graph

64 commits

Author SHA1 Message Date
Myk Melez
25349d2601 Bug 1518283 - prohibit blank lines at the beginning and end of blocks (eslint padded-blocks) r=mossop,Standard8
Differential Revision: https://phabricator.services.mozilla.com/D17526

--HG--
extra : moz-landing-system : lando
2019-01-30 17:26: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
Mark Banner
15b881a65f Bug 1521707 - Move promisePopupEvent to BrowserTestUtils.jsm. r=mak
Differential Revision: https://phabricator.services.mozilla.com/D17259

--HG--
extra : moz-landing-system : lando
2019-01-23 14:04:46 +00:00
Andreas Pehrson
1bd9ff020f Bug 1421724 - Steal back focus from global webrtc indicator when doing second gUM requests. r=florian
Differential Revision: https://phabricator.services.mozilla.com/D15711

--HG--
extra : moz-landing-system : lando
2019-01-04 14:43:46 +00:00
Brian Grinstead
150199f6f8 Bug 1488938 - Round two: migrate mochitest-browser tests to use element variations of firstChild, etc;r=Gijs
This is a follow up to Bug 1482667. The list of callers was gathered by instrumenting
the webidl calls to these methods and dumping JS stack when they are called in browser.xul.

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

--HG--
extra : moz-landing-system : lando
2018-09-07 15:31:38 +00:00
Mark Banner
691543ee89 Bug 1486739 - Add missing dangling commas in browser/, services/, taskcluster/ and toolkit/. r=mossop
Automatic changes by ESLint, except for manual corrections for .xml files.

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

--HG--
extra : moz-landing-system : lando
2018-08-31 05:59:17 +00:00
Kris Maglione
80327d3561 Bug 1484496: Part 5a - Convert browser/ nsISimpleEnumerator users to use JS iteration. r=Gijs
Differential Revision: https://phabricator.services.mozilla.com/D3729

--HG--
extra : rebase_source : e187b8e9a6b6db7ebc762adda5e489b25c7a7e43
extra : histedit_source : 868cb99d09954a51d6be321fcb516475ef70eb33
2018-08-18 19:27:33 -07:00
Boris Zbarsky
c955b98d55 Bug 1476142 part 2. Remove some unnecessary QIs in browser. r=gijs 2018-08-02 15:26:47 -04:00
Dan Minor
a20dc982b3 Bug 1450658 - Add preference to disable focusing source; r=pehrsons
Focusing a source during a test run can cause Firefox to lose focus and
subsequent tests to timeout. This adds a preference which defaults to true
that allows the source to be focused. It is then set to false for tests that
involve screen sharing.

MozReview-Commit-ID: CNq2c87y8Ho

--HG--
extra : rebase_source : 805e2cbf9c09af419419a64d074e020784185b1d
2018-05-29 16:14:38 -04:00
Johann Hofmann
ed9da495e6 Bug 1456899 - Consider the page URI when checking WebRTC permissions in webrtcUI.jsm. r=nhnt11
While we already do a permission check in native webrtc code, the one
in webrtcUI is needed to provide extra protection against permission spam
by checking if there's a temporarily denied permission on the tab.

Since we introduced custom default permission values it is necessary to
also pass the URI to make sure we catch exceptions added by the user.

MozReview-Commit-ID: C8r6ymbKE3a

--HG--
extra : rebase_source : 8f0b3486f2f33bee75536ae2cf7dd091c5a9c568
2018-05-08 15:31:49 +02:00
Bryce Van Dyk
fdb3397eee Bug 1436523 - Make sure webrtc chrome mochitests don't use loopback devices. r=achronop
Before this changeset, mochitests in browser/base/content/test/webrtc will pick
up loopback devices due to the changes made earlier in this patch chain to
device enumeration. This results in test bustages (for unknown reasons).

This changeset updates the setup JS for these tests to make sure loopback
devices prefs are unset. A comment has been added noting that if the tests
can be made to work with loopback devices that would be ideal.

MozReview-Commit-ID: KG0io8s5r86

--HG--
extra : rebase_source : 9f2a23b4889bc484e511a31f0265236ac43fe7fc
2018-03-15 10:41:36 -04:00
Tooru Fujisawa
1adba8c1fc Bug 1442465 - Part 4.2: Stop unnecessarily awaiting on BrowserTestUtils.removeTab (simple part). r=dao 2018-03-19 11:16:45 +09:00
Johann Hofmann
401e316e0d Bug 1442294 - Downgrade global streaming indicator when a higher priority device has been disabled. r=florian
MozReview-Commit-ID: 6GxXNsB6xYy

--HG--
extra : rebase_source : 20668e90464a585421fd854462208f2a592301f8
2018-03-08 14:08:21 +01:00
Johann Hofmann
09d9353e79 Bug 1333468 - Part 2 - Add tests for paused/disabled sharing state indicators. r=florian
MozReview-Commit-ID: G1CTAefOPNB

--HG--
extra : rebase_source : fcdfbf3d646ddf96a43b5facf69f0f7f8211dfa6
2018-02-13 10:09:07 +01: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
Tooru Fujisawa
99fcca089d Bug 1421120 - Wait for the next event tick before resolving promise in webrtc tests. r=Mossop 2017-12-09 06:08:44 +09:00
Dorel Luca
a10c678ff0 Backed out changeset 6050b349b801 (bug 1421120) for failing browser-chrome's browser/base/content/test/permissions/browser_temporary_permissions_expiry.js on Windows 7 debug without e10s r=backout on a CLOSED TREE 2017-12-09 18:58:55 +02:00
Tooru Fujisawa
eae17c5fec Bug 1421120 - Wait for the next event tick before resolving promise in webrtc tests. r=Mossop 2017-12-09 06:08:44 +09:00
Florian Quèze
2924991bf6 Bug 1368456 - remove Promise.jsm imports in tests, r=mconley. 2017-06-23 11:25:52 +02:00
Munro Mengjue Chiang
cea6dddaf6 Bug 861716 - add a mochitest for gUM request queue in MediaManager. r=florian,jib
MozReview-Commit-ID: 750T4pzvf95

--HG--
extra : rebase_source : b683ce86ce43ca33fdb303d55e6da21eea1725d1
2017-05-17 10:59:12 +08:00
Florian Quèze
97f401c644 Bug 1362882 - script-generated patch to switch from Promise.defer() to new Promise() where it's straightforward, rs=Mossop. 2017-05-12 14:55:27 +02:00
Florian Quèze
4e937e42a2 Bug 1353542 - Cleanup the script output and fix tests, r=Mossop. 2017-05-12 14:54:05 +02:00
Florian Quèze
16530fa1e3 Bug 1353542 - smaller script-generated patch converting remaining generators that are likely tasks (actual generators were identified by hand and whitelisted), r=Mossop. 2017-05-12 14:45:01 +02:00
Florian Quèze
5e3539e504 Bug 1353542 - massive script-generated patch converting Task.async and Task.spawn calls, and generators clearly identifiable as tasks, rs=Mossop. 2017-05-12 14:42:39 +02:00
Andreas Pehrson
da339f960b Bug 1320994 - Increase retries for screensharing preview window. r=florian
MozReview-Commit-ID: IPnLWPT8SNP

--HG--
extra : rebase_source : 21b082ae29fed7332ecafcd500d6cb8e25e42217
2017-05-03 22:58:40 +02:00
Andreas Pehrson
ad7fb286f8 Bug 1320994 - Remove chrome-test hacks that accomodated the misaligned MediaManager model. r=florian
MozReview-Commit-ID: BS693gGyVhm

--HG--
extra : rebase_source : 7ce756a9d9b2b965533ceae852fd0c5c95a34a64
2017-04-07 15:30:43 +02:00
Sebastian Hengst
4273e34cb0 Backed out changeset 2f475f8d6ffc (bug 1320994) 2017-05-03 13:14:13 +02:00
Andreas Pehrson
aee0a7853c Bug 1320994 - Remove chrome-test hacks that accomodated the misaligned MediaManager model. r=florian
MozReview-Commit-ID: BS693gGyVhm

--HG--
extra : rebase_source : ae85538f9150f8c272ca90ff43a9348b69f7bc22
2017-04-07 15:30:43 +02:00
Wes Kocher
2768cd5d15 Backed out 5 changesets (bug 1320994) for browser_downloads_panel_block.js permafails on Win7VM a=backout
Backed out changeset b744a070cafb (bug 1320994)
Backed out changeset 5766ba7143b4 (bug 1320994)
Backed out changeset 31afc3fcd61f (bug 1320994)
Backed out changeset 78e5f8775727 (bug 1320994)
Backed out changeset 6d4e1736fd69 (bug 1320994)

MozReview-Commit-ID: 9tPD6gwXttd

--HG--
extra : rebase_source : 56cb2ac6d2e057b171d74cd529b53e617c716b4c
2017-04-26 15:21:54 -07:00
Andreas Pehrson
936d360598 Bug 1320994 - Remove chrome-test hacks that accomodated the misaligned MediaManager model. r=florian
MozReview-Commit-ID: BS693gGyVhm

--HG--
extra : rebase_source : ae85538f9150f8c272ca90ff43a9348b69f7bc22
2017-04-07 15:30:43 +02:00
Florian Queze
37ff4fc7cc Bug 1356569 - Remove addObserver's last parameter when it is false, r=jaws. 2017-04-14 21:51:38 +02:00
Sebastian Hengst
a07223d699 Backed out changeset 322fde2d53bf (bug 1356569) so bug 1355161 can be backed out. r=backout 2017-04-14 23:39:22 +02:00
Florian Queze
95d4d20c17 Bug 1356569 - Remove addObserver's last parameter when it is false, r=jaws. 2017-04-14 21:51:38 +02:00
Blake Kaplan
971ced558b Bug 1353910 - Fix intermittents in webrtc tests while removing code duplication. r=florian
MozReview-Commit-ID: 83XSsnQ0bBY

--HG--
extra : rebase_source : f476470a9d78b1d820a1aa23263e5b289c1e77fa
2017-04-06 11:41:02 -07:00
Florian Quèze
f55f3ce4a0 Bug 1325049 - Fix the global webrtc sharing indicator to work with multiple content processes, r=felipe. 2017-03-15 22:47:06 +01:00
Florian Quèze
8bceb76f8e Bug 1323880 - Wait for the recording-window-ended notification instead of expecting it to happen before recording-device-events when reloading the page, r=past. 2017-02-06 12:28:44 +01:00
Florian Quèze
f187782ccf Bug 1334250 - script-generated patch to avoid removeEventListener calls when a variable is used for the event name, r=jaws. 2017-01-27 10:48:00 +01:00
Florian Quèze
0e0865f4fc Bug 1331599 - script-generated patch to replace removeEventListener calls with the once option when possible, r=jaws. 2017-01-25 07:01:52 +01:00
Munro Mengjue Chiang
2e04d9d8d4 Bug 1270572 - write tests for the umprompted gUM request; r=florian
MozReview-Commit-ID: LYWTNnfli6K

--HG--
extra : rebase_source : cc9a2d6471a51037567a9448a74ee110532779fd
2016-12-20 14:51:36 +08:00
Jared Wein
43bac26d30 Bug 1331661 - Enable the 'quotes' rule for eslint and fix most of the errors with --fix. r=Gijs
MozReview-Commit-ID: 6tv0Z06CO4a

--HG--
extra : rebase_source : 014c0b04d8538dc5f15bc6dd4ed6bd220c55c5d4
2017-01-17 09:48:17 -06:00
Florian Quèze
85611a7b6d Bug 1331081 - script generated patch to omit addEventListener/removeEventListener's third parameter when it's false, r=jaws.
--HG--
extra : rebase_source : a22344ee1569f58f1f0a01017bfe0d46a6a14602
2017-01-17 11:50:25 +01:00
Johann Hofmann
95370d69a3 Bug 1206232 - Add temporary permission states to SitePermissions.jsm. r=jdm,Paolo
MozReview-Commit-ID: zQVeEHhIax

--HG--
extra : rebase_source : bebcd31cfbd638c51ff21c7e2b62bcc536b026da
2016-09-22 23:09:30 +02:00
Jared Wein
9141469edf Bug 1326511 - Enable brace-style and no-multi-spaces eslint rules for toolkit. r=MattN
MozReview-Commit-ID: FuVu8skcqOe

--HG--
extra : rebase_source : 8ab34c4e46a7c3075b459bf44786ec184d10d203
2016-12-30 21:47:25 -05:00
Florian Quèze
50226f6114 Bug 1313324 - Cover the screensharing UI with browser chrome test, r=past.
--HG--
rename : browser/base/content/test/webrtc/browser_devices_get_user_media.js => browser/base/content/test/webrtc/browser_devices_get_user_media_screen.js
2016-12-15 00:56:35 +01:00
Panos Astithas
e0efffb708 Bug 1282768 - Part 3 - Convert permission notifications to use two buttons and a checkbox instead of multiple secondary actions. r=florian
MozReview-Commit-ID: KNf04CBaLKB

--HG--
extra : rebase_source : 6c89da5ce541922f55db3853a7bf6f3b6cbc74fa
2016-11-20 18:40:26 +01:00
Panos Astithas
64baf2d729 Bug 1282768 - Part 2 - Move the secondary actions of doorhanger notifications to their own menu button. r=paolo
MozReview-Commit-ID: 2cnufF7wZJG

--HG--
extra : rebase_source : 6bb9c9f8bd52bd8e6f54cbc2b0ef2760fee25c3a
2016-11-20 18:40:22 +01:00