Commit a1e1af8d authored by Peng Huang's avatar Peng Huang Committed by Commit Bot

Reland "SkiaOutputdeviceBufferQueue: kick off next frame earlier"

The original change was reverted for crbug.com/1052790 (
webgl_conformance_tests on Android device Nexus 5 failing),
However the change is only used for Android 10, it should not affect
Nexus 5. So the CL is not related to the problem.

This is a reland of 75b5db21

Original change's description:
> SkiaOutputdeviceBufferQueue: kick off next frame earlier
>
> PageFlipComplete() is not trivial, so call FinishSwapBuffers()
> before it to kick off next frame in viz thread earlier, and compositor
> will have more time budget to render the next frame.
>
> Bug: None
> Change-Id: Ie6e0f87d75d81b405447757585ec291887ca0063
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2057043
> Commit-Queue: Vasiliy Telezhnikov <vasilyt@chromium.org>
> Auto-Submit: Peng Huang <penghuang@chromium.org>
> Reviewed-by: Vasiliy Telezhnikov <vasilyt@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#741430}

Bug: None
Change-Id: Ic062caff8493e1af63fe4b1c4788f0eb5c53e7f1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2069844Reviewed-by: default avatarVasiliy Telezhnikov <vasilyt@chromium.org>
Commit-Queue: Peng Huang <penghuang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#743923}
parent 2864cb50
...@@ -542,8 +542,8 @@ void SkiaOutputDeviceBufferQueue::DoFinishSwapBuffers( ...@@ -542,8 +542,8 @@ void SkiaOutputDeviceBufferQueue::DoFinishSwapBuffers(
std::unique_ptr<gfx::GpuFence> gpu_fence) { std::unique_ptr<gfx::GpuFence> gpu_fence) {
DCHECK(!gpu_fence); DCHECK(!gpu_fence);
PageFlipComplete(image.get());
FinishSwapBuffers(result, size, latency_info); FinishSwapBuffers(result, size, latency_info);
PageFlipComplete(image.get());
} }
bool SkiaOutputDeviceBufferQueue::Reshape(const gfx::Size& size, bool SkiaOutputDeviceBufferQueue::Reshape(const gfx::Size& size,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment