It would be useful to know what the actual thing to do to get the RWHVA::Focus() to work. Does the implementation of ContentViewCore.requestFocus() do what rwhva/rwh expects? It is a void method but the value to return is available from android.view.requestFocus().

Well, the expectation is:
"""
RenderWidgetHostViewAndroid::Focus is content requesting android to make the native view backing this widget focused. Then as a response (if successful), can call host->GotFocus().
"""

so call requestFocus on the container view. then when view received focus, call back down to host->GotFocus

hopefully that doesn't break anything..

Thanks. I'll do that and see what it breaks : )

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/74b8a0a5ebfe5dc1a1dcd5c0a6ac62e68f8a787f

commit 74b8a0a5ebfe5dc1a1dcd5c0a6ac62e68f8a787f
Author: Jinsuk Kim <jinsukkim@chromium.org>
Date: Thu Aug 10 05:16:36 2017

Keep RenderWidgetHostView focus state in sync with Content

Previously, the request |RWHVA::Focus()| on Android was not propagated
to the content native view but simply shortcut back to the host,
which is not correct. This CL requests focus on the container view
upon request from RWHVA, and reflects the result accordinly.

BUG= 746099 

Change-Id: I87d6be654049787331f73a5ac30ea6cebf13d6b3
Reviewed-on: https://chromium-review.googlesource.com/583412
Reviewed-by: Bo <boliu@chromium.org>
Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#493252}
[modify] https://crrev.com/74b8a0a5ebfe5dc1a1dcd5c0a6ac62e68f8a787f/content/browser/android/content_view_core.cc
[modify] https://crrev.com/74b8a0a5ebfe5dc1a1dcd5c0a6ac62e68f8a787f/content/browser/android/content_view_core.h
[modify] https://crrev.com/74b8a0a5ebfe5dc1a1dcd5c0a6ac62e68f8a787f/content/browser/renderer_host/render_widget_host_view_android.cc
[modify] https://crrev.com/74b8a0a5ebfe5dc1a1dcd5c0a6ac62e68f8a787f/content/public/android/BUILD.gn
[modify] https://crrev.com/74b8a0a5ebfe5dc1a1dcd5c0a6ac62e68f8a787f/content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
[delete] https://crrev.com/1b8abc019a5b476d503bff7405120c44036799eb/content/public/android/java/src/org/chromium/content/browser/ViewUtils.java
[modify] https://crrev.com/74b8a0a5ebfe5dc1a1dcd5c0a6ac62e68f8a787f/content/public/android/java/src/org/chromium/content/browser/input/ImeAdapter.java
[modify] https://crrev.com/74b8a0a5ebfe5dc1a1dcd5c0a6ac62e68f8a787f/ui/android/BUILD.gn
[modify] https://crrev.com/74b8a0a5ebfe5dc1a1dcd5c0a6ac62e68f8a787f/ui/android/java/src/org/chromium/ui/base/ViewAndroidDelegate.java
[add] https://crrev.com/74b8a0a5ebfe5dc1a1dcd5c0a6ac62e68f8a787f/ui/android/java/src/org/chromium/ui/base/ViewUtils.java
[modify] https://crrev.com/74b8a0a5ebfe5dc1a1dcd5c0a6ac62e68f8a787f/ui/android/view_android.cc
[modify] https://crrev.com/74b8a0a5ebfe5dc1a1dcd5c0a6ac62e68f8a787f/ui/android/view_android.h