• Dale Curtis's avatar
    Revert "Add event cache size limit to media devtools" · a00162ef
    Dale Curtis authored
    This reverts commit 79e55e3e.
    
    Reason for revert: Fix failed to land in time.
    
    Original change's description:
    > Add event cache size limit to media devtools
    >
    > This adds a cache for both total number of players as well as total
    > number of events kept by the each individual player, in order to prevent
    > long-living pages from creating an otherwise unbounded number of events
    > and creating memory pressure.
    >
    > There is some logic for selecting which players to prune, but no logic
    > to do anything special for events yet.
    >
    > Events, Messages, and Errors are all subject to the same cache size
    > limits, but the Properties can't grow unbounded so they aren't.
    >
    > R=​dalecurtis
    >
    > Bug: 1145392
    > Change-Id: Iab39207276b6179b9f8828df7ced210154ab97dc
    > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2517928
    > Commit-Queue: Ted Meyer <tmathmeyer@chromium.org>
    > Reviewed-by: Kent Tamura <tkent@chromium.org>
    > Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#827070}
    
    TBR=dalecurtis@chromium.org,caseq@chromium.org,tkent@chromium.org,tmathmeyer@chromium.org
    
    # Not skipping CQ checks because original CL landed > 1 day ago.
    
    Bug: 1145392
    Change-Id: I40b2631948aadcb3643595681b222a5b6805a6dd
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2547403Reviewed-by: default avatarDale Curtis <dalecurtis@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#828775}
    a00162ef
inspector_media_event_handler.cc 3.83 KB