• ericrk's avatar
    Revert of Idle cleanup for worker context (patchset #5 id:200001 of... · a5ce62a7
    ericrk authored
    Revert of Idle cleanup for worker context (patchset #5 id:200001 of https://codereview.chromium.org/2353033003/ )
    
    Reason for revert:
    This appears to be breaking some number of unittests - need to investigate further.
    
    Original issue's description:
    > Idle cleanup for worker context
    >
    > Currently, Skia flushes unused items out of its caches after ~1 second
    > of non-use (50 flushes). This works fine while Skia is in-use, but when
    > a worker context goes idle we stop calling into Skia altogether. This
    > means that Skia will never flush out unused cache items from the last
    > piece of work it did. This can lead to some rather large temporaries
    > being kept around (~20mb).
    >
    > This change adds an idle cleanup process which flushes Skia's caches
    > and cleans up worker context resources after a worker context is idle
    > for 1 second.
    >
    > BUG=624630
    > CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
    >
    > Committed: https://crrev.com/051ae83bf29b52cefd82235ebfb90f203912afbb
    > Cr-Commit-Position: refs/heads/master@{#420496}
    
    TBR=danakj@chromium.org,jbauman@chromium.org
    # Skipping CQ checks because original CL landed less than 1 days ago.
    NOPRESUBMIT=true
    NOTREECHECKS=true
    NOTRY=true
    BUG=624630
    
    Review-Url: https://codereview.chromium.org/2366033002
    Cr-Commit-Position: refs/heads/master@{#420741}
    a5ce62a7
context_cache_controller.h 2.45 KB