Running `clang-tidy '-checks=-*,performance-for-range-copy' ...` on Document.cpp and Element.cpp finds these unnecessary copies: Source/WebCore/dom/Document.cpp:7907:15: warning: loop variable is copied but only used as const reference; consider making it a const reference [performance-for-range-copy] for (auto observer : m_intersectionObserversWithPendingNotifications) { ^ const & Source/WebCore/dom/Element.cpp:1777:19: warning: loop variable is copied but only used as const reference; consider making it a const reference [performance-for-range-copy] for (auto observer : observerData->observers) { ^ const & Source/WebCore/dom/Element.cpp:3399:15: warning: loop variable is copied but only used as const reference; consider making it a const reference [performance-for-range-copy] for (auto observer : observerData->observers) ^ const &
<rdar://problem/46694035>
Created attachment 357226 [details] Patch v1
Comment on attachment 357226 [details] Patch v1 Clearing flags on attachment: 357226 Committed r239160: <https://trac.webkit.org/changeset/239160>
All reviewed patches have been landed. Closing bug.