commit f11b04805eae4cbb004796ca97ee69c696f37865
author Fabian Sommer <fabiansommer@chromium.org> Tue Jun 06 12:05:06 2023
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Jun 06 12:05:06 2023
tree 5435dc980e52f94fc9b2a7a8409060674b10600d
parent 378f81709301b60fe4af221c4c36d955029e90ff

Revert "Reland "Pass soft navigation start time and others to browser""

This reverts commit 59194d3467e54b4c5a50dbdab19cda4c4f19eeb9.

Reason for revert: The added browser test is flaking on the Linux Tests
bot - https://ci.chromium.org/ui/p/chromium/builders/ci/Linux%20Tests/131341/overview

Original change's description:
> Reland "Pass soft navigation start time and others to browser"
>
> This is a reland of commit 690f964dd4deca36451fcc65d1b63b48adfd4ad8
>
> The previous revert is due to this test failure
> https://ci.chromium.org/ui/p/chromium/builders/ci/Linux%20MSan%20Tests/40448/overview.
> The reason is, in normal use cases, the
> PageLoadTracker::potential_ukm_source_id_for_soft_navigation would
> be initialized by a soft navigation detection. In that test, the
> soft navigation detection does not happen. And one side effect of the
> test uses this unitialized field. Hence the use-of-unitialized-value
> error on the Linux MSan bot.
>
> Original change's description:
> > Pass soft navigation start time and others to browser
> >
> > This CL is the first step to implement Ukm collection of soft
> > navigation metrics like LCP. This CL is to pass soft navigation start
> > time and renderer side navigation id along with soft navigation count
> > to the browser side.
> > After this CL is landed, the implementation of Ukm collection of soft navigation metrics LCP will be adapted to use the navigation id to know when a new soft navigation happens and record the metrics. https://chromium-review.googlesource.com/c/chromium/src/+/4329692
> >
> > See privacy review doc  https://docs.google.com/document/d/1FGk4lqfJvuiGCbB3TL7xS0PGYJnX-mmSmm1kDyDL1Wg/edit?usp=sharing&resourcekey=0-abA8bYWJi2R1kC2K1xy0RQ
> > and
> > https://docs.google.com/document/d/1NhqO4mWS3pEdUX_ePCrLpf29nfM-KK12NJzo39lR1fE/edit?usp=sharing
> >
> > Bug: 1370937
> > Change-Id: I464755e3fc8d5f1fce2ff19aed59fd0173692514
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4381537
> > Commit-Queue: Hao Liu <haoliuk@chromium.org>
> > Reviewed-by: Yoav Weiss <yoavweiss@chromium.org>
> > Reviewed-by: Robert Sesek <rsesek@chromium.org>
> > Cr-Commit-Position: refs/heads/main@{#1148594}
>
> Bug: 1370937
> Change-Id: I4e2a9bff68284ef14964c46fb7e38d53d82e6119
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4568705
> Reviewed-by: Sun Yueru <yrsun@chromium.org>
> Commit-Queue: Hao Liu <haoliuk@chromium.org>
> Reviewed-by: Rakina Zata Amni <rakina@chromium.org>
> Reviewed-by: Robert Sesek <rsesek@chromium.org>
> Reviewed-by: Yoav Weiss <yoavweiss@chromium.org>
> Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
> Cr-Commit-Position: refs/heads/main@{#1153246}

Bug: 1370937
Change-Id: If8b3c00a0c92deafb72821deddbb6d38fc60e913
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4590638
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Commit-Queue: Fabian Sommer <fabiansommer@chromium.org>
Owners-Override: Fabian Sommer <fabiansommer@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1153783}