Bug 1833570 - Change popover invoker type from boolean to Element. r=…

…emilio

The invoker type is currently implemented as boolean as suggested at
whatwg/html#9168. This issue is now closed and
has been fixed at whatwg/html#9171.

This patch is to be follow above discussions and implement popover invoker
type as element.

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

UltraBlame original commit: 1e825b150a60ac26b0f652679eca1b223354fb64

dom/base/Document.cpp

@@ -93952,11 +93952,34 @@ list
 ;
 }
 }
-aPopover
-.
-SetHasPopoverInvoker
+auto
+*
+data
+=
+popoverHTMLEl
+-
+>
+GetPopoverData
 (
-false
+)
+;
+MOZ_ASSERT
+(
+data
+"
+Should
+have
+popover
+data
+"
+)
+;
+data
+-
+>
+SetInvoker
+(
+nullptr
 )
 ;
 /

dom/base/Element.cpp

@@ -28142,82 +28142,6 @@ bool
 Element
 :
 :
-HasPopoverInvoker
-(
-)
-const
-{
-auto
-*
-popoverData
-=
-GetPopoverData
-(
-)
-;
-return
-popoverData
-&
-&
-popoverData
--
->
-HasPopoverInvoker
-(
-)
-;
-}
-void
-Element
-:
-:
-SetHasPopoverInvoker
-(
-bool
-aHasInvoker
-)
-{
-if
-(
-aHasInvoker
-)
-{
-EnsurePopoverData
-(
-)
-.
-SetHasPopoverInvoker
-(
-true
-)
-;
-}
-else
-if
-(
-auto
-*
-popoverData
-=
-GetPopoverData
-(
-)
-)
-{
-popoverData
--
->
-SetHasPopoverInvoker
-(
-false
-)
-;
-}
-}
-bool
-Element
-:
-:
 IsAutoPopover
 (
 )
@@ -28465,15 +28389,6 @@ GetFlattenedTreeParentElement
 ;
 /
 /
-TODO
-:
-To
-handle
-the
-button
-invokers
-/
-/
 https
 :
 /
@@ -28489,6 +28404,29 @@ html
 issues
 /
 9160
+RefPtr
+<
+Element
+>
+invoker
+=
+newPopover
+-
+>
+GetPopoverData
+(
+)
+-
+>
+GetInvoker
+(
+)
+;
+checkAncestor
+(
+invoker
+)
+;
 return
 topmostPopoverAncestor
 ;

dom/base/Element.h

@@ -3123,41 +3123,6 @@ CreatePopoverData
 )
 ;
 }
-/
-/
-https
-:
-/
-/
-html
-.
-spec
-.
-whatwg
-.
-org
-/
-multipage
-/
-popover
-.
-html
-#
-popover
--
-invoker
-bool
-HasPopoverInvoker
-(
-)
-const
-;
-void
-SetHasPopoverInvoker
-(
-bool
-)
-;
 bool
 IsAutoPopover
 (

dom/base/PopoverData.h

@@ -123,21 +123,21 @@ mozilla_dom_PopoverData_h
 #
 include
 "
-nsStringFwd
+Element
 .
 h
 "
 #
 include
 "
-nsIRunnable
+nsINode
 .
 h
 "
 #
 include
 "
-nsThreadUtils
+nsIRunnable
 .
 h
 "
@@ -148,6 +148,20 @@ nsIWeakReferenceUtils
 .
 h
 "
+#
+include
+"
+nsStringFwd
+.
+h
+"
+#
+include
+"
+nsThreadUtils
+.
+h
+"
 namespace
 mozilla
 :
@@ -386,26 +400,43 @@ mPreviouslyFocusedElement
 aPreviouslyFocusedElement
 ;
 }
-bool
-HasPopoverInvoker
+RefPtr
+<
+Element
+>
+GetInvoker
 (
 )
 const
 {
 return
-mHasPopoverInvoker
+do_QueryReferent
+(
+mInvokerElement
+)
 ;
 }
 void
-SetHasPopoverInvoker
+SetInvoker
 (
-bool
-aHasPopoverInvoker
+Element
+*
+aInvokerElement
 )
 {
-mHasPopoverInvoker
+mInvokerElement
 =
-aHasPopoverInvoker
+do_GetWeakReference
+(
+static_cast
+<
+nsINode
+*
+>
+(
+aInvokerElement
+)
+)
 ;
 }
 PopoverToggleEventTask
@@ -542,39 +573,68 @@ whatwg
 .
 org
 /
-multipage
-/
-popover
-.
-html
 #
 popover
 -
 invoker
-also
 /
 /
-see
-https
-:
+Since
+having
+a
+popover
+invoker
+only
+makes
+a
+difference
+if
+the
+invoker
 /
 /
-github
-.
-com
+is
+in
+the
+document
+(
+in
+another
+open
+popover
+to
+be
+precise
+)
+we
+can
+make
 /
-whatwg
 /
-html
+this
+a
+weak
+reference
+as
+if
+the
+element
+goes
+away
+it
+'
+s
+necessarily
+not
 /
-issues
 /
-9168
+connected
+to
+our
+document
 .
-bool
-mHasPopoverInvoker
-=
-false
+nsWeakPtr
+mInvokerElement
 ;
 bool
 mIsHiding