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

626 Коммитов

Автор SHA1 Сообщение Дата
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
Maja Frydrychowicz 239b7b4bc9 Bug 1671725 - [puppeteer] Avoid recording second test status after timeout r=remote-protocol-reviewers,whimboo
This prevents intermittent orange jobs where an additional test result is
counted after the test has already ended.

Differential Revision: https://phabricator.services.mozilla.com/D94362
2020-10-22 14:11:21 +00:00
Maja Frydrychowicz 116e28902f Bug 1671725 - [puppeteer] Add missing await. r=remote-protocol-reviewers,whimboo
Differential Revision: https://phabricator.services.mozilla.com/D94361
2020-10-22 14:11:19 +00:00
Maja Frydrychowicz ded3e267ad Bug 1671725 - [puppeteer] Increase mocha timeout to 20 seconds. r=remote-protocol-reviewers,jgraham,whimboo
We originally set 15s to keep the overall job duration under 30 minutes.
Since then, we hit timeouts much less frequently and should be able to
afford a longer timeout. The Puppeteer project's default test timeout
is currently 25 seconds.

Also skipping a Page test due to an async Firefox-specific protocol
error that interferes with teardown.

Differential Revision: https://phabricator.services.mozilla.com/D94245
2020-10-22 14:11:16 +00:00
Maja Frydrychowicz bb67e5edb1 Bug 1671725 - [puppeteer] Handle async TimeoutErrors in several tests r=remote-protocol-reviewers,whimboo
This prevents the errors from being raised later on during an unrelated
test.

Differential Revision: https://phabricator.services.mozilla.com/D94244
2020-10-22 14:12:33 +00:00
Dorel Luca 1ff59cb7a3 Backed out changeset 7558c8821a07 (bug 1654103) for multiple failures. CLOSED TREE 2020-10-22 03:51:06 +03:00
Ricky Stewart 50762dacab 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. 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-21 21:27:27 +00:00
Kershaw Chang 883455945d Bug 1668851 - Try to write data to socket directly without waiting r=necko-reviewers,dragana,remote-protocol-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D92442
2020-10-08 13:26:03 +00:00
Henrik Skupin fca264c1ec Bug 1669230 - [remote] Remove --remote-debugger command line argument. r=remote-protocol-reviewers,jgraham
Using different names for enabling the remote agent via
--remote-debugger and --remote-debugging-port adds confusion
to users. Also given that --remote-debugging is actually
not used by any client library including Puppeteer,
it can be removed.

Using --remote-debugging-port instead will bring us onpar
with Chrome and how it enables CDP.

Differential Revision: https://phabricator.services.mozilla.com/D92603
2020-10-07 13:50:57 +00:00
Razvan Maries e97ee6395c Backed out changeset 35066da8f3c3 (bug 1666499) as per Henrik's request. CLOSED TREE 2020-10-02 10:55:34 +03:00
Maja Frydrychowicz 29d9310975 Bug 1665283 - [puppeteer] Remove datareporting.healthreport.about.reportUrl from Launcher.ts r=remote-protocol-reviewers,whimboo
Differential Revision: https://phabricator.services.mozilla.com/D91450
2020-09-25 15:55:38 +00:00
Anny G b6286bd455 Bug 1662671 - Remove css for adding an unused resize property for Windows, r=NeilDeakin,remote-protocol-reviewers,maja_zf
Before Quantum, we needed resizer icon on Windows, because when the
add-on statusbar was hidden, we could not resize the window. This is not
needed anymore and is dead code. Consequentially, this also helps reduce
unused images loaded at startup time.

Differential Revision: https://phabricator.services.mozilla.com/D91196
2020-09-25 00:10:23 +00:00
Henrik Skupin 4e1c3a4561 Bug 1666499 - [remote] Updated expectation status for test "Page.close should *not* run beforeunload by default" r=remote-protocol-reviewers,maja_zf
Differential Revision: https://phabricator.services.mozilla.com/D91022
2020-09-23 04:33:47 +00:00
Maja Frydrychowicz 1beed99193 Bug 1666325 - Update docs about remote agent preferences r=remote-protocol-reviewers,whimboo DONTBUILD
...and remove outdated examples.

Differential Revision: https://phabricator.services.mozilla.com/D90905
2020-09-22 15:46:35 +00:00
James Graham 0e2ebe15cb Bug 1660893 - Don't crash when the remote agent can't be initialized, r=remote-protocol-reviewers,whimboo
Set the command line handler to null when we can't start the service

Differential Revision: https://phabricator.services.mozilla.com/D90463
2020-09-16 19:51:54 +00:00
Henrik Skupin 13f06a6d95 Bug 1661390 - [remote] Use a UUID as session id for targets. r=remote-protocol-reviewers,jgraham
A session is defined as string in CDP but target specific commands
and events currently return a number based on the last used index.
This also makes it very easy to discover active sessions.

Differential Revision: https://phabricator.services.mozilla.com/D89352
2020-09-07 15:31:27 +00:00
Henrik Skupin 1a57f61d15 Bug 1662687 - [puppeteer] Update expectation data for Frame.waitForFunction should work when resolved right before execution context disposal. r=remote-protocol-reviewers,maja_zf
Differential Revision: https://phabricator.services.mozilla.com/D89127
2020-09-02 13:07:45 +00:00
Henrik Skupin 3feec2e0e0 Bug 1662549 - [puppeteer] Don't set "toolkit.telemetry.server" to prevent side-effects. r=remote-protocol-reviewers,maja_zf
Differential Revision: https://phabricator.services.mozilla.com/D89057
2020-09-01 21:43:41 +00:00
Kris Maglione 3af8c4138c Bug 1651519: Remove nsIDOMWindowUtils::currentInnerWindowID. r=nika,remote-protocol-reviewers,webcompat-reviewers,whimboo
It has some properties which make it footgunny, especially in the face of
Fission. Callers should use WindowGlobalChild.innerWindowId instead.

Differential Revision: https://phabricator.services.mozilla.com/D82801
2020-08-17 20:20:50 +00:00
Kershaw Chang cb8dd8e93e Bug 1640105 - Skip failed tests, r=dragana,remote-protocol-reviewers,necko-reviewers,maja_zf
Differential Revision: https://phabricator.services.mozilla.com/D86224
2020-08-10 09:20:35 +00:00
Maja Frydrychowicz e9022983f1 Bug 1632722 - IO.read hits 'too many arguments' r=remote-protocol-reviewers,jgraham
Differential Revision: https://phabricator.services.mozilla.com/D86262
2020-08-07 09:08:01 +00:00
Ricky Stewart 31755b431d Bug 1657650 - Require that Mach command providers subclass MachCommandBase. r=remote-protocol-reviewers,marionette-reviewers,maja_zf,mhentges,froydnj
Today we don't require that `mach` `CommandProvider`s subclass from any particular parent class and we're very lax about the requirements they must meet. While that's convenient in certain circumstances, it has some unfortunate implications for feature development.

Today the only requirements that we have for `CommandProvider`s are that they have an `__init__()` method that takes either 1 or 2 arguments, the second of which must be called `context` and is populated with the `mach` `CommandContext`. Again, while this flexibility is occasionally convenient, it is limiting. As we add features to `mach`, having a better idea what the shape of our `CommandProvider`s are and how we can instantiate them and use them is increasingly important, and this gives us additional control when having `mach` configure `CommandProvider`s based on data that is only available at the `mach` level. In particular, we plan to leverage this in bugs 985141 and 1654074.

Here we add validation to the `CommandProvider` decorator to ensure all classes inherit from `MachCommandBase`, update all `CommandProvider`s in-tree to inherit from `MachCommandBase`, and update source and test code accordingly.

Follow-up work: we now require (de facto) that the `context` be populated with a `topdir` attribute by the `populate_context_handler` function, since instantiating the `MachCommandBase` requires a `topdir` be provided. This is fine for now in the interest of keeping this patch reasonably sized, but some additional refactoring could make this cleaner.

Differential Revision: https://phabricator.services.mozilla.com/D86255
2020-08-07 18:24:59 +00:00
Anny Gakhokidze d6f531d3a5 Bug 1654922 - Part 2: Remove DocumentChannel pref from mach and mochitest tools, r=mattwoodrow,remote-protocol-reviewers,maja_zf
Depends on D85483

Differential Revision: https://phabricator.services.mozilla.com/D85484
2020-07-31 13:06:08 +00:00
Jonathan Watt b7e932192e Bug 1652270. Convert nsFrameLoader::Print to return a Promise. r=farre,remote-protocol-reviewers,marionette-reviewers,jgraham,whimboo,mixedpuppy
(Instead of requiring callers to pass an nsIWebProgressListener.)

Differential Revision: https://phabricator.services.mozilla.com/D83249
2020-07-22 23:17:45 +00:00
Narcis Beleuzu 687f235fbc Backed out changeset f95c19e45ab2 (bug 1652270) for reftest failures on test-text.html . CLOSED TREE 2020-07-22 20:36:53 +03:00
Jonathan Watt 8c0fb25348 Bug 1652270. Convert nsFrameLoader::Print to return a Promise. r=farre,remote-protocol-reviewers,marionette-reviewers,jgraham,whimboo,mixedpuppy
(Instead of requiring callers to pass an nsIWebProgressListener.)

Differential Revision: https://phabricator.services.mozilla.com/D83249
2020-07-22 15:54:01 +00:00
Dorel Luca 990d081d9f Backed out 2 changesets (bug 1651519) for WPT failures in webdriver/tests/switch_to_frame/switch.py. CLOSED TREE
Backed out changeset 7b973a0e7be9 (bug 1651519)
Backed out changeset 55e9184c5617 (bug 1651519)
2020-07-22 02:47:34 +03:00
Kris Maglione e7d942c77f Bug 1651519: Remove nsIDOMWindowUtils::currentInnerWindowID. r=nika,remote-protocol-reviewers,webcompat-reviewers,whimboo
It has some properties which make it footgunny, especially in the face of
Fission. Callers should use WindowGlobalChild.innerWindowId instead.

Differential Revision: https://phabricator.services.mozilla.com/D82801
2020-07-21 19:02:27 +00:00
Dorel Luca d91c6dc40b Backed out 2 changesets (bug 1651519) for wpt failures in webdriver/tests/switch_to_frame/switch.py. CLOSED TREE
Backed out changeset 52317abc5c68 (bug 1651519)
Backed out changeset 45448dca59a4 (bug 1651519)
2020-07-21 21:55:57 +03:00
Kris Maglione 9c2c147159 Bug 1651519: Remove nsIDOMWindowUtils::currentInnerWindowID. r=nika,remote-protocol-reviewers,webcompat-reviewers,whimboo
It has some properties which make it footgunny, especially in the face of
Fission. Callers should use WindowGlobalChild.innerWindowId instead.

Differential Revision: https://phabricator.services.mozilla.com/D82801
2020-07-21 16:59:28 +00:00
Scott 2bcdc17387 Bug 1649883 - Fix user pref for pocket toggle off. r=gvn,remote-protocol-reviewers,perftest-reviewers,maja_zf
Differential Revision: https://phabricator.services.mozilla.com/D82773
2020-07-15 14:51:02 +00:00
Kris Maglione b6be72f351 Bug 1649554: Part 1 - Move deprecated component registration helpers out of XPCOMUtils. r=mccr8,remote-protocol-reviewers,marionette-reviewers,perftest-reviewers,webcompat-reviewers,whimboo,twisniewski
Differential Revision: https://phabricator.services.mozilla.com/D81749
2020-07-10 23:58:30 +00:00
Kris Maglione 9d78661f88 Bug 1649221: Update ChromeUtils.generateQI callers to pass strings. r=mccr8,remote-protocol-reviewers,marionette-reviewers,perftest-reviewers,webcompat-reviewers,geckoview-reviewers,preferences-reviewers,agi,whimboo,Bebe,twisniewski
Differential Revision: https://phabricator.services.mozilla.com/D81594
2020-07-10 23:58:28 +00:00
Andrea Marchesini 3c1b71a6e4 Bug 1651618 - Use cookie 'SameSite' attribute with capital 'S' everywhere, r=flod,fluent-reviewers,remote-protocol-reviewers,marionette-reviewers,perftest-reviewers,whimboo,sparky
Differential Revision: https://phabricator.services.mozilla.com/D82872
2020-07-09 20:40:15 +00:00
Maja Frydrychowicz 0856418749 Bug 1651469 - [puppeteer-test] Report errors in mocha hooks r=remote-protocol-reviewers,whimboo
Differential Revision: https://phabricator.services.mozilla.com/D82813
2020-07-09 14:01:50 +00:00
Henrik Skupin 74a6bd97cd Bug 1644349 - [remote] Remove usage of windowIdToFrameIdMap in the Network observer. r=remote-protocol-reviewers,maja_zf
Differential Revision: https://phabricator.services.mozilla.com/D82691
2020-07-08 19:55:45 +00:00
Henrik Skupin f8fbd2e85e Bug 1651312 - [remote] Fix invalid data type checks in browser-chrome tests. r=remote-protocol-reviewers,maja_zf
Differential Revision: https://phabricator.services.mozilla.com/D82678
2020-07-08 13:47:22 +00:00
Maja Frydrychowicz 24c63a2d0c Bug 1647439 - Sync Puppeteer to v5.0.0 r=remote-protocol-reviewers,jgraham
Differential Revision: https://phabricator.services.mozilla.com/D82072
2020-07-02 18:57:38 +00:00