Move shared EGL context initialization to a separate function.

- Make shared EGL context used for HW video decoding member
of decoder factory class.
- Add new Peer connection factory method to initialize shared
EGL context.

This provides an option to use single peer connection factory
in the application and create peer connections from the same
factory and reinitialize shared EGL context for video
decoding HW acceleration.

R=wzh@webrtc.org

Committed: https://chromium.googlesource.com/external/webrtc/+/4d2f4d1c6997b4218774cac10527e1cdb88f20d9

[AlexG, 2015-09-01 20:19:24]

glaznev@webrtc.org changed reviewers:
+ wzh@webrtc.org

[AlexG, 2015-09-01 20:19:24]

PTAL

[wzh, 2015-09-01 20:40:45]

lgtm

https://codereview.webrtc.org/1304063011/diff/80001/talk/app/webrtc/java/jni/...
File talk/app/webrtc/java/jni/androidmediadecoder_jni.cc (right):

https://codereview.webrtc.org/1304063011/diff/80001/talk/app/webrtc/java/jni/...
talk/app/webrtc/java/jni/androidmediadecoder_jni.cc:238: use_surface_ =
(render_egl_context_ == NULL) ? false : true;
nit: use_surface = (render_egl_context_ != NULL); is more readable.

[AlexG, 2015-09-01 20:58:46]

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

[commit-bot: I haz the power, 2015-09-01 20:58:57]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1304063011/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1304063011/80001

[commit-bot: I haz the power, 2015-09-01 21:01:51]

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

[commit-bot: I haz the power, 2015-09-01 21:01:52]

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

[AlexG, 2015-09-01 21:10:36]

https://codereview.webrtc.org/1304063011/diff/80001/talk/app/webrtc/java/jni/...
File talk/app/webrtc/java/jni/androidmediadecoder_jni.cc (right):

https://codereview.webrtc.org/1304063011/diff/80001/talk/app/webrtc/java/jni/...
talk/app/webrtc/java/jni/androidmediadecoder_jni.cc:238: use_surface_ =
(render_egl_context_ == NULL) ? false : true;
On 2015/09/01 20:40:45, wzh wrote:
> nit: use_surface = (render_egl_context_ != NULL); is more readable.

Done.

[AlexG, 2015-09-01 21:11:04]

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

[commit-bot: I haz the power, 2015-09-01 21:11:19]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1304063011/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1304063011/100001

[commit-bot: I haz the power, 2015-09-01 22:02:51]

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

[commit-bot: I haz the power, 2015-09-01 22:02:52]

Dry run: This issue passed the CQ dry run.

[AlexG, 2015-09-01 22:03:24]

The CQ bit was checked by glaznev@webrtc.org

[AlexG, 2015-09-01 22:03:24]

The patchset sent to the CQ was uploaded after l-g-t-m from wzh@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1304063011/#ps100001 (title:
"Address comment + fix Linux build.")

[commit-bot: I haz the power, 2015-09-01 22:03:41]

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

[commit-bot: I haz the power, 2015-09-01 22:03:43]

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

[commit-bot: I haz the power, 2015-09-01 22:03:43]

No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[AlexG, 2015-09-01 22:04:26]

Committed patchset #6 (id:100001) manually as
4d2f4d1c6997b4218774cac10527e1cdb88f20d9 (presubmit successful).