| commit | 365c4b25fb7cfcebf166ca7b24d11561731f5b80 | |
| author | miu <miu@chromium.org> | |
| Sat, 7 Mar 2015 03:14:38 +0000 (6 19:14 -0800) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Sat, 7 Mar 2015 03:15:35 +0000 (7 03:15 +0000) | ||
| tree | 8cf134aaa48c1152eb12bb304c57e2ccd81731c0 | treesnapshot (tar.gz zip) |
| parent | 3849bd2b775e9fa55dc43412b17058ff0c1762f9 | commitdiff |
Remove software paint FrameSubscriber from tab capture.
All tab capture platforms have supported capture via event-driven
compositor updates for quite a while now. The "notification of backing
store change" code path never triggers, except in certain edge cases
where extra frame captures will be started. The extra frame captures
are bad for the user experience, since the extra frames reduce video
playback smoothness. The extra frame captures were also throwing-off
the "out-of-order frame delivery" logic, causing wanted frames to be
dropped.
BUG=464475
Review URL: https://codereview.chromium.org/989683004
Cr-Commit-Position: refs/heads/master@{#319549}
All tab capture platforms have supported capture via event-driven
compositor updates for quite a while now. The "notification of backing
store change" code path never triggers, except in certain edge cases
where extra frame captures will be started. The extra frame captures
are bad for the user experience, since the extra frames reduce video
playback smoothness. The extra frame captures were also throwing-off
the "out-of-order frame delivery" logic, causing wanted frames to be
dropped.
BUG=464475
Review URL: https://codereview.chromium.org/989683004
Cr-Commit-Position: refs/heads/master@{#319549}