Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "browser(firefox): instrument all windows, support silent mode… #1615

Merged
merged 2 commits into from
Apr 1, 2020

Conversation

yury-s
Copy link
Member

@yury-s yury-s commented Apr 1, 2020

… (#1612)"

This reverts commit 314eb40.

Tests are hanging on Linux:

Testing on Node v12.6.0
WARN: running Firefox tests with /home/yurys/playwright/browser_patches/firefox/checkout/obj-build-playwright/dist/bin/firefox
Running 1 focused tests out of total 984 on 20 workers

Focused Suites and Tests:

  1. Firefox [Driver] Page.evaluate should work (evaluation.spec.js:31:5)

pw:browser:proc:[1] /home/yurys/playwright/browser_patches/firefox/checkout/obj-build-playwright/dist/bin/firefox -no-remote -headless -profile /tmp/playwright_dev_firefox_profile-WlBvTh -juggler 0 about:blank +0ms
pw:browser:err:[1] *** You are running in headless mode. +0ms
pw:browser:out:[1] 1585699649772 [email protected] WARN Loading extension '[email protected]': Reading manifest: Invalid extension permission: networkStatus +0ms
pw:browser:out:[1] 1585699650540 [email protected] WARN Loading extension '[email protected]': Reading manifest: Invalid extension permission: mozillaAddons +768ms
pw:browser:out:[1] 1585699650541 [email protected] WARN Loading extension '[email protected]': Reading manifest: Invalid extension permission: telemetry +0ms
pw:browser:out:[1] 1585699650541 [email protected] WARN Loading extension '[email protected]': Reading manifest: Invalid extension permission: resource://pdf.js/ +0ms
pw:browser:out:[1] 1585699650541 [email protected] WARN Loading extension '[email protected]': Reading manifest: Invalid extension permission: about:reader* +0ms
pw:browser:out:[1] Juggler listening on ws://127.0.0.1:33883/a943e269-48d9-494c-819f-ab533724739d +171ms
pw:protocol SEND ► {"id":1,"method":"Browser.enable","params":{"attachToDefaultContext":false}} +0ms

Failures:

  1. [worker = null] [CRASHED] Firefox [Driver] Page.evaluate should work (evaluation.spec.js:31:5)

…microsoft#1612)"

This reverts commit 314eb40.

Tests are hanging on Linux:

Testing on Node v12.6.0
WARN: running Firefox tests with /home/yurys/playwright/browser_patches/firefox/checkout/obj-build-playwright/dist/bin/firefox
Running 1 focused tests out of total 984 on 20 workers

Focused Suites and Tests:
  1) Firefox [Driver] Page.evaluate should work (evaluation.spec.js:31:5)

  pw:browser:proc:[1] <launching> /home/yurys/playwright/browser_patches/firefox/checkout/obj-build-playwright/dist/bin/firefox -no-remote -headless -profile /tmp/playwright_dev_firefox_profile-WlBvTh -juggler 0 about:blank +0ms
  pw:browser:err:[1] *** You are running in headless mode. +0ms
  pw:browser:out:[1] 1585699649772      [email protected]     WARN    Loading extension '[email protected]': Reading manifest: Invalid extension permission: networkStatus +0ms
  pw:browser:out:[1] 1585699650540      [email protected]     WARN    Loading extension '[email protected]': Reading manifest: Invalid extension permission: mozillaAddons +768ms
  pw:browser:out:[1] 1585699650541      [email protected]     WARN    Loading extension '[email protected]': Reading manifest: Invalid extension permission: telemetry +0ms
  pw:browser:out:[1] 1585699650541      [email protected]     WARN    Loading extension '[email protected]': Reading manifest: Invalid extension permission: resource://pdf.js/ +0ms
  pw:browser:out:[1] 1585699650541      [email protected]     WARN    Loading extension '[email protected]': Reading manifest: Invalid extension permission: about:reader* +0ms
  pw:browser:out:[1] Juggler listening on ws://127.0.0.1:33883/a943e269-48d9-494c-819f-ab533724739d +171ms
  pw:protocol SEND ► {"id":1,"method":"Browser.enable","params":{"attachToDefaultContext":false}} +0ms

Failures:
1) [worker = null] [CRASHED] Firefox [Driver] Page.evaluate should work (evaluation.spec.js:31:5)
@yury-s yury-s requested a review from dgozman April 1, 2020 00:11
@yury-s yury-s merged commit 2ce85f9 into microsoft:master Apr 1, 2020
@yury-s yury-s deleted the revert-ff branch April 1, 2020 00:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants