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

third_party/WebKit/Source/web/TextFinder.cpp

Index: third_party/WebKit/Source/web/TextFinder.cpp
diff --git a/third_party/WebKit/Source/web/TextFinder.cpp b/third_party/WebKit/Source/web/TextFinder.cpp
index 1b4df8ff00eb63c4d52849368d2e60486da43a48..d5e4454fc6be5117795527523d0e08511ebcd052 100644
--- a/third_party/WebKit/Source/web/TextFinder.cpp
+++ b/third_party/WebKit/Source/web/TextFinder.cpp
@@ -200,7 +200,6 @@ bool TextFinder::find(int identifier, const WebString& searchText, const WebFind
         reportFindInPageSelection(selectionRect, m_activeMatchIndex + 1, identifier);
     }
 
-    m_lastFindRequestCompletedWithNoMatches = false;
     return true;
 }
 
@@ -208,6 +207,7 @@ void TextFinder::clearActiveFindMatch()
 {
     m_currentActiveMatchFrame = false;
     setMarkerActive(m_activeMatch.get(), false);
+    resetActiveMatch();
 }
 
 void TextFinder::stopFindingAndClearSelection()
@@ -285,7 +285,7 @@ void TextFinder::scopeStringMatches(int identifier, const WebString& searchText,
         return;
     }
 
-    if (!shouldScopeMatches(searchText)) {
+    if (!shouldScopeMatches(searchText, options)) {
         finishCurrentScopingEffort(identifier);
         return;
     }
@@ -673,17 +673,23 @@ void TextFinder::unmarkAllTextMatches()
         frame->document()->markers().removeMarkers(DocumentMarker::TextMatch);
 }
 
-bool TextFinder::shouldScopeMatches(const String& searchText)
+bool TextFinder::shouldScopeMatches(const String& searchText, const WebFindOptions& options)
 {
     // Don't scope if we can't find a frame or a view.
     // The user may have closed the tab/application, so abort.
     LocalFrame* frame = ownerFrame().frame();
-    if (!frame || !frame->view() || !frame->page() || !ownerFrame().hasVisibleContent())
+    if (!frame || !frame->view() || !frame->page())
         return false;
 
     DCHECK(frame->document());
     DCHECK(frame->view());
 
+    if (options.force)
+        return true;
+
+    if (!ownerFrame().hasVisibleContent())
+        return false;
+
     // If the frame completed the scoping operation and found 0 matches the last
     // time it was searched, then we don't have to search it again if the user is
     // just adding to the search string or sending the same search string again.