Bug 1637908 - Minor cleanup. r=snorp

Depends on D91025

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

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) ||