Fix header rerender bug which occurs when a table has two header rows…

[anthonyrgreen]

… updated simultaneously.

This is a proposed solution to #29922.

[google-cla]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[andrewseguin]

I may be missing something here, but it looks like this change is purely syntactical. The condition appears to be equivalent between the current and proposed code

[anthonyrgreen]

No, due to short circuiting, the existing implementation has lazy evaluation of the second branch, whereas my version has unconditional evaluation of the second branch.

…

On Thu, Nov 7, 2024 at 11:32 Andrew Seguin ***@***.***> wrote: I may be missing something here, but it looks like this change is purely syntactical. The condition appears to be equivalent between the current and proposed code — Reply to this email directly, view it on GitHub <#29921 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAUZIEMC54UIGHTDWIGO2ELZ7OI2VAVCNFSM6AAAAABQSAO456VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRSGY4TANZRGA> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}