commit 671b2ad5ed3181ca2fccd72fe09b714ed4037c7c
author Kevin Haslett <khaslett@chromium.org> Wed Dec 06 00:29:37 2023
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Dec 06 00:29:37 2023
tree a25dd674ef328ee5a6fa9a4294331d5e4c512617
parent e2f4e0891a98621137b2ce76addb525acf4b7f0d

Skip wl_surface.commit when there are no changes to the (sub)surface

Also move presentation_feedback and frame_callback to the root surface
because it will already be committed each frame.

When measured[1] on a 30fps (not fullscreen) video test page I observed
far fewer total commits. And a modest time savings on the Lacros side:

Before:
30 frames played back in 1s. Totals:
  Total Commits: 600
  Average commits per frame: 20
  Average commit time per frame: 3e-05 s
  Average playback time per frame: 0.000106 s

After:
30 frames played back in 1s. Totals:
  Total Commits: 60
  Average commits per frame: 2
  Average commit time per frame: 1.9e-05 s
  Average playback time per frame: 9.9e-05 s

[1] https://crrev.com/c/5078286

Change-Id: I9bf17662568b8cc10527c3381ff1fc553d909904
Bug: 1506309
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5074674
Commit-Queue: Kevin Haslett <khaslett@chromium.org>
Reviewed-by: Kramer Ge <fangzhoug@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1233677}