• Michael Thiessen's avatar
    Revert "VR: Implement quad buffer for content quad." · bae94e3a
    Michael Thiessen authored
    This reverts commit 1e24d20a.
    
    Reason for revert: Tanks performance. Quad layers are apparently expensive, yo.
    
    Original change's description:
    > VR: Implement quad buffer for content quad.
    > 
    > Huge thanks to bajones for building the prototype I built this CL
    > off of, and for debugging for me ;)
    > 
    > Uses a separate viewport with transform to avoid resampling the
    > content quad when drawing it into our scene. The viewport is placed
    > underneath the viewport for the other browser UI, and a transparent
    > hole is punched through to the underlying viewport.
    > 
    > Also avoids toggling multisampling at runtime (which recreates the
    > swap chain) by re-using the two buffers (one multisampled and one
    > not) across browsing and webVR modes.
    > 
    > Bug: 714304
    > 
    > Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_vr
    > Change-Id: Ibcbb2752aaa957c3f904d4b76bf6dd5fdf9164c8
    > Reviewed-on: https://chromium-review.googlesource.com/1017589
    > Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
    > Reviewed-by: Ian Vollick <vollick@chromium.org>
    > Reviewed-by: Brandon Jones <bajones@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#552212}
    
    TBR=vollick@chromium.org,mthiesse@chromium.org,bajones@chromium.org
    
    Change-Id: Iea71c2d624193c5f90c7554ee212d28fb526bcfc
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 714304
    Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_vr
    Reviewed-on: https://chromium-review.googlesource.com/1021972Reviewed-by: default avatarMichael Thiessen <mthiesse@chromium.org>
    Commit-Queue: Michael Thiessen <mthiesse@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#552333}
    bae94e3a
content_element.cc 8.25 KB