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

Adopt more smart pointers in AuxiliaryProcessProxy.cpp #23126

Conversation

cdumez
Copy link
Contributor

@cdumez cdumez commented Jan 23, 2024

86fd607

Adopt more smart pointers in AuxiliaryProcessProxy.cpp
https://bugs.webkit.org/show_bug.cgi?id=267958

Reviewed by Darin Adler.

* Source/WebKit/UIProcess/AuxiliaryProcessProxy.cpp:
(WebKit::AuxiliaryProcessProxy::~AuxiliaryProcessProxy):
(WebKit::AuxiliaryProcessProxy::terminate):
(WebKit::AuxiliaryProcessProxy::sendMessage):
(WebKit::AuxiliaryProcessProxy::didFinishLaunching):
(WebKit::AuxiliaryProcessProxy::shutDownProcess):
(WebKit::AuxiliaryProcessProxy::setProcessSuppressionEnabled):
(WebKit::AuxiliaryProcessProxy::checkForResponsiveness):

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

98f0d4d

Misc iOS, tvOS & watchOS macOS Linux Windows
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe ✅ 🛠 wincairo
✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug 🧪 wpe-wk2
✅ 🧪 webkitperl ✅ 🧪 ios-wk2 ✅ 🧪 api-mac ✅ 🧪 api-wpe
✅ 🧪 ios-wk2-wpt ✅ 🛠 gtk
✅ 🧪 api-ios ✅ 🧪 mac-wk2 ✅ 🧪 gtk-wk2
✅ 🛠 tv ✅ 🧪 mac-AS-debug-wk2 ✅ 🧪 api-gtk
🛠 tv-sim
✅ 🛠 watch
✅ 🛠 🧪 unsafe-merge ✅ 🛠 watch-sim

@cdumez cdumez self-assigned this Jan 23, 2024
@cdumez cdumez added the WebKit2 Bugs relating to the WebKit2 API layer label Jan 23, 2024
@cdumez cdumez marked this pull request as ready for review January 24, 2024 03:36
@cdumez cdumez added the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Jan 24, 2024
https://bugs.webkit.org/show_bug.cgi?id=267958

Reviewed by Darin Adler.

* Source/WebKit/UIProcess/AuxiliaryProcessProxy.cpp:
(WebKit::AuxiliaryProcessProxy::~AuxiliaryProcessProxy):
(WebKit::AuxiliaryProcessProxy::terminate):
(WebKit::AuxiliaryProcessProxy::sendMessage):
(WebKit::AuxiliaryProcessProxy::didFinishLaunching):
(WebKit::AuxiliaryProcessProxy::shutDownProcess):
(WebKit::AuxiliaryProcessProxy::setProcessSuppressionEnabled):
(WebKit::AuxiliaryProcessProxy::checkForResponsiveness):

Canonical link: https://commits.webkit.org/273387@main
@webkit-commit-queue webkit-commit-queue force-pushed the 267958_AuxiliaryProcessProxy_smart_pointers branch from 98f0d4d to 86fd607 Compare January 24, 2024 04:21
@webkit-commit-queue
Copy link
Collaborator

Committed 273387@main (86fd607): https://commits.webkit.org/273387@main

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

@webkit-commit-queue webkit-commit-queue merged commit 86fd607 into WebKit:main Jan 24, 2024
@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 Jan 24, 2024
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
4 participants