• Calder Kitagawa's avatar
    Revert "[Paint Preview] Pre-warm compositor" · e99fa8aa
    Calder Kitagawa authored
    This reverts commit 3480cbb1.
    
    Reason for revert: This caused some performance/memory regressions that
    are far larger than expected.
    
    Original change's description:
    > [Paint Preview] Pre-warm compositor
    > 
    > This CL adds the ability to pre-warm a compositor process for use by
    > paint preview show-on-startup.
    > 
    > 1. The compositor is started when native is initialized if the
    >    experiment is enabled.
    > 2. Depending on the situation the compositor is then used or killed.
    >    a. Compositor is used: disconnect handler is updated and it is used
    >       as if there was a compositor already.
    >    b. Compositor is not used: the process is killed immediately.
    > 
    > This moves initialization of the compositor service up to O(100 ms)
    > earlier (in local tests).
    > 
    > Bug: 1126180
    > Change-Id: I1e6f627403e8c020b6551c510ad908bba0fc24a2
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2398562
    > Reviewed-by: Mehran Mahmoudi <mahmoudi@chromium.org>
    > Commit-Queue: Calder Kitagawa <ckitagawa@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#805317}
    
    TBR=mahmoudi@chromium.org,ckitagawa@chromium.org
    
    Change-Id: I7cd5067906f3833f9d927f9b3be1fe2600dbb741
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 1126180, 1126665, 1126734, 1126728, 1126648
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2403389Reviewed-by: default avatarCalder Kitagawa <ckitagawa@chromium.org>
    Commit-Queue: Calder Kitagawa <ckitagawa@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#805718}
    e99fa8aa
player_compositor_delegate.cc 11.3 KB