Skip to content

[WebGPU] Remove unused member variables and member functions#33078

Merged
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
mwyrzykowski:eng/WebGPU-Remove-unused-member-variables-and-member-functions
Sep 4, 2024
Merged

[WebGPU] Remove unused member variables and member functions#33078
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
mwyrzykowski:eng/WebGPU-Remove-unused-member-variables-and-member-functions

Conversation

@mwyrzykowski
Copy link
Contributor

@mwyrzykowski mwyrzykowski commented Sep 3, 2024

61424ee

[WebGPU] Remove unused member variables and member functions
https://bugs.webkit.org/show_bug.cgi?id=279074
rdar://135208062

Reviewed by Dan Glastonbury.

Audit for retain cycles revealed some unused member variables
and functions, remove these.

* Source/WebCore/Modules/WebGPU/GPU.h:
* Source/WebCore/Modules/WebGPU/InternalAPI/WebGPUDevice.h:
* Source/WebGPU/WebGPU/RenderPassEncoder.h:
* Source/WebGPU/WebGPU/RenderPassEncoder.mm:
(WebGPU::m_maxDrawCount):
(WebGPU::RenderPassEncoder::endPass):
* Source/WebKit/WebProcess/GPU/graphics/WebGPU/RemoteAdapterProxy.h:
* Source/WebKit/WebProcess/GPU/graphics/WebGPU/RemoteBufferProxy.h:
* Source/WebKit/WebProcess/GPU/graphics/WebGPU/RemoteDeviceProxy.h:
* Source/WebKit/WebProcess/GPU/graphics/WebGPU/RemoteGPUProxy.h:
* Source/WebKit/WebProcess/GPU/graphics/WebGPU/RemoteQueueProxy.h:

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

405da1a

Misc iOS, visionOS, tvOS & watchOS macOS Linux Windows
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe ✅ 🛠 wincairo
✅ 🧪 bindings ✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug ✅ 🧪 wpe-wk2 ✅ 🧪 wincairo-tests
✅ 🧪 webkitperl ✅ 🧪 ios-wk2 ✅ 🧪 api-mac ✅ 🧪 api-wpe
✅ 🧪 ios-wk2-wpt ✅ 🧪 mac-wk1 ✅ 🛠 wpe-cairo
✅ 🧪 api-ios ✅ 🧪 mac-wk2 ✅ 🛠 gtk
✅ 🛠 vision 🧪 mac-AS-debug-wk2 ✅ 🧪 gtk-wk2
✅ 🛠 vision-sim ✅ 🧪 mac-wk2-stress 🧪 api-gtk
✅ 🛠 🧪 merge ✅ 🧪 vision-wk2 ✅ 🧪 mac-intel-wk2
✅ 🛠 tv
✅ 🛠 tv-sim
✅ 🛠 watch
✅ 🛠 watch-sim

@mwyrzykowski mwyrzykowski self-assigned this Sep 3, 2024
@mwyrzykowski mwyrzykowski added the WebGPU For bugs in WebGPU label Sep 3, 2024
@mwyrzykowski mwyrzykowski requested a review from djg September 3, 2024 18:24
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Sep 3, 2024
@mwyrzykowski mwyrzykowski removed the merging-blocked Applied to prevent a change from being merged label Sep 3, 2024
@mwyrzykowski mwyrzykowski force-pushed the eng/WebGPU-Remove-unused-member-variables-and-member-functions branch from f698df9 to 405da1a Compare September 3, 2024 18:32
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Sep 3, 2024
Copy link
Contributor

@djg djg left a comment

Choose a reason for hiding this comment

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

:shipit:

@mwyrzykowski mwyrzykowski added merge-queue Applied to send a pull request to merge-queue and removed merging-blocked Applied to prevent a change from being merged labels Sep 4, 2024
https://bugs.webkit.org/show_bug.cgi?id=279074
rdar://135208062

Reviewed by Dan Glastonbury.

Audit for retain cycles revealed some unused member variables
and functions, remove these.

* Source/WebCore/Modules/WebGPU/GPU.h:
* Source/WebCore/Modules/WebGPU/InternalAPI/WebGPUDevice.h:
* Source/WebGPU/WebGPU/RenderPassEncoder.h:
* Source/WebGPU/WebGPU/RenderPassEncoder.mm:
(WebGPU::m_maxDrawCount):
(WebGPU::RenderPassEncoder::endPass):
* Source/WebKit/WebProcess/GPU/graphics/WebGPU/RemoteAdapterProxy.h:
* Source/WebKit/WebProcess/GPU/graphics/WebGPU/RemoteBufferProxy.h:
* Source/WebKit/WebProcess/GPU/graphics/WebGPU/RemoteDeviceProxy.h:
* Source/WebKit/WebProcess/GPU/graphics/WebGPU/RemoteGPUProxy.h:
* Source/WebKit/WebProcess/GPU/graphics/WebGPU/RemoteQueueProxy.h:

Canonical link: https://commits.webkit.org/283127@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/WebGPU-Remove-unused-member-variables-and-member-functions branch from 405da1a to 61424ee Compare September 4, 2024 02:57
@webkit-commit-queue
Copy link
Collaborator

Committed 283127@main (61424ee): https://commits.webkit.org/283127@main

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

@webkit-commit-queue webkit-commit-queue merged commit 61424ee into WebKit:main Sep 4, 2024
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

WebGPU For bugs in WebGPU

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants