cc: Skip generating video release sync token if not needed

Desktop / Chromium - Sunny Sachanandani [chromium.org] - 2 August 2017 19:58 EDT

I removed the optimizations in SyncPointClientImpl where it'd skip generating sync tokens if there was no existing release sync token. That caused a 5% cpu time regression in the video benchmarks.

I also added some tests to verify that sync tokens are being generated at the right places.

R=jbauman BUG=750872

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel Change-Id: If739e3fb6705cf8b0739fc029ed3bd75dc43eabf Reviewed-on: https://chromium-review.googlesource.com/596424

390899a cc: Skip generating video release sync token if not needed.
cc/resources/video_resource_updater.cc | 32 ++++--
cc/resources/video_resource_updater_unittest.cc | 145 ++++++++++++++++++++----
2 files changed, 143 insertions(+), 34 deletions(-)

Upstream: git.chromium.org


  • Share