[webkit-changes] [WebKit/WebKit] bd0c3f: [visionOS] Under thermal throttling Safari webpage...

mwyrzykowski noreply at github.com
Tue Sep 19 14:54:03 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: bd0c3f38c2e8a609fe52a4c87e2fd4ee2e6fe06b
      https://github.com/WebKit/WebKit/commit/bd0c3f38c2e8a609fe52a4c87e2fd4ee2e6fe06b
  Author: Mike Wyrzykowski <mwyrzykowski at apple.com>
  Date:   2023-09-19 (Tue, 19 Sep 2023)

  Changed paths:
    M Source/WebCore/WebCore.xcodeproj/project.pbxproj
    M Source/WebCore/platform/ThermalMitigationNotifier.cpp
    M Source/WebCore/platform/ThermalMitigationNotifier.h
    M Source/WebCore/platform/cocoa/ThermalMitigationNotifier.mm
    M Source/WebKit/UIProcess/AuxiliaryProcessProxy.cpp

  Log Message:
  -----------
  [visionOS] Under thermal throttling Safari webpage went blank when navigating
https://bugs.webkit.org/show_bug.cgi?id=261374
<radar://113230434>

Reviewed by Dean Jackson.

When the system is throttled due to thermal pressure, we can easily hit process
timeouts and then web pages will fail to load.

* Source/WebKit/UIProcess/AuxiliaryProcessProxy.cpp:
(WebKit::adjustedTimeoutForThermalState):
(WebKit::AuxiliaryProcessProxy::AuxiliaryProcessProxy):
Query if thermal mitigation is enabled and increase process timeout if so.

* Source/WebCore/platform/ThermalMitigationNotifier.cpp:
(WebCore::ThermalMitigationNotifier::isThermalMitigationEnabled):
Default implementation.

* Source/WebCore/platform/ThermalMitigationNotifier.h:
* Source/WebCore/platform/cocoa/ThermalMitigationNotifier.mm:
(WebCore::isThermalMitigationEnabled):
(-[WebThermalMitigationObserver thermalMitigationEnabled]):
(WebCore::ThermalMitigationNotifier::isThermalMitigationEnabled):
Add static member function to query if thermal mitigation is enabled
as we don't need an instance function in AuxiliaryProcessProxy

* Source/WebCore/WebCore.xcodeproj/project.pbxproj:
Make ThermalMitigationNotifier.h a private header.

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




More information about the webkit-changes mailing list