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 1548155] Add a WPT test to ensure that getComputedStyle() round-trips when scrollbars are present. #17329

Merged
merged 1 commit into from Jun 14, 2019

Conversation

moz-wptsync-bot
Copy link
Collaborator

See the note in the test about the .tentative.html bit. I really think Gecko
behavior is sane here.

I filed https://bugs.chromium.org/p/chromium/issues/detail?id=973171 about this,
and told the Blink folks to file a csswg-drafts issue if they disagreed.

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1548155
gecko-commit: d92e38d8982377ba5197f9900d1cac0bb1948ae9
gecko-integration-branch: autoland
gecko-reviewers: bzbarsky

…ollbars are present.

See the note in the test about the .tentative.html bit. I really think Gecko
behavior is sane here.

I filed https://bugs.chromium.org/p/chromium/issues/detail?id=973171 about this,
and told the Blink folks to file a csswg-drafts issue if they disagreed.

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1548155
gecko-commit: d92e38d8982377ba5197f9900d1cac0bb1948ae9
gecko-integration-branch: autoland
gecko-reviewers: bzbarsky
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.

Already reviewed downstream.

@moz-wptsync-bot moz-wptsync-bot merged commit 79d2069 into master Jun 14, 2019
@moz-wptsync-bot moz-wptsync-bot deleted the gecko/1548155 branch June 14, 2019 03:49
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