• Eric Karl's avatar
    Revert "Use GL_COMMANDS_ISSUED_CHROMIUM query for shared memory GpuMemoryBuffers" · 6eb9eee3
    Eric Karl authored
    This reverts commit 370c8bc2.
    
    Reason for revert: This is breaking desktop sites (SW raster) on Android when OOP-R is enabled. see crbug.com/879823
    
    Original change's description:
    > Use GL_COMMANDS_ISSUED_CHROMIUM query for shared memory GpuMemoryBuffers
    > 
    > GL_COMMANDS_ISSUED_CHROMIUM is sufficient for shared memory
    > GpuMemoryBuffers because they're uploaded using glTexImage2D (see
    > GLImageMemory::CopyTexImage).
    > 
    > This should reduce the number of idle wakeups in the GPU process, and
    > reduce the CPU time spent on such queries which is quite high on
    > Windows.
    > 
    > Bug: 830084, 622491
    > Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
    > Change-Id: I8111b3d7f12abbf900383d45e2a74394848e83fd
    > Reviewed-on: https://chromium-review.googlesource.com/1189126
    > Commit-Queue: Sunny Sachanandani <sunnyps@chromium.org>
    > Reviewed-by: Eric Karl <ericrk@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#587695}
    
    TBR=sunnyps@chromium.org,ericrk@chromium.org
    
    # Not skipping CQ checks because original CL landed > 1 day ago.
    
    Bug: 830084, 622491, 879823
    Change-Id: I7c78d2cc92e9ad274e8f443969d2a72d2410d5db
    Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
    Reviewed-on: https://chromium-review.googlesource.com/1199629Reviewed-by: default avatarEric Karl <ericrk@chromium.org>
    Commit-Queue: Eric Karl <ericrk@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#588274}
    6eb9eee3
one_copy_raster_buffer_provider.cc 21.2 KB