Commit a84a2200 authored Jan 12, 2022 by Vishnu Nair

SF: Update static sequence counter when injecting a sequence number

When generating layer trace, layers will be created with a sequence
number. However any supporting layer that is created (when mirroring
or color layer for blast) will use the static counter to gets its
sequence number. This might result in layers with duplicate sequence
numbers.

Fix this by updating the static sequence counter if a sequence number
is injected.

Test: atest TransactionTracingTest
Bug: 200284593
Change-Id: Ib64227a088d1cbf76b977604eaeb3b1421af63f5

parent 7b0f18b0

services/surfaceflinger/Layer.cpp

+3 −1

Original line number	Diff line number	Diff line
		@@ -101,7 +101,9 @@ Layer::Layer(const LayerCreationArgs& args)
		if (args.flags & ISurfaceComposerClient::eSecure) layerFlags \|= layer_state_t::eLayerSecure;
		if (args.flags & ISurfaceComposerClient::eSkipScreenshot)
		layerFlags \|= layer_state_t::eLayerSkipScreenshot;

		if (args.sequence) {
		sSequence = *args.sequence + 1;
		}
		mDrawingState.flags = layerFlags;
		mDrawingState.active_legacy.transform.set(0, 0);
		mDrawingState.crop.makeInvalid();