There is an ambiguity in the documentation regarding whether calling stop() resets the decoder metrics (e.g., METRIC_FRAMES_DECODED).

[hari22yuva]

There is an ambiguity in the documentation regarding whether calling stop() resets the decoder metrics (e.g., METRIC_FRAMES_DECODED).
Reference 1: Property.aidl The documentation for METRIC_FRAMES_DECODED explicitly states that the metric is reset only on open() and flush():

"This metric is reset on open() and flush() calls."

Reference 2: IVideoDecoderController.aidl The documentation for stop() states that stop() is effectively a flush:

"The decoder enters the STOPPING state... This is effectively the same as a flush."

Question Does stop() implicitly trigger the same metric reset as flush(), or does "effectively the same as a flush" only refer to the buffer management (freeing pending buffers)?

If stop() is intended to reset metrics, Property.aidl should likely be updated to include stop() in the list of reset triggers to avoid vendor implementation discrepancies.

[hari22yuva]

current discussion out put is
stop also will clear the metrics. so need to update the Property.aidl should updated with stop() also reset the metrics .

[hari22yuva]

created :
#305

[Ulrond]

Concluded — stop() resets metrics along with open() and flush(). The Property.aidl documentation has been updated to reflect this. Closing.