Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Regression(262025@main) WebProcesses on macOS 13.0 are still marked as managed by RunningBoard #13599

Conversation

cdumez
Copy link
Contributor

@cdumez cdumez commented May 8, 2023

99c9a20

Regression(262025@main) WebProcesses on macOS 13.0 are still marked as managed by RunningBoard
https://bugs.webkit.org/show_bug.cgi?id=256489
rdar://108417115

Reviewed by Geoffrey Garen and Ben Nham.

WebProcesses on macOS 13.0 are still marked as managed by RunningBoard even
though 262025@main made it so that we only use RunningBoard assertions on macOS
13.3+.

USE(RUNNINGBOARD) requires `__MAC_OS_X_VERSION_MIN_REQUIRED >= 130300` after
262025@main but update-info-plist-for-runningboard.sh was still checking for
130000. This meant RunningBoard was managing process priority on macOS 13.0+
even though we are only taking RunningBoard assertion on macOS 13.3+. This
was causing our processes to have low priority.

* Source/WebKit/Scripts/update-info-plist-for-runningboard.sh:

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

5baef03

Misc iOS, tvOS & watchOS macOS Linux Windows
βœ… πŸ§ͺ style βœ… πŸ›  ios βœ… πŸ›  mac βœ… πŸ›  wpe   πŸ›  wincairo
βœ… πŸ›  ios-sim βœ… πŸ›  mac-AS-debug   πŸ§ͺ wpe-wk2
βœ… πŸ§ͺ webkitperl   πŸ§ͺ ios-wk2   πŸ§ͺ api-mac βœ… πŸ›  gtk
βœ… πŸ§ͺ webkitpy   πŸ§ͺ ios-wk2-wpt   πŸ§ͺ gtk-wk2
  πŸ§ͺ api-ios   πŸ§ͺ mac-wk2   πŸ§ͺ api-gtk
βœ… πŸ›  tv   πŸ§ͺ mac-AS-debug-wk2
  πŸ›  tv-sim
  πŸ›  πŸ§ͺ merge βœ… πŸ›  watch
βœ… πŸ›  πŸ§ͺ unsafe-merge   πŸ›  watch-sim

@cdumez cdumez self-assigned this May 8, 2023
@cdumez cdumez added the WebKit2 Bugs relating to the WebKit2 API layer label May 8, 2023
@cdumez cdumez force-pushed the 256489_processes_managed_by_runningboard branch from a2ca13c to 6b542ce Compare May 8, 2023 21:38
@cdumez cdumez force-pushed the 256489_processes_managed_by_runningboard branch from 6b542ce to 5baef03 Compare May 8, 2023 21:45
@cdumez cdumez requested review from bnham and geoffreygaren May 8, 2023 21:46
Copy link
Contributor

@geoffreygaren geoffreygaren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r=me

@cdumez cdumez added the merge-queue Applied to send a pull request to merge-queue label May 8, 2023
@cdumez cdumez added unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing and removed merge-queue Applied to send a pull request to merge-queue labels May 8, 2023
…s managed by RunningBoard

https://bugs.webkit.org/show_bug.cgi?id=256489
rdar://108417115

Reviewed by Geoffrey Garen and Ben Nham.

WebProcesses on macOS 13.0 are still marked as managed by RunningBoard even
though 262025@main made it so that we only use RunningBoard assertions on macOS
13.3+.

USE(RUNNINGBOARD) requires `__MAC_OS_X_VERSION_MIN_REQUIRED >= 130300` after
262025@main but update-info-plist-for-runningboard.sh was still checking for
130000. This meant RunningBoard was managing process priority on macOS 13.0+
even though we are only taking RunningBoard assertion on macOS 13.3+. This
was causing our processes to have low priority.

* Source/WebKit/Scripts/update-info-plist-for-runningboard.sh:

Canonical link: https://commits.webkit.org/263830@main
@webkit-commit-queue webkit-commit-queue force-pushed the 256489_processes_managed_by_runningboard branch from 5baef03 to 99c9a20 Compare May 8, 2023 22:12
@webkit-commit-queue
Copy link
Collaborator

Committed 263830@main (99c9a20): https://commits.webkit.org/263830@main

Reviewed commits have been landed. Closing PR #13599 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit 99c9a20 into WebKit:main May 8, 2023
@webkit-commit-queue webkit-commit-queue removed the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label May 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WebKit2 Bugs relating to the WebKit2 API layer
Projects
None yet
5 participants