• Kevin McNee's avatar
    Prevent speculative RWHVAs from handling input meant for existing views · 1843005f
    Kevin McNee authored
    Currently, when we create a RenderWidgetHostViewAndroid, we attach
    its corresponding native view to the hierarchy. However, RWHVAs
    may be created for a speculative RenderFrameHost for a cross-site
    navigation, and would not replace the old RWHVA until the navigation
    commits. This gives us two RWHVAs in the hierarchy. In this state,
    input events are targeted to the speculative RWHVA.
    
    Moreover, if the cross-site navigation is canceled before it commits,
    then we stay in this state where input events are sent to the unused
    RWHVA, causing the page to be seemingly unresponsive to input.
    
    We now move new RWHVAs behind any existing ones in the view hierarchy.
    
    Bug: 867932
    Change-Id: I845d99ecbde7f721f6fa4b6e1b0243abb89a97fc
    Reviewed-on: https://chromium-review.googlesource.com/1165482Reviewed-by: default avatarBo <boliu@chromium.org>
    Commit-Queue: Kevin McNee <mcnee@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#582206}
    1843005f
view_android_unittests.cc 9.83 KB