EGL10.eglCreateWindowSurface(): Replace Surface input with SurfaceHolder

EGL10.eglCreateWindowSurface(): Replace Surface input with SurfaceHolder

Sending a Surface as input to EGL10.eglCreateWindowSurface() is not supported everywhere. See this code as reference:
https://android.googlesource.com/platform/frameworks/native.git/+/ae9610220b5f509687b840532f95f3638ee0146b/opengl/tools/glgen/stubs/egl/eglCreateWindowSurface.java#42

Sending a SurfaceHolder as input instead should hopefully be supported everywhere, and this is also what GlSurfaceView does:
http://grepcode.com/file/repository.grepcode.com/java/ext/com.google.android/android/5.1.1_r1/android/opengl/GLSurfaceView.java#1076

Committed: https://crrev.com/c3402fc3ef0dfebdc82f71ca0bc9a8b549539a2b
Cr-Commit-Position: refs/heads/master@{#10392}

[magjed_webrtc, 2015-10-23 16:47:14]

Description was changed from

==========
EGL10.eglCreateWindowSurface(): Replace Surface input with SurfaceHolder

Sending a Surface as input to EGL10.eglCreateWindowSurface() is not supported
everywhere. See this code as reference:
https://android.googlesource.com/platform/frameworks/native.git/+/ae9610220b5...

Sending a SurfaceHolder as input instead should hopefully be supported
everywhere, and this is also what GlSurfaceView does:
http://grepcode.com/file/repository.grepcode.com/java/ext/com.google.android/...
==========

to

==========
EGL10.eglCreateWindowSurface(): Replace Surface input with SurfaceHolder

Sending a Surface as input to EGL10.eglCreateWindowSurface() is not supported
everywhere. See this code as reference:
https://android.googlesource.com/platform/frameworks/native.git/+/ae9610220b5...

Sending a SurfaceHolder as input instead should hopefully be supported
everywhere, and this is also what GlSurfaceView does:
http://grepcode.com/file/repository.grepcode.com/java/ext/com.google.android/...
==========

[magjed_webrtc, 2015-10-23 16:47:14]

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

[magjed_webrtc, 2015-10-23 22:45:38]

perkj - Please take a look.

[perkj_webrtc, 2015-10-24 00:28:57]

On 2015/10/23 22:45:38, magjed_webrtc wrote:
> perkj - Please take a look.

lgtm

[perkj_webrtc, 2015-10-24 00:29:07]

The CQ bit was checked by perkj@webrtc.org

[commit-bot: I haz the power, 2015-10-24 00:29:10]

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

[commit-bot: I haz the power, 2015-10-24 01:13:18]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-10-24 01:13:29]

Patchset 1 (id:??) landed as
https://crrev.com/c3402fc3ef0dfebdc82f71ca0bc9a8b549539a2b
Cr-Commit-Position: refs/heads/master@{#10392}