Make WebLocalFrameImpl::Load() to reset active match

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 5320 matching lines @@
 
   EXPECT_TRUE(frame->EnsureTextFinder().ScopingInProgress());
 
   while (frame->EnsureTextFinder().ScopingInProgress())
     RunPendingTasks();
 
   EXPECT_EQ(3, client.Count());
   EXPECT_EQ(3, client.ActiveIndex());
 }
 
+TEST_P(ParameterizedWebFrameTest,
+       FindInPageStopFindActionKeepSelectionInAnotherDocument) {
+  RegisterMockedHttpURLLoad("find.html");
+  RegisterMockedHttpURLLoad("hello_world.html");
+  FrameTestHelpers::WebViewHelper web_view_helper;
+  web_view_helper.InitializeAndLoad(base_url_ + "find.html");
+  ASSERT_TRUE(web_view_helper.WebView()->MainFrameImpl());
+  WebLocalFrame* frame = web_view_helper.WebView()->MainFrameImpl();
+  const int kFindIdentifier = 12345;
+  WebFindOptions options;
+
+  // Set active match
+  ASSERT_TRUE(
+      frame->Find(kFindIdentifier, WebString::FromUTF8("foo"), options, false));
+  // Move to another page.
+  FrameTestHelpers::LoadFrame(frame, base_url_ + "hello_world.html");
+
+  // Stop Find-In-Page. |TextFinder::active_match_| still hold a |Range| in
+  // "find.html".
+  frame->StopFinding(WebLocalFrame::kStopFindActionKeepSelection);
+
+  // Pass if not crash. See http://crbug.com/719880 for details.
+}
+
 static WebPoint TopLeft(const WebRect& rect) {
   return WebPoint(rect.x, rect.y);
 }
 
 static WebPoint BottomRightMinusOne(const WebRect& rect) {
   // FIXME: If we don't subtract 1 from the x- and y-coordinates of the
   // selection bounds, selectRange() will select the *next* element. That's
   // strictly correct, as hit-testing checks the pixel to the lower-right of
   // the input coordinate, but it's a wart on the API.
   return WebPoint(rect.x + rect.width - 1, rect.y + rect.height - 1);
@@ skipping 6925 matching lines @@
     if (obj->IsText()) {
       LayoutText* layout_text = ToLayoutText(obj);
       text = layout_text->GetText();
       break;
     }
   }
   EXPECT_EQ("foo alt", text.Utf8());
 }
 
 }  // namespace blink