• Daniele Castagna's avatar
    cc: Don't use BGRA_EXT with glCopyTexImage2d · bf7c3813
    Daniele Castagna authored
    When GLRenderer reads back from the framebuffer with glCopyTexImage2d,
    it tries to be smart about what internal format to use for the new texture.
    
    Unfortunately on CrOS it ends up using GL_BGRA_EXT, that seems to hit
    a particularly slow path in gles2_cmd_decoder.
    
    This CL changes it to RGB/RGBA, decreasing the CPU cost of CopyTexImage2d
    of two orders of magnitude when applying fullscreen backdrop effects.
    
    Bug: 900046
    
    Change-Id: I19bd8a9d008109e336f0fd346df130c2d7878154
    Reviewed-on: https://chromium-review.googlesource.com/c/1306653Reviewed-by: default avatarKristian H. Kristensen <hoegsberg@chromium.org>
    Commit-Queue: Daniele Castagna <dcastagna@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#604062}
    bf7c3813
gl_renderer.cc 142 KB