[refactor] Fix autofill features for payments when the form is inside an OOPIF

content/public/renderer/render_frame_observer.h

Index: content/public/renderer/render_frame_observer.h
diff --git a/content/public/renderer/render_frame_observer.h b/content/public/renderer/render_frame_observer.h
index 7211fdcd334185658fccdb079a15da1fd66a60b1..0464ff57e864e5807fdabb597281de5acde2cc31 100644
--- a/content/public/renderer/render_frame_observer.h
+++ b/content/public/renderer/render_frame_observer.h
@@ -118,6 +118,14 @@ class CONTENT_EXPORT RenderFrameObserver : public IPC::Listener,
   // Called when the focused node has changed to |node|.
   virtual void FocusedNodeChanged(const blink::WebNode& node) {}
 
+  // Called when an element has gained focus inside the frame and after all the
+  // required animations (if any)  have finished.

[Charlie Reis, 2017/04/05 20:13:44]

nit: Fix double space before "have"

[EhsanK, 2017/04/06 21:31:27]

Done. Thanks!

+  virtual void DidCompleteFocusChangeInFrame() {}
+
+  // Called when a gesture tap or mouse down occurs inside |node| in this frame.
+  virtual void DidCompleteLeftMouseDownOrGestureTapInNode(

[Charlie Reis, 2017/04/05 20:13:44]

This name is a bit awkward, but I'm ok with it if you need to be this specific. 
That said, it looks like it's not hooked up.  :)

[EhsanK, 2017/04/06 21:31:27]

Yes I was not sure how it should be named. In reality the method was called when
there is a mouse down or tap. The name inside blink is onMouseDown but I figured
that might be a bit incorrect given that it is not necessarily on mouse down.
But I am open to any suggestion regarding the naming (that is if this method
survives the patch given that there is a possibility of moving the logic to
WebAutofillClient).

Also, this is implemented in PageClickTracker which is one of the 
RenderFrameObservers in this patch; and should get called from
RenderFrameImpl::didCompleteLeftMouseDownOrGestureTapInNode().

[Charlie Reis, 2017/04/10 06:22:33]

Ah, my mistake!  I was reading it wrong.

+      const blink::WebNode& node) {}
+
   // Called when accessibility is enabled or disabled.
   virtual void AccessibilityModeChanged() {}