This CL introduces Perfetto-based tracing in the Digital Credentials API browser-side implementation for the create() method. The tracing captures critical points in the request lifecycle to aid debugging and performanceplease wrap this line
"Traces for the Digital Credntials API"),typo
TRACE_EVENT_INSTANT("content.digitalcredentials", "OnCompleteRequest:Get",since this is now not in an if block anymore, you no longer need the _INSTANT
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
This CL introduces Perfetto-based tracing in the Digital Credentials API browser-side implementation for the create() method. The tracing captures critical points in the request lifecycle to aid debugging and performanceAnushka Kulkarniplease wrap this line
Done
"Traces for the Digital Credntials API"),Anushka Kulkarnitypo
Done
TRACE_EVENT_INSTANT("content.digitalcredentials", "OnCompleteRequest:Get",since this is now not in an if block anymore, you no longer need the _INSTANT
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
TRACE_EVENT("content.digitalcredentials", "OnCompleteRequest:Get", "status",This is invoked both Get and Create flows.
We have the same issue in the Get tracing CL.
I do think we need to condition it on the request_type or make it generic (e.g. OnCompleteRequest)
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
TRACE_EVENT("content.digitalcredentials", "OnCompleteRequest:Get", "status",This is invoked both Get and Create flows.
We have the same issue in the Get tracing CL.I do think we need to condition it on the request_type or make it generic (e.g. OnCompleteRequest)
Made it generic as adding the conditions changes the scope of the traces.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
LGTM
But maybe rebase this on top of https://crrev.com/c/7127319 and some code will be disappear.
Either do it now, or land one after the other!
Thank you!
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
LGTM
Thank you!
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Hi Jan,
Could you please take a look?
(We need another +1)
Thank you,
Mohamed
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Add tracing to Digital Credentials API create() flow
This CL introduces Perfetto-based tracing in the Digital Credentials API
browser-side implementation for the create() method. The tracing
captures critical points in the request lifecycle to aid debugging and
performance analysis.
Trace category: "content.digitalcredentials"
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |