Bug 1934146. Only build compositor hit test info if we are painting t…

…o the window. r=layout-reviewers,emilio

If we aren't painting to the window then the hit test info has no where to go as we must be painting to a flat simple surface.

I moved the bools closer together so they doesn't require two memory reads.

A couple of less fuzzy required in reftest snapshot due to no longer having hit test info items.

Differential Revision: https://phabricator.services.mozilla.com/D230629

UltraBlame original commit: 15ed73287645c809601c764e4d200f6e2009728b

layout/painting/nsDisplayList.cpp

@@ -3680,6 +3680,16 @@ mIsPaintingToWindow
 (
 false
 )
+mAsyncPanZoomEnabled
+(
+nsLayoutUtils
+:
+:
+AsyncPanZoomEnabled
+(
+aReferenceFrame
+)
+)
 mUseHighQualityScaling
 (
 false
@@ -3718,16 +3728,6 @@ mContainsNonMinimalDisplayPort
 (
 false
 )
-mAsyncPanZoomEnabled
-(
-nsLayoutUtils
-:
-:
-AsyncPanZoomEnabled
-(
-aReferenceFrame
-)
-)
 mBuildingInvisibleItems
 (
 false
@@ -3778,15 +3778,6 @@ MOZ_COUNT_CTOR
 nsDisplayListBuilder
 )
 ;
-mBuildCompositorHitTestInfo
-=
-mAsyncPanZoomEnabled
-&
-&
-IsForPainting
-(
-)
-;
 ShouldRebuildDisplayListDueToPrefChange
 (
 )

layout/painting/nsDisplayList.h

@@ -1210,7 +1210,10 @@ BuildCompositorHitTestInfo
 const
 {
 return
-mBuildCompositorHitTestInfo
+mAsyncPanZoomEnabled
+&
+&
+mIsPaintingToWindow
 ;
 }
 bool
@@ -5655,6 +5658,9 @@ bool
 mIsPaintingToWindow
 ;
 bool
+mAsyncPanZoomEnabled
+;
+bool
 mUseHighQualityScaling
 ;
 bool
@@ -5679,9 +5685,6 @@ bool
 mContainsNonMinimalDisplayPort
 ;
 bool
-mAsyncPanZoomEnabled
-;
-bool
 mBuildingInvisibleItems
 ;
 bool
@@ -5691,9 +5694,6 @@ bool
 mInInvalidSubtree
 ;
 bool
-mBuildCompositorHitTestInfo
-;
-bool
 mDisablePartialUpdates
 ;
 bool

layout/reftests/bugs/reftest.list

@@ -31268,10 +31268,10 @@ fuzzy
 if
 (
 useDrawSnapshot
-33
+0
 -
 33
-218
+0
 -
 218
 )

layout/reftests/transform-3d/reftest.list

@@ -2044,10 +2044,10 @@ fuzzy
 if
 (
 useDrawSnapshot
-4
+0
 -
 4
-13
+0
 -
 13
 )