Restart search in page when new text is found.

third_party/WebKit/public/web/WebFrame.h

Index: third_party/WebKit/public/web/WebFrame.h
diff --git a/third_party/WebKit/public/web/WebFrame.h b/third_party/WebKit/public/web/WebFrame.h
index 9ae9234584f187ee39b1ae311a96a13d014484d4..dde5a7b15b6756cf18edb47bb087bd16886fc3b2 100644
--- a/third_party/WebKit/public/web/WebFrame.h
+++ b/third_party/WebKit/public/web/WebFrame.h
@@ -561,10 +561,11 @@ public:
     //
     // 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