tree 17f9dd10ab9532a81bed83998de1859af3728b4e
parent 750efbe5cec97caa7c2898cfdc3df8ece603f05b
author Taylor Brandstetter <deadbeef@webrtc.org> 1527709210 -0700
committer Commit Bot <commit-bot@chromium.org> 1527755925 +0000

Fixing null rtc::Optional dereference in VideoQualityObserver.

Was crashing if |is_paused_| is true for the first few frames,
resulting in |interframe_delays_| being given fewer samples than
|num_frames_decoded_|. So checking |num_frames_decoded_| wasn't
sufficient; really should just check if |interframe_delays_.Avg|
returns a nullopt or not.

Bug: webrtc:9338
Change-Id: Ie74e88f7ec5ecef85a07145b9576f54b2a089f63
Reviewed-on: https://webrtc-review.googlesource.com/80040
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Reviewed-by: Åsa Persson <asapersson@webrtc.org>
Commit-Queue: Ilya Nikolaevskiy <ilnik@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#23463}