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

chrome/browser/ui/views/drag_and_drop_interactive_uitest.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <algorithm>
 #include <initializer_list>
 #include <memory>
 #include <string>
 #include <vector>
 
@@ skipping 684 matching lines @@
     frame = GetFrameByName(frame_name);
     DCHECK(frame);
 
     // Wait until frame contents have painted and are ready for hit testing.
     WaitForChildFrameSurfaceReady(frame);
 
     return true;
   }
 
   content::RenderFrameHost* GetFrameByName(const std::string& name_to_find) {
-    content::RenderFrameHost* result = nullptr;
-    for (content::RenderFrameHost* rfh : web_contents()->GetAllFrames()) {
-      if (rfh->GetFrameName() == name_to_find) {
-        if (result) {
-          ADD_FAILURE() << "More than one frame named "
-                        << "'" << name_to_find << "'";
-          return nullptr;
-        }
-        result = rfh;
-      }
-    }
-
-    EXPECT_TRUE(result) << "Couldn't find a frame named "
-                        << "'" << name_to_find << "'";
-    return result;
+    return content::FrameMatchingPredicate(
+        web_contents(), base::Bind(&content::FrameMatchesName, name_to_find));

[ncarter (slow), 2017/04/27 22:03:42]

You've removed some ADD_FAILURES here, but I don't see that this test is
especially sensitive to duplicate names, so I think it's okay.

[Łukasz Anforowicz, 2017/04/27 22:59:28]

Yes - I think replacing gtest asserts with a DCHECK (inside
FrameMatchingPredicate) is fine.

   }
 
   void AssertTestPageIsLoaded() {
     ASSERT_EQ(kTestPagePath, web_contents()->GetLastCommittedURL().path());
   }
 
   std::unique_ptr<DragAndDropSimulator> drag_simulator_;
 
   DISALLOW_COPY_AND_ASSIGN(DragAndDropBrowserTest);
 };
@@ skipping 539 matching lines @@
 // of a drag operation, and cross-site drags should be allowed across a
 // navigation.
 
 INSTANTIATE_TEST_CASE_P(
     SameSiteSubframe, DragAndDropBrowserTest, ::testing::Values(false));
 
 INSTANTIATE_TEST_CASE_P(
     CrossSiteSubframe, DragAndDropBrowserTest, ::testing::Values(true));
 
 }  // namespace chrome