VideoCapturerAndroid: Changed camera freeze check to check that all frames are pending before repor…

VideoCapturerAndroid: Changed camera freeze check to check that all frames are pending before reporting a client error.

BUG=b/25514149

Committed: https://crrev.com/83dfad685322119ffe85ff5670cabbbaf385a111
Cr-Commit-Position: refs/heads/master@{#10563}

[perkj_webrtc, 2015-11-09 12:43:29]

perkj@webrtc.org changed reviewers:
+ magjed@webrtc.org

[perkj_webrtc, 2015-11-09 12:43:30]

please?

[magjed_webrtc, 2015-11-09 12:54:46]

https://codereview.webrtc.org/1423073006/diff/1/talk/app/webrtc/java/android/...
File talk/app/webrtc/java/android/org/webrtc/VideoCapturerAndroid.java (right):

https://codereview.webrtc.org/1423073006/diff/1/talk/app/webrtc/java/android/...
talk/app/webrtc/java/android/org/webrtc/VideoCapturerAndroid.java:141: if
(CAMERA_OBSERVER_PERIOD_MS * freezePeriodCount > CAMERA_FREEZE_REPORT_TIMOUT_MS
&&
&& should be on the next line

https://codereview.webrtc.org/1423073006/diff/1/talk/app/webrtc/java/android/...
talk/app/webrtc/java/android/org/webrtc/VideoCapturerAndroid.java:482: throw new
RuntimeException(e);
s/new RuntimeException(e)/e/g

[perkj_webrtc, 2015-11-09 13:46:58]

PTAL

https://codereview.webrtc.org/1423073006/diff/1/talk/app/webrtc/java/android/...
File talk/app/webrtc/java/android/org/webrtc/VideoCapturerAndroid.java (right):

https://codereview.webrtc.org/1423073006/diff/1/talk/app/webrtc/java/android/...
talk/app/webrtc/java/android/org/webrtc/VideoCapturerAndroid.java:141: if
(CAMERA_OBSERVER_PERIOD_MS * freezePeriodCount > CAMERA_FREEZE_REPORT_TIMOUT_MS
&&
On 2015/11/09 12:54:46, magjed_webrtc wrote:
> && should be on the next line

Done.

https://codereview.webrtc.org/1423073006/diff/1/talk/app/webrtc/java/android/...
talk/app/webrtc/java/android/org/webrtc/VideoCapturerAndroid.java:482: throw new
RuntimeException(e);
On 2015/11/09 12:54:46, magjed_webrtc wrote:
> s/new RuntimeException(e)/e/g

Done.

[magjed_webrtc, 2015-11-09 14:12:28]

lgtm

[perkj_webrtc, 2015-11-09 14:15:02]

The CQ bit was checked by perkj@webrtc.org

[commit-bot: I haz the power, 2015-11-09 14:15:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1423073006/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1423073006/20001

[commit-bot: I haz the power, 2015-11-09 15:39:52]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-09 15:40:01]

Patchset 2 (id:??) landed as
https://crrev.com/83dfad685322119ffe85ff5670cabbbaf385a111
Cr-Commit-Position: refs/heads/master@{#10563}