[webkit-changes] [WebKit/WebKit] 2f7481: Move off of UIKit SPI: -[UIScrollView _is(Vertical...

Wenson Hsieh noreply at github.com
Sun Sep 24 16:14:17 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 2f74816e605d1c2539f2613ca928bda62dd21c3c
      https://github.com/WebKit/WebKit/commit/2f74816e605d1c2539f2613ca928bda62dd21c3c
  Author: Wenson Hsieh <wenson_hsieh at apple.com>
  Date:   2023-09-24 (Sun, 24 Sep 2023)

  Changed paths:
    M Source/WebKit/Platform/spi/ios/UIKitSPI.h
    M Source/WebKit/UIProcess/ios/UIKitUtilities.h
    M Source/WebKit/UIProcess/ios/UIKitUtilities.mm
    M Source/WebKit/UIProcess/ios/WKScrollView.mm
    M Tools/WebKitTestRunner/WebKitTestRunner.xcodeproj/project.pbxproj
    M Tools/WebKitTestRunner/cocoa/TestRunnerWKWebView.mm
    A Tools/WebKitTestRunner/ios/UIKitSPIForTesting.h

  Log Message:
  -----------
  Move off of UIKit SPI: -[UIScrollView _is(Vertical|Horizontal)Bouncing]
https://bugs.webkit.org/show_bug.cgi?id=262013
rdar://112473874

Reviewed by Aditya Keerthi.

Stop using `-_isVerticalBouncing` and `-_isHorizontalBouncing` on `UIScrollView`; as an alternative,
add a new category helper method, `-[UIScrollView _wk_isScrolledBeyondExtents]`, that returns
whether or not the current content offset is scrolled beyond the minimum or maximum extents of the
scroll view, taking the `-adjustedContentInset` into account.

* Source/WebKit/Platform/spi/ios/UIKitSPI.h:
* Source/WebKit/UIProcess/ios/UIKitUtilities.h:
* Source/WebKit/UIProcess/ios/UIKitUtilities.mm:
(-[UIScrollView _wk_isScrolledBeyondExtents]):
* Source/WebKit/UIProcess/ios/WKScrollView.mm:
(-[WKScrollView _setContentSizePreservingContentOffsetDuringRubberband:]):
* Tools/WebKitTestRunner/WebKitTestRunner.xcodeproj/project.pbxproj:
* Tools/WebKitTestRunner/cocoa/TestRunnerWKWebView.mm:
* Tools/WebKitTestRunner/ios/UIKitSPIForTesting.h: Copied from Source/WebKit/UIProcess/ios/UIKitUtilities.h.

Continue using `-_isVerticalBouncing` and `-_isHorizontalBouncing` in WebKitTestRunner harness logic
to determine when scrolling/zooming animations are complete. We also add an SPI header, specific to
WebKitTestRunner for now, to house these new forward declarations and SPI include points. I'll move
various existing testing-only declarations and includes in UIKitSPI.h into this header in a
subsequent patch.

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




More information about the webkit-changes mailing list