Commit 22c674ea authored by Nathan Zabriskie's avatar Nathan Zabriskie Committed by Commit Bot

Increment |frame_token|'s in ClusterFuzz CompositorFrames

This change adds FrameTokenGenerators to CompositorFrame builders in
ClusterFuzz. During normal operation subsequent CompositorFrames are
expected to have incremented |frame_token|'s, this change makes fuzzed
frames follow that behavior.

Bug: 997147
Change-Id: Ib41ac8318bb2c14edfd0d9b51061f3fbde490df4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787992Reviewed-by: default avatarkylechar <kylechar@chromium.org>
Reviewed-by: default avatarMax Moroz <mmoroz@chromium.org>
Commit-Queue: Nathan Zabriskie <nazabris@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#695679}
parent 0e72fa06
......@@ -153,12 +153,14 @@ class FuzzedCompositorFrameBuilder {
FuzzedData FuzzedCompositorFrameBuilder::Build(
const proto::RenderPass& render_pass_spec) {
static FrameTokenGenerator next_frame_token;
data_.frame.metadata.begin_frame_ack.source_id =
BeginFrameArgs::kManualSourceId;
data_.frame.metadata.begin_frame_ack.sequence_number =
BeginFrameArgs::kStartingFrameNumber;
data_.frame.metadata.begin_frame_ack.has_damage = true;
data_.frame.metadata.frame_token = 1;
data_.frame.metadata.frame_token = ++next_frame_token;
data_.frame.metadata.device_scale_factor = 1;
data_.frame.metadata.local_surface_id_allocation_time =
base::TimeTicks::Now();
......
......@@ -113,7 +113,7 @@ CompositorFrame FuzzerBrowserProcess::BuildBrowserUICompositorFrame(
SurfaceId renderer_surface_id) {
CompositorFrame frame;
frame.metadata.frame_token = 1;
frame.metadata.frame_token = ++next_frame_token_;
frame.metadata.begin_frame_ack.source_id = BeginFrameArgs::kManualSourceId;
frame.metadata.begin_frame_ack.sequence_number =
BeginFrameArgs::kStartingFrameNumber;
......
......@@ -58,6 +58,8 @@ class FuzzerBrowserProcess {
ParentLocalSurfaceIdAllocator lsi_allocator_;
FrameTokenGenerator next_frame_token_;
DISALLOW_COPY_AND_ASSIGN(FuzzerBrowserProcess);
};
......
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