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

orientation.unlock doesn't return promise #37097

Merged

Conversation

makotokato
Copy link
Member

hidden_document.html uses promise_rejects_dom for orientation.unlock, but it doesn't return promise. So we should use assert_throws_dom instead.

Also, since this test has some issues, this fix includes it

  • We don't have screen-orientation/resources/orientation_.js
  • Import orientation-utils.js to use getOppositeOrientation
  • Typo fix of make_cleanup.

`hidden_document.html` uses `promise_rejects_dom` for `orientation.unlock`, but
it doesn't return promise. So we should use `assert_throws_dom` instead.

Also, since this test has some issues, this fix includes it
- We don't have `screen-orientation/resources/orientation_.js`
- Import orientation-utils.js to use `getOppositeOrientation`
- Typo fix of `make_cleanup`.
@marcoscaceres marcoscaceres merged commit 181bb8b into web-platform-tests:master Nov 23, 2022
@marcoscaceres
Copy link
Contributor

Thanks for fixing these @makotokato! Not sure how these got through.

@engedy
Copy link

engedy commented Nov 29, 2022

Chromium Build Sheriff here, we are wondering if these changes could have resulted in non-deterministic harness errors we are observing with screen-orientation/hidden_document.html, see https://crbug.com/1393567.

@makotokato If you have a bit of time, could you please take a look?

@makotokato makotokato deleted the fix-hidden_document branch November 29, 2022 12:15
@makotokato
Copy link
Member Author

Chromium Build Sheriff here, we are wondering if these changes could have resulted in non-deterministic harness errors we are observing with screen-orientation/hidden_document.html, see https://crbug.com/1393567.

@makotokato If you have a bit of time, could you please take a look?

This PR is a fix of broken test, and this is for w3c/screen-orientation#232. You should ask @michaelwasserman or WebDriver team for it. (This test uses test_driver.set_window_rect and test_driver.minimize_window).

@engedy
Copy link

engedy commented Nov 29, 2022

Thank you, I will follow up with @michaelwasserman

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants