• Ian Vollick's avatar
    Revert "[vr] Ensure that hit testing uses the same stacking order as rendering" · 76346c14
    Ian Vollick authored
    This reverts commit 053d939f.
    
    Reason for revert: Hit testing artifacts.
    
    Original change's description:
    > [vr] Ensure that hit testing uses the same stacking order as rendering
    >
    > The rendering code was updated to sort elements in tree rather than
    > depth order in this cl:
    > https://chromium-review.googlesource.com/c/chromium/src/+/727342
    >
    > The hit testing code, however, was not similarly updated. Consequently,
    > there can be discrepancies. This CL updates the hit testing code to
    > use nearly the same stacking order as rendering. Note: it is not quite
    > perfect because the ui renderer still summons swaths of elements from
    > the scene explicitly. Eventually, the ui renderer will simply sort
    > and render all elements in the tree and these will be truly identical.
    > However, until then, this ensures we do use the same element sorting
    > routines.
    >
    > Bug: None
    > Change-Id: I12483713f19b678aa9a69b91e5d910e408b73007
    > Reviewed-on: https://chromium-review.googlesource.com/754554
    > Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
    > Commit-Queue: Ian Vollick <vollick@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#514136}
    
    TBR=vollick@chromium.org,mthiesse@chromium.org,cjgrant@chromium.org
    
    Change-Id: I920d8ffcb2dc7aad53ed021d272302cad060f083
    Reviewed-on: https://chromium-review.googlesource.com/763007
    Commit-Queue: Ian Vollick <vollick@chromium.org>
    Reviewed-by: default avatarIan Vollick <vollick@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#515465}
    76346c14
ui_input_manager.cc 15.8 KB