• Mark Yacoub's avatar
    Revert "Skip promoting SingleOnTop Overlay when contents don't change" · f213924a
    Mark Yacoub authored
    This reverts commit b4fd0cc9.
    
    Reason for revert: The Squid app uses SingleOnTop for overlay without changing the resource_id. This revert will undo the fix for the other apps, but they're lower priority.
    
    Original change's description:
    > Skip promoting SingleOnTop Overlay when contents don't change
    >
    > If a candidate content has not changed for 3 frames, do not promote it as
    > Single On Top overlay as it can be just be flattened, allowing other
    > strategies to kick in if possible.
    >
    > Bug: 885180
    > Test: SingleOverlayOnTopTest.DoNotPromoteIfPreviouslyPromoted
    > Change-Id: I516dab628c129fe9fae09c713e2783a6ce889c56
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1521591
    > Commit-Queue: Mark Yacoub <markyacoub@google.com>
    > Reviewed-by: Daniele Castagna <dcastagna@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#645367}
    
    TBR=dcastagna@chromium.org,markyacoub@google.com
    
    # Not skipping CQ checks because original CL landed > 1 day ago.
    
    Bug: 885180
    Change-Id: I799bd22f06a56d86844a63d5ac346f41bcfb8707
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1635931Reviewed-by: default avatarDaniele Castagna <dcastagna@chromium.org>
    Commit-Queue: Mark Yacoub <markyacoub@google.com>
    Cr-Commit-Position: refs/heads/master@{#664734}
    f213924a
overlay_unittest.cc 160 KB