commit 546bbc227501b51a7aac4e007a7e9341f3c90f2b
author Oksana Zhuravlova <oksamyt@chromium.org> Mon Aug 19 15:46:43 2024
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon Aug 19 15:46:43 2024
tree ee51f7cb0ecbb279bd40c230c44a9ff36ae42179
parent b8c4b1ede5565a389c3b085d31a4d55ebac439e1

Reland "Reland "Reland "[tracing] Add Metadata source for concurrent sessions"""

This reverts commit 7e88219699418aa98746887ede1d1c9d929e19ca.

Reason for revert: updated the test to select an int instead of a string

Original change's description:
> Revert "Reland "Reland "[tracing] Add Metadata source for concurrent sessions"""
>
> This reverts commit 7348b291ba3b831d90386f456ad0f4a9ca8e81e2.
>
> Reason for revert: CI failures, see crbug.com/360068621
>
> Original change's description:
> > Reland "Reland "[tracing] Add Metadata source for concurrent sessions""
> >
> > This reverts commit d45de7951c8dd61f355f383b912ee19080d2d500.
> >
> > Reason for revert: addressed the build failures
> >
> > Original change's description:
> > > Revert "Reland "[tracing] Add Metadata source for concurrent sessions""
> > >
> > > This reverts commit fcce1248ad8a888a036da9163b978faa750146f1.
> > >
> > > Reason for revert: https://crbug.com/358162540
> > > https://ci.chromium.org/ui/p/chromium/builders/ci/chromeos-arm64-generic-rel/46983/overview
> > >
> > > See also https://b.corp.google.com/issues/356930672#comment10
> > >
> > > Original change's description:
> > > > Reland "[tracing] Add Metadata source for concurrent sessions"
> > > >
> > > > This reverts commit 119504c12f148dfd0a631ea273c3fd3e88b0fefb.
> > > >
> > > > The reland CL updates the test to skip the field trials check if there are no field trials
> > > >
> > > > Original change's description:
> > > > > Revert "[tracing] Add Metadata source for concurrent sessions"
> > > > >
> > > > > This reverts commit d2f50017e1d72fe55b65c32937b36bf44adec29b.
> > > > >
> > > > > Reason for revert: [Gardener] Failing on ci/linux-chromeos-chrome (eg: https://ci.chromium.org/ui/p/chrome/builders/ci/linux-chromeos-chrome/43660/overview)
> > > > >
> > > > > Original change's description:
> > > > > > [tracing] Add Metadata source for concurrent sessions
> > > > > >
> > > > > > This change adds a new MetadataDataSource class that implements basic
> > > > > > metadata (currently only version code and field trial hashes) for
> > > > > > concurrent sessions. Metadata is recorded at the end of the session.
> > > > > >
> > > > > > The new data source is called 'org.chromium.trace_metadata2'
> > > > > > to avoid collisions with the existing 'org.chromium.trace_metadata'
> > > > > > source.
> > > > > >
> > > > > > Bug: b:354749410
> > > > > > Change-Id: I0b385d7248b2f5d0e73a0c491894f5e9969a522f
> > > > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5736232
> > > > > > Reviewed-by: Mark Pearson <mpearson@chromium.org>
> > > > > > Reviewed-by: Alexander Timin <altimin@chromium.org>
> > > > > > Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org>
> > > > > > Cr-Commit-Position: refs/heads/main@{#1336121}
> > > > >
> > > > > Bug: b:354749410
> > > > > Change-Id: I01eae482d55b83229ce92c051c5142289eb4db63
> > > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5757585
> > > > > Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> > > > > Auto-Submit: Brad Triebwasser <btriebw@chromium.org>
> > > > > Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> > > > > Owners-Override: Brad Triebwasser <btriebw@chromium.org>
> > > > > Cr-Commit-Position: refs/heads/main@{#1336271}
> > > >
> > > > Bug: b:354749410
> > > > Change-Id: I0defc16ec324a1efa1f0ba3566f9a7d03f2aee87
> > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5762075
> > > > Reviewed-by: Mark Pearson <mpearson@chromium.org>
> > > > Reviewed-by: Alexander Timin <altimin@chromium.org>
> > > > Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org>
> > > > Cr-Commit-Position: refs/heads/main@{#1338533}
> > >
> > > Bug: b:354749410, 358162540
> > > Change-Id: I24f4e3eb0bca0b08b769afacb31ce333bba5cbf4
> > > No-Presubmit: true
> > > No-Tree-Checks: true
> > > No-Try: true
> > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5770699
> > > Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> > > Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
> > > Owners-Override: Dave Tapuska <dtapuska@chromium.org>
> > > Auto-Submit: Ben Pastene <bpastene@chromium.org>
> > > Reviewed-by: Orin Jaworski <orinj@chromium.org>
> > > Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
> > > Cr-Commit-Position: refs/heads/main@{#1338644}
> >
> > Bug: b:354749410, 358162540
> > Change-Id: Ic4334857b09dc5f486fdf8c6b54efb53e22672df
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5773445
> > Reviewed-by: Mark Pearson <mpearson@chromium.org>
> > Reviewed-by: Alexander Timin <altimin@chromium.org>
> > Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org>
> > Cr-Commit-Position: refs/heads/main@{#1341867}
>
> Bug: b:354749410, 358162540
> Change-Id: If6f88069c4786a6ce3bf29359ee32cbce82e268e
> Fixed: 360068621
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5789123
> Auto-Submit: Victor Vianna <victorvianna@google.com>
> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
> Owners-Override: Victor Vianna <victorvianna@google.com>
> Commit-Queue: Victor Vianna <victorvianna@google.com>
> Cr-Commit-Position: refs/heads/main@{#1342202}

Bug: b:354749410, 358162540
Change-Id: I9cda2950fa28efe94949a42dad9091f55688c767
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5791919
Commit-Queue: Oksana Zhuravlova <oksamyt@chromium.org>
Reviewed-by: Mark Pearson <mpearson@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Reviewed-by: Victor Vianna <victorvianna@google.com>
Cr-Commit-Position: refs/heads/main@{#1343531}