FrameTree::Find only searches relative to local frames.

third_party/WebKit/Source/web/tests/WebFrameTest.cpp

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 138 matching lines @@
 #include "public/web/WebSettings.h"
 #include "public/web/WebTextCheckClient.h"
 #include "public/web/WebTextCheckingCompletion.h"
 #include "public/web/WebTextCheckingResult.h"
 #include "public/web/WebViewClient.h"
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "v8/include/v8.h"
 #include "web/TextFinder.h"
 #include "web/WebRemoteFrameImpl.h"
+#include "web/WebViewImpl.h"
 #include "web/tests/FrameTestHelpers.h"
 #include "web/tests/sim/SimDisplayItemList.h"
 #include "web/tests/sim/SimRequest.h"
 #include "web/tests/sim/SimTest.h"
 
 using blink::URLTestHelpers::ToKURL;
 using blink::testing::RunPendingTasks;
 using testing::ElementsAre;
 using testing::Mock;
 using testing::_;
@@ skipping 4197 matching lines @@
   RegisterMockedHttpURLLoad("iframe_redirect.html");
   RegisterMockedHttpURLLoad("visible_iframe.html");
 
   FrameTestHelpers::WebViewHelper web_view_helper;
   web_view_helper.InitializeAndLoad(base_url_ + "iframe_redirect.html", true);
   // Pump pending requests one more time. The test page loads script that
   // navigates.
   FrameTestHelpers::PumpPendingRequestsForFrameToLoad(
       web_view_helper.WebView()->MainFrame());
 
-  WebFrame* iframe = web_view_helper.WebView()->FindFrameByName(
-      WebString::FromUTF8("ifr"), nullptr);
+  WebFrame* iframe = static_cast<WebViewImpl*>(web_view_helper.WebView())

[dcheng, 2017/05/26 20:16:19]

WebView() already returns a WebViewImpl, no static cast needed.

[Łukasz Anforowicz, 2017/05/26 20:51:19]

Doh - thanks for pointing this out.  Done (but still keeping the new include for
IWYU).

+                         ->MainFrameImpl()
+                         ->FindFrameByName(WebString::FromUTF8("ifr"));
   ASSERT_TRUE(iframe);
   WebDataSource* iframe_data_source = iframe->DataSource();
   ASSERT_TRUE(iframe_data_source);
   WebVector<WebURL> redirects;
   iframe_data_source->RedirectChain(redirects);
   ASSERT_EQ(2U, redirects.size());
   EXPECT_EQ(ToKURL("about:blank"), KURL(redirects[0]));
   EXPECT_EQ(ToKURL("http://internal.test/visible_iframe.html"),
             KURL(redirects[1]));
 }
@@ skipping 7789 matching lines @@
     if (obj->IsText()) {
       LayoutText* layout_text = ToLayoutText(obj);
       text = layout_text->GetText();
       break;
     }
   }
   EXPECT_EQ("foo alt", text.Utf8());
 }
 
 }  // namespace blink