Implement a shell of FindRequestManager, and hook it up to WebContentsImpl.

content/public/browser/render_frame_host.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_PUBLIC_BROWSER_RENDER_FRAME_HOST_H_
 #define CONTENT_PUBLIC_BROWSER_RENDER_FRAME_HOST_H_
 
 #include <string>
 
 #include "base/callback_forward.h"
@@ skipping 144 matching lines @@
   virtual void AccessibilitySetSelection(int anchor_object_id,
                                          int anchor_offset,
                                          int focus_object_id,
                                          int focus_offset) = 0;
 
   // This is called when the user has committed to the given find in page
   // request (e.g. by pressing enter or by clicking on the next / previous
   // result buttons). It triggers sending a native accessibility event on
   // the result object on the page, navigating assistive technology to that
   // result.
   virtual void ActivateFindInPageResultForAccessibility(int request_id) = 0;

[ncarter (slow), 2016/04/06 19:08:03]

Is this method on your road map? Should it move to web_contents.h as well, to
match ActivateNearestFindResult()?

[paulmeyer, 2016/04/07 17:36:15]

As far as I know, no. ActivateNearestFindResult() needs coordination by
FindRequestManager (it can't know which process/frame to send the IPC to without
some work from FindRequestManager), so I moved the it out of RenderFrameHost so
that it can't be called without going through FindRequestManager. This doesn't
seem to be the case for ActivateFindInPageResultForAccessibility (I'm not aware
of changes needed to the way it works for multi-process find-in-page), though
I'm not as familiar with how it works.

[ncarter (slow), 2016/04/07 18:19:14]

Acknowledged.

 
   // Roundtrips through the renderer and compositor pipeline to ensure that any
   // changes to the contents resulting from operations executed prior to this
   // call are visible on screen. The call completes asynchronously by running
   // the supplied |callback| with a value of true upon successful completion and
   // false otherwise (when the frame is destroyed, detached, etc..).
   typedef base::Callback<void(bool)> VisualStateCallback;
   virtual void InsertVisualStateCallback(
       const VisualStateCallback& callback) = 0;
 
   // Temporary until we get rid of RenderViewHost.
   virtual RenderViewHost* GetRenderViewHost() = 0;
 
   // Returns the ServiceRegistry for this frame.
   virtual ServiceRegistry* GetServiceRegistry() = 0;
 
   // Returns the visibility state of the frame. The different visibility states
   // of a frame are defined in Blink.
   virtual blink::WebPageVisibilityState GetVisibilityState() = 0;
 
   // Returns whether the RenderFrame in the renderer process has been created
   // and still has a connection.  This is valid for all frames.
   virtual bool IsRenderFrameLive() = 0;
 
   // Get the number of proxies to this frame, in all processes. Exposed for
   // use by resource metrics.
   virtual int GetProxyCount() = 0;
 
-#if defined(OS_ANDROID)
-  // Selects and zooms to the find result nearest to the point (x,y)
-  // defined in find-in-page coordinates.
-  virtual void ActivateNearestFindResult(int request_id, float x, float y) = 0;
-
-  // Asks the renderer process to send the rects of the current find matches.
-  virtual void RequestFindMatchRects(int current_version) = 0;
-#endif
-
  private:
   // This interface should only be implemented inside content.
   friend class RenderFrameHostImpl;
   RenderFrameHost() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_RENDER_FRAME_HOST_H_