-
Mohit Saini authored
This is a reland of commit c801d529 perfetto::DynamicString was supported in perfetto very recently. This CL failed earlier because latest perfetto was not rolled in v8. Relanding it after rolling the latest perfetto in v8 (see parent CL). Original change's description: > Refactor dynamic name to perfetto::DynamicString > > Recently perfetto introduced `perfetto::DynamicString` to allow clients > to wrap dynamic event name strings. So that clients don't have to > manually set event name inside trace lambda. > > With that: > > TRACE_EVENT("cat", nullptr, [&](EventContext ctx) { > ctx.event().set_name(dynamic_name_str) > }); > > is simplified to: > > TRACE_EVENT("cat", perfetto::DynamicString{dynamic_name_str}); > > In this change we are making use of perfetto::DynamicString to pass > dynamic event name string. > > Change-Id: Ic6b501df67409d6faa4d60b59095ad0e79ce585e > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3716473 > Reviewed-by: Manos Koukoutos <manoskouk@chromium.org> > Commit-Queue: Mohit Saini <mohitms@google.com> > Cr-Commit-Position: refs/heads/main@{#81298} Cq-Include-Trybots: luci.v8.try:v8_linux64_perfetto_dbg_ng Change-Id: I70103606f397cd420a5dfccba703bee8962f822b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3714946 Commit-Queue: Mohit Saini <mohitms@google.com> Reviewed-by: Michael Achenbach <machenbach@chromium.org> Auto-Submit: Mohit Saini <mohitms@google.com> Cr-Commit-Position: refs/heads/main@{#81420}
7ddb0d6b