From 30be708fe0218861b563f2ff52f3184cbe4c7fdd Mon Sep 17 00:00:00 2001 From: Kartikaya Gupta Date: Tue, 22 Sep 2020 20:42:52 +0000 Subject: [PATCH] Bug 1637908 - Minor cleanup. r=snorp Depends on D91025 Differential Revision: https://phabricator.services.mozilla.com/D91026 --- layout/base/PositionedEventTargeting.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/layout/base/PositionedEventTargeting.cpp b/layout/base/PositionedEventTargeting.cpp index 1a9d3e1ebbfe..35d849917717 100644 --- a/layout/base/PositionedEventTargeting.cpp +++ b/layout/base/PositionedEventTargeting.cpp @@ -102,7 +102,6 @@ struct EventRadiusPrefs { mReposition = false; // Always false, unlike mouse events. } else if (aEventClassID == eMouseEventClass) { - // Mostly for testing purposes mEnabled = StaticPrefs::ui_mouse_radius_enabled(); mVisitedWeight = StaticPrefs::ui_mouse_radius_visitedWeight(); mRadiusTopmm = StaticPrefs::ui_mouse_radius_topmm(); @@ -181,7 +180,7 @@ static bool IsDescendant(nsIFrame* aFrame, nsIContent* aAncestor, } static nsIContent* GetClickableAncestor( - nsIFrame* aFrame, nsAtom* stopAt = nullptr, + nsIFrame* aFrame, nsAtom* aStopAt = nullptr, nsAutoString* aLabelTargetId = nullptr) { // If the frame is `cursor:pointer` or inherits `cursor:pointer` from an // ancestor, treat it as clickable. This is a heuristic to deal with pages @@ -205,7 +204,7 @@ static nsIContent* GetClickableAncestor( // ancestors to look for elements accepting the click. for (nsIContent* content = aFrame->GetContent(); content; content = content->GetFlattenedTreeParent()) { - if (stopAt && content->IsHTMLElement(stopAt)) { + if (aStopAt && content->IsHTMLElement(aStopAt)) { break; } if (HasTouchListener(content) || HasMouseListener(content) ||