• Wei-Yin Chen (陳威尹)'s avatar
    Do not limit concurrent readbacks in TabContentManager · f8ba2a26
    Wei-Yin Chen (陳威尹) authored
    The maximum number of concurrent readbacks was capped at 1. When a
    readback cannot be returned because the tab is no longer attached to
    the view, all the subsequent readback would be abandoned, causing
    blank or stale thumbnails in Grid Tab Switcher. When this happens,
    the symptom persists until the tab is reattached to the view or a
    restart. The tab can be reattached to the view by switching to it,
    but there is no way for a user to know which tab to switch to.
    
    This CL removes the limitation. The maximum number of concurrent
    readbacks is still limited by the number of tabs, since a tab can only
    have one readback. Removing the limitation should not increase the
    system loading because the extra concurrent readbacks would be in
    pending state waiting for their tabs to be reattached.
    
    Bug: 986047
    Change-Id: I741159e208757839bc25a9ead386a464e4759736
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1713847
    Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
    Reviewed-by: default avatarMatthew Jones <mdjones@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#683848}
    f8ba2a26
tab_content_manager.cc 15.5 KB