use contains for membership test

Summary:
This change might make the intention of the code a bit more clearer.
More importantly, calling find (as opposed to constFind) on the temporary
could detach the container, which is wholly unnecessary

Reviewers: Elisa, mgallien

Reviewed By: Elisa, mgallien

Differential Revision: https://phabricator.kde.org/D10953