> In TEST_F(BufferQueueTest, slow_client_framerate_matches_compositor) we're
> removing the expectation ASSERT_THAT(client_frames, Lt(compose_frames * 1.2f))
> - which means we're no longer validating that the "client framerate matches
> compositor".
>
> Maybe we just need a better name for the test?
> In TEST_F( BufferQueueTest , slow_client_ framerate_ matches_ compositor) we're THAT(client_ frames, Lt(compose_frames * 1.2f))
> removing the expectation ASSERT_
> - which means we're no longer validating that the "client framerate matches
> compositor".
>
> Maybe we just need a better name for the test?
I've added back the upper bound assertion.