[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