Removed limitation on searching frames for the same string prefix.

third_party/WebKit/Source/web/TextFinder.h

 /*
  * Copyright (C) 2009 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 212 matching lines @@
   // ended our last search so we can continue from where we left of.
   //
   // This range is collapsed to the end position of the last successful
   // search; the new search should start from this position.
   Member<Range> m_resumeScopingFromRange;
 
   // Keeps track of the last string this frame searched for. This is used for
   // short-circuiting searches in the following scenarios: When a frame has
   // been searched and returned 0 results, we don't need to search that frame
   // again if the user is just adding to the search (making it more specific).
   WTF::String m_lastSearchString;

[dcheng, 2016/10/17 18:17:41]

Can we get rid of this (and maybe m_lastMatchCount)? It's not clear to me what
parts of this accounting are important.

 
   // Keeps track of how many matches this frame has found so far, so that we
   // don't lose count between scoping efforts, and is also used (in conjunction
   // with m_lastSearchString) to figure out if we need to search the frame
   // again.
   int m_lastMatchCount;
 
   // This variable keeps a cumulative total of matches found so far in this
   // frame, and is only incremented by calling IncreaseMatchCount.
   int m_totalMatchCount;
@@ skipping 25 matching lines @@
   IntSize m_contentsSizeForCurrentFindMatchRects;
 
   // This flag is used by the scoping effort to determine if we need to figure
   // out which rectangle is the active match. Once we find the active
   // rectangle we clear this flag.
   bool m_locatingActiveRect;
 
   // Keeps track of whether there is an scoping effort ongoing in the frame.
   bool m_scopingInProgress;
 
-  // Keeps track of whether the last find request completed its scoping effort
-  // without finding any matches in this frame.
-  bool m_lastFindRequestCompletedWithNoMatches;
-
   // Determines if the rects in the find-in-page matches cache of this frame
   // are invalid and should be recomputed.
   bool m_findMatchRectsAreValid;
 };
 
 }  // namespace blink
 
 WTF_ALLOW_INIT_WITH_MEM_FUNCTIONS(blink::TextFinder::FindMatch);
 
 #endif  // TextFinder_h