Revert of Handling new frames and frame navigations with find-in-page during a find session.

content/browser/find_request_manager_browsertest.cc

Index: content/browser/find_request_manager_browsertest.cc
diff --git a/content/browser/find_request_manager_browsertest.cc b/content/browser/find_request_manager_browsertest.cc
index 499d4e4e717671239f47d3551fa138696ef43e7a..dce46635ca2be1e9c2bfb767a1121cd05a45a4de 100644
--- a/content/browser/find_request_manager_browsertest.cc
+++ b/content/browser/find_request_manager_browsertest.cc
@@ -7,7 +7,6 @@
 #include "content/browser/web_contents/web_contents_impl.h"
 #include "content/public/browser/notification_types.h"
 #include "content/public/common/content_switches.h"
-#include "content/public/test/browser_test_utils.h"
 #include "content/public/test/content_browser_test.h"
 #include "content/public/test/content_browser_test_utils.h"
 #include "content/public/test/test_navigation_observer.h"
@@ -477,124 +476,8 @@
   results = delegate()->GetFindResults();
   EXPECT_EQ(12, results.number_of_matches);
   EXPECT_EQ(8, results.active_match_ordinal);
-}
-
-// Tests adding a frame during a find session.
-IN_PROC_BROWSER_TEST_P(FindRequestManagerTest, MAYBE(AddFrame)) {
-  LoadMultiFramePage(2 /* height */, GetParam() /* cross_process */);
-
-  blink::WebFindOptions options;
-  Find("result", options);
-  options.findNext = true;
-  Find("result", options);
-  Find("result", options);
-  Find("result", options);
-  Find("result", options);
-  delegate()->WaitForFinalReply();
-
-  FindResults results = delegate()->GetFindResults();
-  EXPECT_EQ(last_request_id(), results.request_id);
-  EXPECT_EQ(21, results.number_of_matches);
-  EXPECT_EQ(5, results.active_match_ordinal);
-
-  // Add a frame. It contains 5 new matches.
-  std::string url = embedded_test_server()->GetURL(
-      GetParam() ? "b.com" : "a.com", "/find_in_simple_page.html").spec();
-  std::string script = std::string() +
-      "var frame = document.createElement('iframe');" +
-      "frame.src = '" + url + "';" +
-      "document.body.appendChild(frame);";
-  delegate()->MarkNextReply();
-  ASSERT_TRUE(ExecuteScript(shell(), script));
-  delegate()->WaitForNextReply();
-
-  // The number of matches should update automatically to include the matches
-  // from the newly added frame.
-  results = delegate()->GetFindResults();
-  EXPECT_EQ(26, results.number_of_matches);
-  EXPECT_EQ(5, results.active_match_ordinal);
-}
-
-// Tests adding a frame during a find session where there were previously no
-// matches.
-IN_PROC_BROWSER_TEST_F(FindRequestManagerTest, MAYBE(AddFrameAfterNoMatches)) {
-  TestNavigationObserver navigation_observer(contents());
-  NavigateToURL(shell(), GURL("about:blank"));
-  EXPECT_TRUE(navigation_observer.last_navigation_succeeded());
-
-  blink::WebFindOptions default_options;
-  Find("result", default_options);
-  delegate()->WaitForFinalReply();
-
-  // Initially, there are no matches on the page.
-  FindResults results = delegate()->GetFindResults();
-  EXPECT_EQ(last_request_id(), results.request_id);
-  EXPECT_EQ(0, results.number_of_matches);
-  EXPECT_EQ(0, results.active_match_ordinal);
-
-  // Add a frame. It contains 5 new matches.
-  std::string url =
-      embedded_test_server()->GetURL("/find_in_simple_page.html").spec();
-  std::string script = std::string() +
-      "var frame = document.createElement('iframe');" +
-      "frame.src = '" + url + "';" +
-      "document.body.appendChild(frame);";
-  delegate()->MarkNextReply();
-  ASSERT_TRUE(ExecuteScript(shell(), script));
-  delegate()->WaitForNextReply();
-
-  // The matches from the new frame should be found automatically, and the first
-  // match in the frame should be activated.
-  results = delegate()->GetFindResults();
-  EXPECT_EQ(5, results.number_of_matches);
-  EXPECT_EQ(1, results.active_match_ordinal);
-}
-
-// Tests a frame navigating to a different page during a find session.
-IN_PROC_BROWSER_TEST_P(FindRequestManagerTest, MAYBE(NavigateFrame)) {
-  LoadMultiFramePage(2 /* height */, GetParam() /* cross_process */);
-
-  blink::WebFindOptions options;
-  Find("result", options);
-  options.findNext = true;
-  options.forward = false;
-  Find("result", options);
-  Find("result", options);
-  Find("result", options);
-  delegate()->WaitForFinalReply();
-
-  FindResults results = delegate()->GetFindResults();
-  EXPECT_EQ(last_request_id(), results.request_id);
-  EXPECT_EQ(21, results.number_of_matches);
-  EXPECT_EQ(19, results.active_match_ordinal);
-
-  // Navigate one of the empty frames to a page with 5 matches.
-  FrameTreeNode* root =
-      static_cast<WebContentsImpl*>(shell()->web_contents())->
-      GetFrameTree()->root();
-  GURL url(embedded_test_server()->GetURL(
-      GetParam() ? "b.com" : "a.com", "/find_in_simple_page.html"));
-  delegate()->MarkNextReply();
-  TestNavigationObserver navigation_observer(contents());
-  NavigateFrameToURL(root->child_at(0)->child_at(1)->child_at(0), url);
-  EXPECT_TRUE(navigation_observer.last_navigation_succeeded());
-  delegate()->WaitForNextReply();
-
-  // The navigation results in an extra reply before the one we care about. This
-  // extra reply happens because the RenderFrameHost changes before it navigates
-  // (because the navigation is cross-origin). The first reply will not change
-  // the number of matches because the frame that is navigating was empty
-  // before.
-  if (delegate()->GetFindResults().number_of_matches == 21) {
-    delegate()->MarkNextReply();
-    delegate()->WaitForNextReply();
-  }
-
-  // The number of matches and the active match ordinal should update
-  // automatically to include the new matches.
-  results = delegate()->GetFindResults();
-  EXPECT_EQ(26, results.number_of_matches);
-  EXPECT_EQ(24, results.active_match_ordinal);
+
+  // TODO(paulemeyer): Once adding frames mid-session is handled, test that too.
 }
 
 // Tests Searching in a hidden frame. Matches in the hidden frame should be
@@ -696,6 +579,7 @@
   results = delegate()->GetFindResults();
   EXPECT_EQ(last_request_id(), results.request_id);
   EXPECT_EQ(5, results.number_of_matches);
+  EXPECT_EQ(1, results.active_match_ordinal);
 }
 
 #if defined(OS_ANDROID)