Do not reinvent the wheel / use content APIs to find a frame by name.

chrome/browser/subresource_filter/subresource_filter_browsertest.cc

Index: chrome/browser/subresource_filter/subresource_filter_browsertest.cc
diff --git a/chrome/browser/subresource_filter/subresource_filter_browsertest.cc b/chrome/browser/subresource_filter/subresource_filter_browsertest.cc
index 273ad8bab8a3fd3986f54237ae7ae96fc3d4ae1c..aa75c2696b87bda69b5982e18f4cc231aac22ac5 100644
--- a/chrome/browser/subresource_filter/subresource_filter_browsertest.cc
+++ b/chrome/browser/subresource_filter/subresource_filter_browsertest.cc
@@ -315,11 +315,8 @@ class SubresourceFilterBrowserTestImpl : public InProcessBrowserTest {
   }
 
   content::RenderFrameHost* FindFrameByName(const std::string& name) {
-    for (content::RenderFrameHost* frame : web_contents()->GetAllFrames()) {
-      if (frame->GetFrameName() == name)
-        return frame;
-    }
-    return nullptr;
+    return content::FrameMatchingPredicate(
+        web_contents(), base::Bind(&content::FrameMatchesName, name));
   }
 
   bool WasParsedScriptElementLoaded(content::RenderFrameHost* rfh) {