Skip to content

Conversation

@hortont424
Copy link
Contributor

@hortont424 hortont424 commented Jan 9, 2023

6e41546

REGRESSION (258521@main): Can end up scrolled to an invalid offset, blank space above page
https://bugs.webkit.org/show_bug.cgi?id=250335
rdar://103977580

Reviewed by Dean Jackson and Megan Gardner.

* Source/WebKit/UIProcess/API/ios/WKWebViewIOS.mm:
(-[WKWebView _updateScrollViewForTransaction:]):
Ensure that we don't scroll to an invalid offset, the same way we do elsewhere.

Canonical link: https://commits.webkit.org/258687@main

86827bd

Misc iOS, tvOS & watchOS macOS Linux Windows
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe
✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug ✅ 🛠 gtk 🛠 wincairo
✅ 🧪 webkitperl 🧪 ios-wk2 🧪 api-mac 🧪 gtk-wk2
🧪 api-ios ✅ 🧪 mac-wk1 🧪 api-gtk
✅ 🛠 tv ✅ 🧪 mac-wk2
✅ 🛠 tv-sim ✅ 🧪 mac-AS-debug-wk2
✅ 🛠 watch ✅ 🧪 mac-wk2-stress
✅ 🛠 🧪 merge ✅ 🛠 watch-sim

@hortont424 hortont424 requested a review from cdumez as a code owner January 9, 2023 20:45
@hortont424 hortont424 self-assigned this Jan 9, 2023
@hortont424 hortont424 added the WebKit Misc. For miscellaneous bugs in the WebKit framework (and not JavaScriptCore or WebCore). label Jan 9, 2023
@hortont424 hortont424 added the merge-queue Applied to send a pull request to merge-queue label Jan 9, 2023
…lank space above page

https://bugs.webkit.org/show_bug.cgi?id=250335
rdar://103977580

Reviewed by Dean Jackson and Megan Gardner.

* Source/WebKit/UIProcess/API/ios/WKWebViewIOS.mm:
(-[WKWebView _updateScrollViewForTransaction:]):
Ensure that we don't scroll to an invalid offset, the same way we do elsewhere.

Canonical link: https://commits.webkit.org/258687@main
@webkit-early-warning-system webkit-early-warning-system force-pushed the eng/REGRESSION-258521main-Can-end-up-scrolled-to-an-invalid-offset-blank-space-above-page branch from 86827bd to 6e41546 Compare January 9, 2023 22:54
@webkit-early-warning-system webkit-early-warning-system merged commit 6e41546 into WebKit:main Jan 9, 2023
@webkit-commit-queue
Copy link
Collaborator

Committed 258687@main (6e41546): https://commits.webkit.org/258687@main

Reviewed commits have been landed. Closing PR #8412 and removing active labels.

@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Jan 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

WebKit Misc. For miscellaneous bugs in the WebKit framework (and not JavaScriptCore or WebCore).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants