tree 549889c7aaf6a17d5c6bbf9ced7c2915320ad5c6
parent 568ff8adda1eb0e76fe01f0af31cb254a932cf9d
author Zengfeng Ye <zenye@microsoft.com> 1713588424 +0000
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> 1713588424 +0000

Revert "[iOS] Avoid forcing layout location bar for unanimated expand/contract"

This reverts commit d12bc76a4e72515de379457c19d06d2809ac8f50.

Reason for revert: Caused a UIViewKeyframeAnimations crash
(https://g-issues.chromium.org/issues/333331927), and it can't be fixed
soon, revert it first.

Original change's description:
> [iOS] Avoid forcing layout location bar for unanimated expand/contract
>
> Expand/contract location bar will force layout immediately with a call
> to `layoutIfNeeded`, but the layout is not necessary at startup. The
> layout adds ~25ms to the startup latency on the iPhone 7 (according to
> Instruments).
> This CL avoids forcing layout location bar for unanimated
> expand/contract to reduce unnecessary costs at startup.
>
> Instruments measurements:
> https://docs.google.com/document/d/1gyVFWPn2WDPIyi5zY1WFUjhsL2Z26RsUyHLNw87lIWE/edit?usp=sharing
>
> Bug: 1515588
> Change-Id: I57528e590a45e0e77c890cd264cb1b1f9e749f85
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5252574
> Reviewed-by: Joemer Ramos <joemerramos@chromium.org>
> Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
> Commit-Queue: Zengfeng Ye <zenye@microsoft.com>
> Cr-Commit-Position: refs/heads/main@{#1254871}

Bug: 1515588
Change-Id: I560d5bf9d878c7728dadfcbdad00f86276669776
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5465605
Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Commit-Queue: Zengfeng Ye <zenye@microsoft.com>
Reviewed-by: Joemer Ramos <joemerramos@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1290309}