Add VideoSink interface to SurfaceViewRenderer.

Add VideoSink interface to SurfaceViewRenderer.

BUG=webrtc:7749, webrtc:7760
Review-Url: https://codereview.webrtc.org/3002563002
Cr-Commit-Position: refs/heads/master@{#19347}
Committed: https://chromium.googlesource.com/external/webrtc/+/6bdcefce80b42ddc7e8c754988f86c8e214aee83

[sakal, 2017-08-14 11:48:06]

The CQ bit was checked by sakal@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-08-14 11:48:15]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-08-14 12:17:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-08-14 12:17:07]

Dry run: Try jobs failed on following builders:
  android_rel on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/android_rel/builds/26542)

[sakal, 2017-08-14 13:01:30]

The CQ bit was checked by sakal@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-08-14 13:01:36]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[sakal, 2017-08-14 14:41:29]

Patchset #1 (id:1) has been deleted

[sakal, 2017-08-14 14:41:35]

Patchset #1 (id:20001) has been deleted

[sakal, 2017-08-14 14:41:56]

sakal@webrtc.org changed reviewers:
+ kthelgason@webrtc.org

[sakal, 2017-08-14 14:41:57]

PTAL

[kthelgason, 2017-08-15 07:45:24]

lgtm

https://codereview.webrtc.org/3002563002/diff/40001/webrtc/sdk/android/api/or...
File webrtc/sdk/android/api/org/webrtc/SurfaceViewRenderer.java (right):

https://codereview.webrtc.org/3002563002/diff/40001/webrtc/sdk/android/api/or...
webrtc/sdk/android/api/org/webrtc/SurfaceViewRenderer.java:374: post(new
Runnable() {
Can you not use the nice lambda syntax here?

[sakal, 2017-08-15 08:18:05]

The CQ bit was checked by sakal@webrtc.org

[sakal, 2017-08-15 08:18:06]

The patchset sent to the CQ was uploaded after l-g-t-m from
kthelgason@webrtc.org
Link to the patchset: https://codereview.webrtc.org/3002563002/#ps80001 (title:
"Address comments.")

[commit-bot: I haz the power, 2017-08-15 08:18:10]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[sakal, 2017-08-15 08:18:13]

https://codereview.webrtc.org/3002563002/diff/40001/webrtc/sdk/android/api/or...
File webrtc/sdk/android/api/org/webrtc/SurfaceViewRenderer.java (right):

https://codereview.webrtc.org/3002563002/diff/40001/webrtc/sdk/android/api/or...
webrtc/sdk/android/api/org/webrtc/SurfaceViewRenderer.java:374: post(new
Runnable() {
On 2017/08/15 07:45:24, kthelgason wrote:
> Can you not use the nice lambda syntax here?

Done.

[commit-bot: I haz the power, 2017-08-15 08:56:02]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1502785085834180,
"parent_rev": "ee21f374cac86ab4f963caff17ab17770ad9d1ff", "commit_rev":
"6bdcefce80b42ddc7e8c754988f86c8e214aee83"}

[commit-bot: I haz the power, 2017-08-15 08:56:19]

Description was changed from

==========
Add VideoSink interface to SurfaceViewRenderer.

BUG=webrtc:7749, webrtc:7760
==========

to

==========
Add VideoSink interface to SurfaceViewRenderer.

BUG=webrtc:7749, webrtc:7760

Review-Url: https://codereview.webrtc.org/3002563002
Cr-Commit-Position: refs/heads/master@{#19347}
Committed:
https://chromium.googlesource.com/external/webrtc/+/6bdcefce80b42ddc7e8c75498...
==========

[commit-bot: I haz the power, 2017-08-15 08:56:20]

Committed patchset #3 (id:80001) as
https://chromium.googlesource.com/external/webrtc/+/6bdcefce80b42ddc7e8c75498...