Only dlopen() OMX/VAAPI libs in the GPU process, and only lazily dlsym() their symbols.

Only dlopen() OMX/VAAPI libs in the GPU process, and only lazily dlsym() their symbols.

BUG=140412
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=150007

[Ami GONE FROM CHROMIUM, 2012-08-04 00:44:11]

piman: please review
posciak: FYI

[piman, 2012-08-04 01:02:23]

+Jorge just in case.
LGTM for me.

https://chromiumcodereview.appspot.com/10829182/diff/1/content/common/gpu/med...
File content/common/gpu/media/vaapi_video_decode_accelerator.cc (right):

https://chromiumcodereview.appspot.com/10829182/diff/1/content/common/gpu/med...
content/common/gpu/media/vaapi_video_decode_accelerator.cc:70: static bool
vaapi_functions_initialized = PostSandboxInitialization();
nit: Do you want to use pre_sandbox_init_done_ instead? Or remove that one?
(either way works for me)

[Jorge Lucangeli Obes, 2012-08-04 01:10:55]

As long as we dlopen() before calling InitializeSandbox, we're good =).

lgtm

[Ami GONE FROM CHROMIUM, 2012-08-04 01:44:42]

https://chromiumcodereview.appspot.com/10829182/diff/1/content/common/gpu/med...
File content/common/gpu/media/vaapi_video_decode_accelerator.cc (right):

https://chromiumcodereview.appspot.com/10829182/diff/1/content/common/gpu/med...
content/common/gpu/media/vaapi_video_decode_accelerator.cc:70: static bool
vaapi_functions_initialized = PostSandboxInitialization();
On 2012/08/04 01:02:24, piman wrote:
> nit: Do you want to use pre_sandbox_init_done_ instead? Or remove that one?
> (either way works for me)

Removed (was vestigial).

[commit-bot: I haz the power, 2012-08-04 01:45:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/fischman@chromium.org/10829182/5001

[commit-bot: I haz the power, 2012-08-04 03:02:50]

Change committed as 150007