Commit Graph

663 Commits

Author SHA1 Message Date
Bogdan Tara
ad1e568c64 Backed out 3 changesets (bug 1690472, bug 1690469, bug 1690468) for test_Sync.js failures CLOSED TREE
Backed out changeset a997c2b34b36 (bug 1690469)
Backed out changeset fc757eec91bd (bug 1690468)
Backed out changeset b60bf057d62f (bug 1690472)
2021-02-05 17:02:21 +02:00
Henrik Skupin
c8e15bb92e Bug 1690469 - [remote] Reorganize Remote Agent Rust module. r=remote-protocol-reviewers,jgraham
Depends on D104159

Differential Revision: https://phabricator.services.mozilla.com/D104209
2021-02-05 13:31:33 +00:00
Henrik Skupin
7cb9dcf75e Bug 1690468 - [remote] Move Remote Agent component related files to components/. r=remote-protocol-reviewers,jgraham
Depends on D103944

Differential Revision: https://phabricator.services.mozilla.com/D104159
2021-02-05 13:23:40 +00:00
Henrik Skupin
696ff9be8a Bug 1690472 - [remote] Move sharable top-level JS modules to shared/ subfolder. r=remote-protocol-reviewers,jgraham
Differential Revision: https://phabricator.services.mozilla.com/D103944
2021-02-05 13:21:03 +00:00
Masayuki Nakano
cb448c5095 Bug 1685491 - part.1: Map typical commands to synthesized keyboard events for test on Linux and macOS r=smaug,remote-protocol-reviewers
Currently, we don't allow keyboard events synthesized for tests retrieve native
key bindings in content process.  However, due to this, we cannot test keyboard
navigation, deleting per word, etc on Linux and macOS either with mochitest
or WPT.  For making better compatibility with the other browsers, we should
write WPT more with the test driver.  Therefore, we should allow keyboard
events synthesized for tests retrieve native key bindings.

On the other hand, if we make them retrieve customized keyboard shortcuts
in the environment, some developers may not be able to run tests locally without
resetting their customization.  Therefore, this patch makes `NativeKeyBindings`
set "standard" shortcut keys on the platform instead of retrieving actual
shortcut key results.

If referring the default shortcut key bindings is not good thing for
WebDriver/CDP, perhaps, `TextInputProcessor` should have a new flag which can
refer customized shortcut keys even in content process.  But I think that it
should be done in another bug because some edit commands are mapped forcibly
like this patch.
https://searchfox.org/mozilla-central/rev/c03e8de87cdb0ce0378c0886d3c0ce8bbf9dc44e/remote/domains/parent/Input.jsm#82-102

Differential Revision: https://phabricator.services.mozilla.com/D102877
2021-02-02 03:02:30 +00:00
Henrik Skupin
45553af4bb Bug 1683392 - [puppeteer] Use "npm ci" only when tests are run on TaskCluster. r=remote-protocol-reviewers,jdescottes
This patch adds a new "--ci" argument for the
"mach puppeteer-test" command. As such it can
also be used locally to simulate a test job in CI.

Differential Revision: https://phabricator.services.mozilla.com/D101780
2021-02-01 10:59:38 +00:00
Narcis Beleuzu
868935867c Backed out 2 changesets (bug 1596897) for bc failures on browser_search_discovery.js
Backed out changeset 635972f169e7 (bug 1596897)
Backed out changeset a84b667007e2 (bug 1596897)
2021-01-30 04:01:21 +02:00
Paul Zuehlcke
3d476753eb Bug 1596897 - Updated tests for decoupled permissions panel. r=johannh,remote-protocol-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D99893
2021-01-29 16:11:08 +00:00
David Burns
3cc69b8cc4 Bug 1687021: Allow connection header names to be case insensitive. r=remote-protocol-reviewers,whimboo
Some client bindings will send over the headers with either a capital
first letter and some might not. We need to handle all cases

Differential Revision: https://phabricator.services.mozilla.com/D103384
2021-01-29 08:17:38 +00:00
Masatoshi Kimura
54c1b8dee1 Bug 1687108 - Remove unused content policy types. r=ckerschb,remote-protocol-reviewers,smaug
Differential Revision: https://phabricator.services.mozilla.com/D102083
2021-01-19 09:42:13 +00:00
Micah Tigley
1b5b46ec79 Bug 1680637 - Update tests using the old dialog UI to only use the old UI. r=jaws,Gijs,marionette-reviewers,whimboo,remote-protocol-reviewers
Depends on D100172

Differential Revision: https://phabricator.services.mozilla.com/D101388
2021-01-14 21:45:14 +00:00
Kartik Gautam
7ae6aea145 Bug 1684173 - Add newline character at end of files when missing r=sylvestre,geckoview-reviewers,agi
Differential Revision: https://phabricator.services.mozilla.com/D100484
2021-01-07 08:53:08 +00:00
Cosmin Sabou
2978aa00a3 Backed out changeset dbed1cdf588f (bug 1684173) for mochitest plain and devtools failures. a=backout DONTBUILD 2020-12-28 00:43:51 +02:00
Kartik Gautam
775cdec032 Bug 1684173 - Add newline character at end of files when missing r=sylvestre
Depends on D100443

Differential Revision: https://phabricator.services.mozilla.com/D100484
2020-12-27 11:43:41 +00:00
Henrik Skupin
300b5a7010 Bug 1606604 - [remote] Disable experimental SameSite=Lax by default. r=remote-protocol-reviewers,jgraham
Differential Revision: https://phabricator.services.mozilla.com/D100168
2020-12-22 12:54:55 +00:00
Brindusan Cristian
65644db919 Backed out 2 changesets (bug 1606604) for build bustages on remote_default. CLOSED TREE
Backed out changeset 7357db102337 (bug 1606604)
Backed out changeset 8c6c88dc5956 (bug 1606604)
2020-12-22 13:57:29 +02:00
Henrik Skupin
0d33e4db2d Bug 1606604 - [remote] Disable experimental SameSite=Lax by default. r=remote-protocol-reviewers,jgraham
Differential Revision: https://phabricator.services.mozilla.com/D100168
2020-12-21 12:50:44 +00:00
Henrik Skupin
016c762add Bug 1680504 - [remote] Implement Runtime.exceptionThrown. r=remote-protocol-reviewers,jgraham
Differential Revision: https://phabricator.services.mozilla.com/D100026
2020-12-21 12:35:40 +00:00
Henrik Skupin
915e1d9809 Bug 1681855 - [remote] Fallback to default timeout in browser_consoleAPICalled.js. r=remote-protocol-reviewers,jgraham
The 250ms are too short given that the call to any console API could be
delayed due to IPC communication. As such use the default timeout
value as done by all the other tests that are using the events
recorder.

Differential Revision: https://phabricator.services.mozilla.com/D99657
2020-12-14 09:25:49 +00:00
Xidorn Quan
3173c39eb9 Bug 1681935 - Replace dependency to failure with thiserror in remote crate. r=remote-protocol-reviewers,jgraham
Differential Revision: https://phabricator.services.mozilla.com/D99442
2020-12-11 14:34:23 +00:00
Henrik Skupin
f4dc4d8584 Bug 1553854 - [remote] Implement Runtime.consoleAPICalled. r=remote-protocol-reviewers,maja_zf
Differential Revision: https://phabricator.services.mozilla.com/D98599
2020-12-10 21:54:10 +00:00
Sam Foster
a42c193442 Bug 1654684 - Move chevron.svg to toolkit. r=remote-protocol-reviewers,desktop-theme-reviewers,ntim
Differential Revision: https://phabricator.services.mozilla.com/D97394
2020-12-09 20:49:25 +00:00
Csoregi Natalia
4eae934956 Backed out 2 changesets (bug 1654684) for browser chrome failures with .printPreviewCurrentPageNumber. CLOSED TREE
Backed out changeset 2fdc1b5569e1 (bug 1654684)
Backed out changeset 8ae1d037aa9c (bug 1654684)
2020-12-09 16:11:42 +02:00
Sam Foster
c5963a5340 Bug 1654684 - Move chevron.svg to toolkit. r=remote-protocol-reviewers,desktop-theme-reviewers,ntim
Differential Revision: https://phabricator.services.mozilla.com/D97394
2020-12-09 02:45:29 +00:00
Henrik Skupin
f56f5b402f Bug 1680814 - [remote] Target events have to emit a complete TargetInfo dictionary. r=remote-protocol-reviewers,jdescottes
Differential Revision: https://phabricator.services.mozilla.com/D98848
2020-12-08 09:19:39 +00:00
Henrik Skupin
8bbcc2773b Bug 1680962 - [remote] sendResponse has to always return a result in the response payload. r=remote-protocol-reviewers,jdescottes
Differential Revision: https://phabricator.services.mozilla.com/D98854
2020-12-07 14:01:17 +00:00
Brindusan Cristian
6474307318 Backed out 2 changesets (bug 1654684) for bc failures at browser_ext_tabs_printPreview. CLOSED TREE
Backed out changeset 5a0333ca8a19 (bug 1654684)
Backed out changeset 87d06a4d6ab6 (bug 1654684)
2020-12-07 12:59:10 +02:00
Sam Foster
acf2633244 Bug 1654684 - Move chevron.svg to toolkit. r=remote-protocol-reviewers,desktop-theme-reviewers,ntim
Differential Revision: https://phabricator.services.mozilla.com/D97394
2020-12-07 08:44:06 +00:00
Henrik Skupin
5167e2ba69 Bug 1680740 - [remote] Implement Target.getTargets. r=remote-protocol-reviewers,maja_zf
Differential Revision: https://phabricator.services.mozilla.com/D98803
2020-12-04 19:56:36 +00:00
Maja Frydrychowicz
ef2934709b Bug 1672461 - [remote] Pin package dependencies when installing Puppeteer in Mozilla CI r=remote-protocol-reviewers,whimboo
Calling `npm ci` instead of `npm install` generates a reproducible
node_modules directory based on the entries in Puppeteer's `package-lock.json`

The most-recent Puppeteer sync (Bug 1670286) commits package-lock.json
to mozilla-central for the first time.

Differential Revision: https://phabricator.services.mozilla.com/D97811
2020-12-04 15:08:17 +00:00
Maja Frydrychowicz
dc68e2c439 Bug 1670286 - [remote] Update Puppeteer-vendoring documentation r=remote-protocol-reviewers,whimboo
Differential Revision: https://phabricator.services.mozilla.com/D97810
2020-12-04 15:08:14 +00:00
Maja Frydrychowicz
48a8400533 Bug 1670286 - [puppeteer] Sync vendored puppeteer to v5.5.0 r=remote-protocol-reviewers,whimboo
Differential Revision: https://phabricator.services.mozilla.com/D97809
2020-12-04 15:08:06 +00:00
Maja Frydrychowicz
3ac661f84d Bug 1670286 - [puppeteer] Remove unwanted docs directories when vendoring r=remote-protocol-reviewers,whimboo
Differential Revision: https://phabricator.services.mozilla.com/D97808
2020-12-04 09:11:46 +00:00
Henrik Skupin
dacc23265c Bug 1680733 - [remote] Use a case-insensitive check for WebSocket upgrade header value. r=remote-protocol-reviewers,jgraham
Differential Revision: https://phabricator.services.mozilla.com/D98751
2020-12-04 13:28:28 +00:00
Henrik Skupin
36a7a875c8 Bug 1680101 - [remote] Temporarily force disabling Fission until Remote Agent is compatible. r=remote-protocol-reviewers,maja_zf
Differential Revision: https://phabricator.services.mozilla.com/D98377
2020-12-01 20:33:40 +00:00
Maja Frydrychowicz
31e88a9520 Bug 1677665 - Update Architecture docs DONTBUILD
Differential Revision: https://phabricator.services.mozilla.com/D97250
2020-11-17 03:35:16 +00:00
Nika Layzell
42028efc71 Bug 1671983 - Part 4: Stop awaiting BrowserTestUtils.loadURI, r=annyG,remote-protocol-reviewers,extension-reviewers,preferences-reviewers,whimboo,zombie
This method only is async in order to allow callers to wait for a process switch
triggered by the call to `loadURI` to be finished before resolving. With
DocumentChannel, we should never trigger a process switch eagerly like this
again, so we don't need any of the async behaviour here anymore.

This part is largely mechanical changes to tests, removing the `await` calls on
`loadURI`, and a follow-up part will remove the actual async logic from
`BrowserTestUtils.loadURI`.

Differential Revision: https://phabricator.services.mozilla.com/D94641
2020-11-12 18:01:03 +00:00
Henrik Skupin
e9e39992b5 Bug 1605784 - [remote] Implement Emulation.setTouchEmulationEnabled. r=remote-protocol-reviewers,jdescottes
Differential Revision: https://phabricator.services.mozilla.com/D96705
2020-11-11 19:22:46 +00:00
Henrik Skupin
c113dc1c42 Bug 1675266 - [remote] Report correct browser name/version, and user agent string for "/json/version" httpd handler. r=remote-protocol-reviewers,maja_zf
Differential Revision: https://phabricator.services.mozilla.com/D96009
2020-11-06 18:36:46 +00:00
Henrik Skupin
28bce1f7d9 Bug 1675266 - [remote] Add debuggerAddress as attribute on the Remote Agent interface. r=remote-protocol-reviewers,maja_zf
To allow other components in Firefox to easily find the address
under which the Remote Agent is accessible, add the host and port
information as "debuggerAddress" to the Remote Agent's interface
definition.

Differential Revision: https://phabricator.services.mozilla.com/D96008
2020-11-06 18:36:49 +00:00
Csoregi Natalia
cf51e7ef26 Backed out 3 changesets (bug 1675266, bug 1671205) for failures on test_capabilities.js. CLOSED TREE
Backed out changeset a1900289bdac (bug 1671205)
Backed out changeset 04e39b99f9c4 (bug 1675266)
Backed out changeset 3fd5c9f2bd29 (bug 1675266)
2020-11-06 20:32:14 +02:00
Henrik Skupin
e2db3e91fc Bug 1675266 - [remote] Report correct browser name/version, and user agent string for "/json/version" httpd handler. r=remote-protocol-reviewers,maja_zf
Differential Revision: https://phabricator.services.mozilla.com/D96009
2020-11-06 16:41:52 +00:00
Henrik Skupin
53c4528426 Bug 1675266 - [remote] Add debuggerAddress as attribute on the Remote Agent interface. r=remote-protocol-reviewers,maja_zf
To allow other components in Firefox to easily find the address
under which the Remote Agent is accessible, add the host and port
information as "debuggerAddress" to the Remote Agent's interface
definition.

Differential Revision: https://phabricator.services.mozilla.com/D96008
2020-11-06 17:01:23 +00:00
Maja Frydrychowicz
0c4b2387b8 Bug 1674939 - [puppeteer] Update metadata for Page.close and Frame.goto tests r=remote-protocol-reviewers,whimboo
Differential Revision: https://phabricator.services.mozilla.com/D95851
2020-11-04 13:58:12 +00:00
Razvan Maries
10425eddfc Backed out 7 changesets (bug 1658084, bug 1671983) for perma failures on browser_async_remove_tab.js and browser_e10s_chrome_process.js. CLOSED TREE
Backed out changeset 2e6309c1cdbd (bug 1658084)
Backed out changeset 99aafd9304ef (bug 1671983)
Backed out changeset 80280b85280a (bug 1671983)
Backed out changeset 008db2659002 (bug 1671983)
Backed out changeset 32bd45c7fe3a (bug 1671983)
Backed out changeset 56e227e6580c (bug 1671983)
Backed out changeset a404f809f79d (bug 1671983)
2020-11-04 04:23:47 +02:00
Nika Layzell
5f9c85da95 Bug 1671983 - Part 4: Stop awaiting BrowserTestUtils.loadURI, r=annyG,remote-protocol-reviewers,extension-reviewers,preferences-reviewers,whimboo,zombie
This method only is async in order to allow callers to wait for a process switch
triggered by the call to `loadURI` to be finished before resolving. With
DocumentChannel, we should never trigger a process switch eagerly like this
again, so we don't need any of the async behaviour here anymore.

This part is largely mechanical changes to tests, removing the `await` calls on
`loadURI`, and a follow-up part will remove the actual async logic from
`BrowserTestUtils.loadURI`.

Differential Revision: https://phabricator.services.mozilla.com/D94641
2020-11-04 00:55:36 +00:00
Maja Frydrychowicz
ab28f100d2 Bug 1673350 - [puppeteer] Mark click test as known intermittent r=remote-protocol-reviewers,whimboo
Differential Revision: https://phabricator.services.mozilla.com/D94829
2020-10-27 14:50:02 +00:00
Ricky Stewart
02a7b4ebdf Bug 1654103: Standardize on Black for Python code in mozilla-central.
Allow-list all Python code in tree for use with the black linter, and re-format all code in-tree accordingly.

To produce this patch I did all of the following:

1. Make changes to tools/lint/black.yml to remove include: stanza and update list of source extensions.

2. Run ./mach lint --linter black --fix

3. Make some ad-hoc manual updates to python/mozbuild/mozbuild/test/configure/test_configure.py -- it has some hard-coded line numbers that the reformat breaks.

4. Make some ad-hoc manual updates to `testing/marionette/client/setup.py`, `testing/marionette/harness/setup.py`, and `testing/firefox-ui/harness/setup.py`, which have hard-coded regexes that break after the reformat.

5. Add a set of exclusions to black.yml. These will be deleted in a follow-up bug (1672023).

# ignore-this-changeset

Differential Revision: https://phabricator.services.mozilla.com/D94045
2020-10-26 18:34:53 +00:00
Bogdan Tara
da1098d4aa Backed out 10 changesets (bug 1654103, bug 1672023, bug 1518999) for PanZoomControllerTest.touchEventForResult gv-junit failures CLOSED TREE
Backed out changeset ff3fb0b4a512 (bug 1672023)
Backed out changeset e7834b600201 (bug 1654103)
Backed out changeset 807893ca8069 (bug 1518999)
Backed out changeset 13e6b92440e9 (bug 1518999)
Backed out changeset 8b2ac5a6c98a (bug 1518999)
Backed out changeset 575748295752 (bug 1518999)
Backed out changeset 65f07ce7b39b (bug 1518999)
Backed out changeset 4bb80556158d (bug 1518999)
Backed out changeset 8ac8461d7bd7 (bug 1518999)
Backed out changeset e8ba13ee17f5 (bug 1518999)
2020-10-24 03:36:18 +03:00
Ricky Stewart
c0cea3b0fa Bug 1654103: Standardize on Black for Python code in mozilla-central. r=remote-protocol-reviewers,marionette-reviewers,webdriver-reviewers,perftest-reviewers,devtools-backward-compat-reviewers,jgilbert,preferences-reviewers,sylvestre,maja_zf,webcompat-reviewers,denschub,ntim,whimboo,sparky
Allow-list all Python code in tree for use with the black linter, and re-format all code in-tree accordingly.

To produce this patch I did all of the following:

1. Make changes to tools/lint/black.yml to remove include: stanza and update list of source extensions.

2. Run ./mach lint --linter black --fix

3. Make some ad-hoc manual updates to python/mozbuild/mozbuild/test/configure/test_configure.py -- it has some hard-coded line numbers that the reformat breaks.

4. Make some ad-hoc manual updates to `testing/marionette/client/setup.py`, `testing/marionette/harness/setup.py`, and `testing/firefox-ui/harness/setup.py`, which have hard-coded regexes that break after the reformat.

5. Add a set of exclusions to black.yml. These will be deleted in a follow-up bug (1672023).

# ignore-this-changeset

Differential Revision: https://phabricator.services.mozilla.com/D94045
2020-10-23 20:40:42 +00:00