Use propsOrStateMapsEqual in memo so that tearoffs don't cause unnecessary rerenders

[aaronlademann-wf]

Motivation

Tearoffs aren't guaranteed to be identical, and we don't want the use of them to trigger unnecessary rerenders in memoized function components.

Closely related to how we handled this same problem for connected components

Changes

1. Use propsOrStateMapsEqual for areEqual instead of default JS equality when the consumer does not specify their own custom handler for the areEqual parameter.
2. Note that we're using our own utility to do the default shallow comparison in the doc comment for memo.
3. Add a regression test.

[aviary2-wf]

Security Insights

No security relevant content was detected by automated scans.

Action Items

• Review PR for security impact; comment "security review required" if needed or unsure
• Verify aviary.yaml coverage of security relevant code

---

Questions or Comments? Reach out on Slack: #support-infosec.

[greglittlefield-wf]

+10

@Workiva/release-management-p

[rmconsole-wf]

+1 from RM