Skip to content

protovm: adds the surfaceflinger patch to the trace packet#5451

Merged
BogdanFloris merged 1 commit intomainfrom
dev/bogdanfloris/surfaceflinger-patch-field
Apr 15, 2026
Merged

protovm: adds the surfaceflinger patch to the trace packet#5451
BogdanFloris merged 1 commit intomainfrom
dev/bogdanfloris/surfaceflinger-patch-field

Conversation

@BogdanFloris
Copy link
Copy Markdown
Contributor

@BogdanFloris BogdanFloris commented Apr 14, 2026

Adds the LayersPatchProto message to the winscope extensions proto message.

@BogdanFloris BogdanFloris requested a review from rukkal April 14, 2026 12:22
@BogdanFloris BogdanFloris self-assigned this Apr 14, 2026
@LalitMaganti
Copy link
Copy Markdown
Member

Why is this being added here instead of an extension? We are reducing how much stuff we put in the main TracePacket instead, putting more things in extensions.

@BogdanFloris BogdanFloris force-pushed the dev/bogdanfloris/surfaceflinger-patch-field branch 6 times, most recently from 92b7d1d to b65f12c Compare April 14, 2026 13:24
@BogdanFloris
Copy link
Copy Markdown
Contributor Author

Wasn't aware we should be adding this to winscope extensions, thanks for the notice. Just updated the pr.

@BogdanFloris BogdanFloris force-pushed the dev/bogdanfloris/surfaceflinger-patch-field branch from b65f12c to 8d447ad Compare April 15, 2026 07:16
@BogdanFloris BogdanFloris requested a review from a team April 15, 2026 07:33
@BogdanFloris BogdanFloris merged commit a22fdf7 into main Apr 15, 2026
23 checks passed
@BogdanFloris BogdanFloris deleted the dev/bogdanfloris/surfaceflinger-patch-field branch April 15, 2026 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants