Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-11-27 14:52:16 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Backed out changeset 451ce16f5fd1 (bug 1817691) for causing region update related assertion failures. CLOSED TREE

Browse Source
This commit is contained in:
Cosmin Sabou 2023-03-03 01:02:21 +02:00
parent 5efd10cbf6
commit 8c80747241
1 changed files with 1 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
gfx/layers/NativeLayerCA.mm
View File

@ -1230,9 +1230,6 @@ void NativeLayerCA::HandlePartialUpdate(const MutexAutoLock& aProofOfLock,
// some combination of the update region and the front surface's valid region. Because
// this check is complex, we only do it in Nightly.
//
// As a special case, we'll tolerate a remaining area that is only one pixel tall or wide,
// which is also tolerated in NotifySurfaceReady (until Bug 1818540 is fixed).
//
// Also, since this condition will hit a later release assert in NotifySurfaceReady, we
// choose to crash now.
@ -1246,10 +1243,7 @@ void NativeLayerCA::HandlePartialUpdate(const MutexAutoLock& aProofOfLock,
exposedRegion.AndWith(mFrontSurface->mInvalidRegion);
}
gfx::IntRegion invalidRegion(exposedRegion);
invalidRegion.SubOut(aUpdateRegion);
IntRect invalidBounds = invalidRegion.GetBounds();
if (invalidBounds.width > 1 || invalidBounds.height > 1) {
if (!aUpdateRegion.Contains(exposedRegion)) {
// Let's crash now instead of later.
std::ostringstream reason;
reason << "The update region " << aUpdateRegion << " must cover the invalid region "