Return from estimateCaptureFps after timeout even if no frames are actually captured.

[ian-arkver]

Fixes FPS Test button causing OpenPnP to hang
if camera is lost/unplugged or stops receiving frames.

Description

estimateCameraFps function has a while loop which only exits when the camera indicates it has a new frame.

Assuming hasNewFrame returns false when the camera is unplugged then exiting the while loop when the timeout expires will prevent OpenPnP from hanging endlessly.

Justification

Hopefully fixes bug reported on the Google group.

Instructions for Use

N/A. Problem was seen on loss of camera during an FPS test.

Implementation Details

1. How did you test the change? Compile tested only - I don't use OpenPnPCaptureCamera and this FPS test is specific to that.
2. Coding style followed.
3. No spi or model changes.
4. Maven tests passed, but don't cover the changed code.

[markmaker]

Hi Ian,

Thanks for this! 💯💯💯

_Mark