corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1906562. Remove useless IsPopup check in nsIFrame::GetTransformMatrix. r=layout-reviewers,emilio

Browse Source 
Bug 1801025 removed any special handling for popups in this function, but we didn't remove popups from the stopping conditions. Skipping the call saves some time, and also avoids extra function calls.

Differential Revision: https://phabricator.services.mozilla.com/D215863
This commit is contained in:
Timothy Nikkel
2024-07-13 08:13:34 +00:00
parent 6f80c122c6
commit 39fa95b71c
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
layout/generic/nsIFrame.cpp
View File

@@ -7469,7 +7469,7 @@ Matrix4x4Flagged nsIFrame::GetTransformMatrix(ViewportType aViewportType,
const nsIFrame* current = this;
auto shouldStopAt = [](const nsIFrame* aCurrent, nsIFrame* aAncestor,
uint32_t aFlags) {
return aAncestor->IsTransformed() || nsLayoutUtils::IsPopup(aAncestor) ||
return aAncestor->IsTransformed() ||
ViewportUtils::IsZoomedContentRoot(aAncestor) ||
((aFlags & STOP_AT_STACKING_CONTEXT_AND_DISPLAY_PORT) &&
(aAncestor->IsStackingContext() ||