Bug 1863491 - Use weak pointers for LargestContentfulPaint to avoid k…

…eeping the element alive for unnecessary period r=emilio,mccr8

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

UltraBlame original commit: abf27349f36e96612106b51dd196526df2efd501

dom/performance/LargestContentfulPaint.cpp

@@ -475,10 +475,6 @@ mURI
 (
 aURI
 )
-mElement
-(
-aElement
-)
 mLCPImageEntryKey
 (
 aLCPImageEntryKey
@@ -491,7 +487,7 @@ mPerformance
 ;
 MOZ_ASSERT
 (
-mElement
+aElement
 )
 ;
 /
@@ -506,7 +502,7 @@ pseudo
 element
 if
 (
-mElement
+aElement
 -
 >
 ChromeOnlyAccess
@@ -516,6 +512,8 @@ ChromeOnlyAccess
 {
 mElement
 =
+do_GetWeakReference
+(
 Element
 :
 :
@@ -528,6 +526,17 @@ FindFirstNonChromeOnlyAccessContent
 (
 )
 )
+)
+;
+}
+else
+{
+mElement
+=
+do_GetWeakReference
+(
+aElement
+)
 ;
 }
 if
@@ -596,16 +605,27 @@ GetElement
 )
 const
 {
-return
+nsCOMPtr
+<
+Element
+>
+element
+=
+do_QueryReferent
+(
 mElement
+)
+;
+return
+element
 ?
 nsContentUtils
 :
 :
 GetAnElementForTiming
 (
-mElement
-mElement
+element
+element
 -
 >
 GetComposedDoc
@@ -625,6 +645,15 @@ BufferEntryIfNeeded
 (
 )
 {
+MOZ_ASSERT
+(
+mLCPImageEntryKey
+.
+isNothing
+(
+)
+)
+;
 mPerformance
 -
 >
@@ -3249,12 +3278,32 @@ ReportLCPToNavigationTimings
 (
 )
 {
+nsCOMPtr
+<
+Element
+>
+element
+=
+do_QueryReferent
+(
+mElement
+)
+;
+if
+(
+!
+element
+)
+{
+return
+;
+}
 const
 Document
 *
 document
 =
-mElement
+element
 -
 >
 OwnerDoc