Implement the basic testing infrastructure for FindRequestManager.

content/browser/find_request_manager.cc

Index: content/browser/find_request_manager.cc
diff --git a/content/browser/find_request_manager.cc b/content/browser/find_request_manager.cc
index ad6b24e14371b7986725e79fc34521f8577bfca2..a43bef78e8feb073905ee8718f6a81ffc88bdc25 100644
--- a/content/browser/find_request_manager.cc
+++ b/content/browser/find_request_manager.cc
@@ -64,9 +64,12 @@ void FindRequestManager::OnFindReply(RenderFrameHost* rfh,
     return;
 
   // Update the stored results.
-  number_of_matches_ = number_of_matches;
-  selection_rect_ = selection_rect;
-  active_match_ordinal_ = active_match_ordinal;
+  if (number_of_matches != -1)
+    number_of_matches_ = number_of_matches;
+  if (!selection_rect.IsEmpty())
+    selection_rect_ = selection_rect;
+  if (active_match_ordinal > 0)
+    active_match_ordinal_ = active_match_ordinal;

[nasko, 2016/04/18 21:02:39]

See my comment in the test file. Updating these conditionally for each variable
can lead to storing values from different IPCs.

[paulmeyer, 2016/04/19 15:15:00]

This was intentional, and is something that FindRequestManager will do
eventually, but I'll leave it out until it's needed. Something similar will
still be needed in the TestWebContentsDelegate though (see comment there).

 
   NotifyFindReply(request_id, final_update);
 }