Bug 1650444 - Remove browser.find.anonymous_content.enabled. r=jfkthame

Other than this, there hasn't been any other major regression since we
introduced that switch. I don't think there's a point in keeping it
around.

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

modules/libpref/init/StaticPrefList.yaml

@@ -1137,11 +1137,6 @@
   value: "#551A8B"
   mirror: never
 
-- name: browser.find.anonymous_content.enabled
-  type: bool
-  value: true
-  mirror: always
-
 #---------------------------------------------------------------------------
 # Prefs starting with "canvas."
 #---------------------------------------------------------------------------

toolkit/components/find/nsFind.cpp

@@ -168,7 +168,7 @@ static bool ShouldFindAnonymousContent(const nsIContent& aContent) {
     return aContent.IsEditable();
   }
 
-  return StaticPrefs::browser_find_anonymous_content_enabled();
+  return true;
 }
 
 static bool SkipNode(const nsIContent* aContent) {
@@ -192,12 +192,11 @@ static bool SkipNode(const nsIContent* aContent) {
       }
     }
 
-    if (content->IsInNativeAnonymousSubtree()) {
-      if (!ShouldFindAnonymousContent(*content)) {
-        DEBUG_FIND_PRINTF("Skipping node: ");
-        DumpNode(content);
-        return true;
-      }
+    if (content->IsInNativeAnonymousSubtree() &&
+        !ShouldFindAnonymousContent(*content)) {
+      DEBUG_FIND_PRINTF("Skipping node: ");
+      DumpNode(content);
+      return true;
     }
 
     // Only climb to the nearest block node