There is no need to call `List.contains` before `List.remove` call. `history` is an either `horizontalHistory` or `verticalHistory`. Both of them are ArrayList's.
------------- Commit messages: - [PATCH] Remove unnecessary List.contains key from SpringLayout.Constraints.pushConstraint Changes: https://git.openjdk.org/jdk/pull/24852/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=24852&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8362067 Stats: 5 lines in 1 file changed: 0 ins; 3 del; 2 mod Patch: https://git.openjdk.org/jdk/pull/24852.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/24852/head:pull/24852 PR: https://git.openjdk.org/jdk/pull/24852