1. 26 Oct, 2019 1 commit
    • Eric Karl's avatar
      Revert "DeJelly - Fix SurfaceAggregator's handling of non-merged OOPIF surfaces" · 8766bc4b
      Eric Karl authored
      This reverts commit 8ccc248e.
      
      Reason for revert: This fix was quick / imperfect due to merge constraints. A more complete fix will be landed following this revert.
      
      Original change's description:
      > DeJelly - Fix SurfaceAggregator's handling of non-merged OOPIF surfaces
      > 
      > In cases where a SurfaceDrawQuad's root render pass was not merged into
      > its parent, we did not correctly handle cases where the root render
      > pass represented a subset of the SurfaceDrawQuad's area. This was
      > possible in OOPIF cases where the root render pass was clipped to
      > visible bounds.
      > 
      > This CL adjusts the root render pass so that it always represents the
      > full SurfaceDrawQuad area.
      > 
      > While it's possible to hit this bug without de-jelly, this is much
      > more common with de-jelly, so the change is restricted to reduce merge
      > risk. A  follow-up will implement this change more generally.
      > 
      > Bug: 1016677, 995965
      > Change-Id: If733065277128d53f0b59483f167d177aa777f54
      > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1873277
      > Commit-Queue: Eric Karl <ericrk@chromium.org>
      > Reviewed-by: Victor Miura <vmiura@chromium.org>
      > Cr-Commit-Position: refs/heads/master@{#708288}
      
      TBR=vmiura@chromium.org,ericrk@chromium.org
      
      # Not skipping CQ checks because original CL landed > 1 day ago.
      
      Bug: 1016677, 995965
      Change-Id: I13b953da9b96bd4a111a393b448359d08be18b38
      Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1879696Reviewed-by: default avatarEric Karl <ericrk@chromium.org>
      Commit-Queue: Eric Karl <ericrk@chromium.org>
      Cr-Commit-Position: refs/heads/master@{#709705}
      8766bc4b
  2. 25 Oct, 2019 39 commits