[webkit-changes] [WebKit/WebKit] 345d5c: Signal ARQL when a download fails

Dean Jackson noreply at github.com
Thu Apr 27 18:29:52 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 345d5c9e9dae6bcdf0a7cb4d9ddc34d5e921e348
      https://github.com/WebKit/WebKit/commit/345d5c9e9dae6bcdf0a7cb4d9ddc34d5e921e348
  Author: Dean Jackson <dino at apple.com>
  Date:   2023-04-27 (Thu, 27 Apr 2023)

  Changed paths:
    M Source/WebCore/PAL/pal/spi/ios/SystemPreviewSPI.h
    M Source/WebKit/UIProcess/Cocoa/SystemPreviewControllerCocoa.mm
    M Source/WebKit/UIProcess/SystemPreviewController.h

  Log Message:
  -----------
  Signal ARQL when a download fails
https://bugs.webkit.org/show_bug.cgi?id=254451
rdar://104039022

Reviewed by Tim Horton.

ARQL doesn't get told when a download fails. Add this using their new SPI.

* Source/WebCore/PAL/pal/spi/ios/SystemPreviewSPI.h: Add the new SPI.
* Source/WebKit/UIProcess/Cocoa/SystemPreviewControllerCocoa.mm:
    - Open the file handle as soon as we start downloading, so that
      we can make a URL.
    - ARQL fails if you give it a .reality file that doesn't
      have the right file extension, so add that if necessary.
    - Close the file handle if we fail the download. Also, tell
      ARQL that the load failed.
    - Lastly, make sure we always attach the fragment identifier, so
      ARQL knows that the cancel is related to a previous launch call.
* Source/WebKit/UIProcess/SystemPreviewController.h:

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




More information about the webkit-changes mailing list