[webkit-changes] [WebKit/WebKit] 9ed33f: WebKit shouldn't be modifying the value of UIAppli...

Chris Dumez noreply at github.com
Tue Aug 1 16:54:39 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 9ed33f8f5890bf7acc64e10e0efe32f275a5a646
      https://github.com/WebKit/WebKit/commit/9ed33f8f5890bf7acc64e10e0efe32f275a5a646
  Author: Chris Dumez <cdumez at apple.com>
  Date:   2023-08-01 (Tue, 01 Aug 2023)

  Changed paths:
    M Source/WebCore/PAL/pal/Logging.cpp
    M Source/WebCore/PAL/pal/Logging.h
    M Source/WebCore/PAL/pal/spi/ios/UIKitSPI.h
    M Source/WebCore/PAL/pal/system/ios/SleepDisablerIOS.mm

  Log Message:
  -----------
  WebKit shouldn't be modifying the value of UIApplication.idleTimerDisabled
https://bugs.webkit.org/show_bug.cgi?id=259706

Reviewed by Jer Noble.

WebKit shouldn't be modifying the value of UIApplication.idleTimerDisabled
because we're a framework. The hosting application can rely on this flag to
prevent screen sleep and it is not good for WebKit to be potentially overriding
a value set by the application.

To address the issue, the UIKit code indicates that frameworks are expected to
call the `_setIdleTimerDisabled:forReason:` SPI instead.

* Source/WebCore/PAL/pal/spi/ios/UIKitSPI.h:
* Source/WebCore/PAL/pal/system/ios/SleepDisablerIOS.mm:
(PAL::ScreenSleepDisabler::updateState):

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




More information about the webkit-changes mailing list