[Android] Adding Smart GO/NEXT feature in Chrome

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 08a77e2290a517292c0844a5ea6781195b74cf5b..839c3770fdf4e562c4b9a6cdf91a03c609e437a4 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -179,6 +179,7 @@
 #include "third_party/WebKit/public/platform/URLConversion.h"
 #include "third_party/WebKit/public/platform/WebCachePolicy.h"
 #include "third_party/WebKit/public/platform/WebData.h"
+#include "third_party/WebKit/public/platform/WebFocusType.h"
 #include "third_party/WebKit/public/platform/WebKeyboardEvent.h"
 #include "third_party/WebKit/public/platform/WebMediaPlayer.h"
 #include "third_party/WebKit/public/platform/WebMediaPlayerSource.h"
@@ -1581,6 +1582,7 @@ bool RenderFrameImpl::OnMessageReceived(const IPC::Message& msg) {
     IPC_MESSAGE_HANDLER(FrameMsg_SetFrameOwnerProperties,
                         OnSetFrameOwnerProperties)
     IPC_MESSAGE_HANDLER(FrameMsg_AdvanceFocus, OnAdvanceFocus)
+    IPC_MESSAGE_HANDLER(FrameMsg_AdvanceFocusInForm, OnAdvanceFocusInForm)
     IPC_MESSAGE_HANDLER(FrameMsg_SetFocusedFrame, OnSetFocusedFrame)
     IPC_MESSAGE_HANDLER(FrameMsg_SetTextTrackSettings,
                         OnTextTrackSettingsChanged)
@@ -2206,6 +2208,12 @@ void RenderFrameImpl::OnAdvanceFocus(blink::WebFocusType type,
       type, source_frame->web_frame(), frame_);
 }
 
+void RenderFrameImpl::OnAdvanceFocusInForm(bool forward) {
+  blink::WebFocusType focus_type =
+      forward ? blink::WebFocusTypeForward : blink::WebFocusTypeBackward;
+  render_view_->webview()->advanceFocusInForm(focus_type);

[EhsanK, 2017/04/25 22:43:08]

Following nasko@'s suggestion I wonder if moving the logic to WebLocalFrame and
doing something like this might be better here:

if (render_view_->webview()->FocusedFrame() != frame_)
  return;
frame_->advanceFocusInForm(forward ? blink::WebFocusTypeForward :
blink::WebFocusTypeBackward);

(the first part for checking focused frame might not be necessary, but I believe
it is possible to have a different frame focused on the renderer side by the
time the IPC arrives).

[AKVT, 2017/04/26 10:33:04]

Done.

+}
+
 void RenderFrameImpl::OnSetFocusedFrame() {
   // This uses focusDocumentView rather than setFocusedFrame so that focus/blur
   // events are properly dispatched on any currently focused elements.