Restart search in page when new text is found.

third_party/WebKit/public/web/WebFrame.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 543 matching lines @@
     //
     // If a match is found, this function will select it (scrolling down to
     // make it visible if needed) and fill in selectionRect with the
     // location of where the match was found (in window coordinates).
     //
     // If no match is found, this function clears all tickmarks and
     // highlighting.
     //
     // Returns true if the search string was found, false otherwise.
     virtual bool find(int identifier,
-                      const WebString& searchText,
-                      const WebFindOptions& options,
-                      bool wrapWithinFrame,
-                      WebRect* selectionRect) = 0;
+        const WebString& searchText,
+        const WebFindOptions&,
+        bool wrapWithinFrame,
+        WebRect* selectionRect,
+        bool& newTextFound) = 0;

[Finnur, 2016/01/20 14:14:03]

style: The style guide says |out| parameters should be pointers, not references.

Also, |newTextFound| is a bit ambiguous in the context of find in page. I think
I would prefer something like |nowActive|.

Also, to answer your question: I don't see a reason to do an enum for this, this
is basically a binary result, at least that's the amount of information we care
about.

[dvadym, 2016/01/29 09:44:17]

Thanks, I've changed to pointer and renamed to activeNow in whole patch.

 
     // Notifies the frame that we are no longer interested in searching.
     // This will abort any asynchronous scoping effort already under way
     // (see the function scopeStringMatches for details) and erase all
     // tick-marks and highlighting from the previous search.  If
     // clearSelection is true, it will also make sure the end state for the
     // find operation does not leave a selection.  This can occur when the
     // user clears the search string but does not close the find box.
     virtual void stopFinding(bool clearSelection) = 0;
 
@@ skipping 165 matching lines @@
     WebFrame* m_firstChild;
     WebFrame* m_lastChild;
 
     WebFrame* m_opener;
     WebPrivateOwnPtr<OpenedFrameTracker> m_openedFrameTracker;
 };
 
 } // namespace blink
 
 #endif