Navigation Menu

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

The size of OverflowClipRect is negative #19566

Merged
merged 1 commit into from Oct 8, 2019

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Oct 8, 2019

When scrollbar wider than the scrollar, the size of OverflowClipRect
is negative which will affect the value of scroll offset. This patch
make the size non-negative.

Bug: 987566
Change-Id: I7fa80e4a17bc48303713a1a191230d0b3e7d0dd5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1840492
Commit-Queue: cathie chen <cathiechen@igalia.com>
Reviewed-by: David Bokan <bokan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#703662}

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.

@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-1840492 branch 2 times, most recently from dec2f50 to 48d958a Compare October 8, 2019 10:08
When scrollbar wider than the scrollar, the size of OverflowClipRect
is negative which will affect the value of scroll offset. This patch
make the size non-negative.

Bug: 987566
Change-Id: I7fa80e4a17bc48303713a1a191230d0b3e7d0dd5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1840492
Commit-Queue: cathie chen <cathiechen@igalia.com>
Reviewed-by: David Bokan <bokan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#703662}
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