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.

Sign up for GitHub

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

Jump to bottom

Avoid double HashMap lookup in RemoteResourceCacheProxy::recordNativeImageUse() #18168

Merged
merged 1 commit into from
Sep 25, 2023
Merged

Avoid double HashMap lookup in RemoteResourceCacheProxy::recordNativeImageUse() #18168

merged 1 commit into from
Sep 25, 2023

Conversation

cdumez
Copy link
Contributor

@cdumez cdumez commented Sep 25, 2023 •
edited by webkit-early-warning-system
Loading

b458477

Avoid double HashMap lookup in RemoteResourceCacheProxy::recordNativeImageUse()
https://bugs.webkit.org/show_bug.cgi?id=262048

Reviewed by Simon Fraser.

* Source/WebKit/WebProcess/GPU/graphics/RemoteResourceCacheProxy.cpp:
(WebKit::RemoteResourceCacheProxy::recordNativeImageUse):

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

4e496f7

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
✅ 🧪 ios-wk2-wpt ✅ 🧪 gtk-wk2
✅ 🧪 api-ios ✅ 🧪 mac-wk2 ✅ 🧪 api-gtk
✅ 🛠 tv ✅ 🧪 mac-AS-debug-wk2
✅ 🛠 tv-sim
✅ 🛠 🧪 merge ✅ 🛠 watch
✅ 🛠 watch-sim

@cdumez cdumez self-assigned this Sep 25, 2023
@cdumez cdumez added the Layout and Rendering For bugs with layout and rendering of Web pages. label Sep 25, 2023
@webkit-early-warning-system
Copy link
Collaborator

webkit-early-warning-system commented Sep 25, 2023 •
edited
Loading

EWS run on current version of this PR (hash 4e496f7)

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
✅ 🧪 ios-wk2-wpt ✅ 🧪 gtk-wk2
✅ 🧪 api-ios ✅ 🧪 mac-wk2 ✅ 🧪 api-gtk
✅ 🛠 tv ✅ 🧪 mac-AS-debug-wk2
✅ 🛠 tv-sim
✅ 🛠 🧪 merge ✅ 🛠 watch
✅ 🛠 watch-sim

@cdumez cdumez marked this pull request as ready for review September 25, 2023 18:43
@cdumez cdumez added the merge-queue Applied to send a pull request to merge-queue label Sep 25, 2023
@cdumez
Avoid double HashMap lookup in RemoteResourceCacheProxy::recordNative…
b458477 
…ImageUse()

https://bugs.webkit.org/show_bug.cgi?id=262048

Reviewed by Simon Fraser.

* Source/WebKit/WebProcess/GPU/graphics/RemoteResourceCacheProxy.cpp:
(WebKit::RemoteResourceCacheProxy::recordNativeImageUse):

Canonical link: https://commits.webkit.org/268410@main
@webkit-commit-queue
Copy link
Collaborator

Committed 268410@main (b458477): https://commits.webkit.org/268410@main

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

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

@smfr smfr smfr approved these changes

@shallawa shallawa Awaiting requested review from shallawa

@litherum litherum Awaiting requested review from litherum

Assignees

@cdumez cdumez

Labels
Layout and Rendering For bugs with layout and rendering of Web pages.
Projects
None yet
Milestone
No milestone
4 participants
@cdumez @webkit-early-warning-system @webkit-commit-queue @smfr