The `activeBenchmarks` reference can change anytime, so I think there should be a
```java
final ImmutableOpenMap<String, BenchmarkState> activeBenchmarks = this.activeBenchmarks;
```
at the beginning of the method to make sure we are always talking to the same instance.