Fix clang tidy bugprone-move-forwarding-reference static analyzer warnings in CompletionHandler.h. Found by clang static analyzer.
<rdar://problem/88529996>
Created attachment 450995 [details] Patch v1
Created attachment 450996 [details] Patch v2
(In reply to David Kilzer (:ddkilzer) from comment #2) > Created attachment 450995 [details] > Patch v1 Oops. Changed std::forward() to std::forward<CallableType>().
Comment on attachment 450996 [details] Patch v2 Marking xq+ since enough builds and tests have run to show this didn't cause a regression.
Committed r289162 (246858@main): <https://commits.webkit.org/246858@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 450996 [details].
Comment on attachment 450996 [details] Patch v2 r=me, yes, it can be a lref or rref (b/c CallableType is template type of this function), thus we should use std::forward.
Ah, oops it was already landed.