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

[Gecko Bug 1865618] [wdspec] Rename test module set_viewport.py to viewport.py. #43342

Merged
merged 3 commits into from Nov 26, 2023

Conversation

moz-wptsync-bot
Copy link
Collaborator

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1865618
gecko-commit: 65228ad8a272389c4048f39b7d468b19a284e3b3
gecko-reviewers: webdriver-reviewers, Sasha

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1865618
gecko-commit: 65228ad8a272389c4048f39b7d468b19a284e3b3
gecko-reviewers: webdriver-reviewers, Sasha
…cePixelRatio" tests.

Tests were not failing as expected because of the missing "viewport"
argument.

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1865618
gecko-commit: a9f154bc28c739143d8d3226aa305bf6cc184566
gecko-reviewers: webdriver-reviewers, Sasha
Without an actual page loaded, changing the viewport will not trigger
any reflow and as such doesn't test real use-case scenarios.

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1865618
gecko-commit: 988211aadb0d43726c920e868dfa7dd055f7fa77
gecko-reviewers: webdriver-reviewers, Sasha
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Firefox project.

@moz-wptsync-bot moz-wptsync-bot merged commit b19c370 into master Nov 26, 2023
19 checks passed
@moz-wptsync-bot moz-wptsync-bot deleted the gecko/1865618 branch November 26, 2023 10:35
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.

None yet

3 participants