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

third_party/WebKit/Source/web/ChromeClientImpl.cpp

Index: third_party/WebKit/Source/web/ChromeClientImpl.cpp
diff --git a/third_party/WebKit/Source/web/ChromeClientImpl.cpp b/third_party/WebKit/Source/web/ChromeClientImpl.cpp
index cf15d6dd1d0e6dbf307d6ffaf2fd2589d005ffde..fb5000ddd1299aa6081ef4141456d88a08337f4d 100644
--- a/third_party/WebKit/Source/web/ChromeClientImpl.cpp
+++ b/third_party/WebKit/Source/web/ChromeClientImpl.cpp
@@ -1086,8 +1086,12 @@ void ChromeClientImpl::showUnhandledTapUIIfNeeded(
 }
 
 void ChromeClientImpl::onMouseDown(Node* mouseDownNode) {
-  if (m_webView->client())
-    m_webView->client()->onMouseDown(WebNode(mouseDownNode));
+  if (!mouseDownNode)

[dcheng, 2017/04/03 19:17:04]

Can this actually be null?

[EhsanK, 2017/04/04 18:01:49]

Thanks. Apparently it can't. Code search returns the two calls to be from
EventHandler and GestureManager and in both cases we only call this method if
HitTestResult.innerNode() is not nullptr.

+    return;
+
+  WebLocalFrameImpl::fromFrame(mouseDownNode->document().frame())
+      ->client()
+      ->didCompleteLeftMouseDownOrGestureTapInNode(WebNode(mouseDownNode));
 }
 
 void ChromeClientImpl::handleKeyboardEventOnTextField(