Add gpu targets to untrusted build, and add Graphics3D code to untrusted build.

Add gpu targets to untrusted build, and add Graphics3D code to untrusted build.
BUG=116317
TEST=builds,manual

With this change, the IPC-based PPAPI proxy builds all code needed for 3d and OpenGL ES.
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=150432

[bbudge, 2012-08-07 19:25:37]

This is a re-do of http://codereview.chromium.org/10796038/.
I had to revert because it broke the static build.

OpenGLES conformance tests pass.
browser_tests Graphics3D (PPAPI, OutOfProcessPPAPI, and NaCl versions) pass.

[bbudge, 2012-08-07 19:26:31]

On 2012/08/07 19:25:37, bbudge1 wrote:
> This is a re-do of http://codereview.chromium.org/10796038/.
> I had to revert because it broke the static build.
> 
> OpenGLES conformance tests pass.
> browser_tests Graphics3D (PPAPI, OutOfProcessPPAPI, and NaCl versions) pass.

Dependencies were left out for gpu.gyp:gpu target. gpu.gyp is the only file that
has changed.

[brettw, 2012-08-07 20:02:53]

lgtm

http://codereview.chromium.org/10837145/diff/1/ppapi/shared_impl/ppb_image_da...
File ppapi/shared_impl/ppb_image_data_shared.cc (right):

http://codereview.chromium.org/10837145/diff/1/ppapi/shared_impl/ppb_image_da...
ppapi/shared_impl/ppb_image_data_shared.cc:5: #include "build/build_config.h"
This should go below (The .h always comes first with a blank line after it. If
the .h depends on build_config, it should include it itself).

[bbudge, 2012-08-07 20:07:57]

http://codereview.chromium.org/10837145/diff/1/ppapi/shared_impl/ppb_image_da...
File ppapi/shared_impl/ppb_image_data_shared.cc (right):

http://codereview.chromium.org/10837145/diff/1/ppapi/shared_impl/ppb_image_da...
ppapi/shared_impl/ppb_image_data_shared.cc:5: #include "build/build_config.h"
On 2012/08/07 20:02:53, brettw wrote:
> This should go below (The .h always comes first with a blank line after it. If
> the .h depends on build_config, it should include it itself).

Done.

[greggman, 2012-08-07 20:09:06]

lgtm

[nfullagar, 2012-08-07 20:11:34]

Plz double check CWS apps still work with NaCl proxy - Bastion, Mini-Ninjas,
From Dust, etc.  And examples in the NaCl SDK still build & link & run.

[piman, 2012-08-07 20:11:40]

lgtm

[Ami GONE FROM CHROMIUM, 2012-08-28 18:24:38]

post-commit drive-by

https://chromiumcodereview.appspot.com/10837145/diff/3002/ppapi/proxy/resourc...
File ppapi/proxy/resource_creation_proxy.cc (right):

https://chromiumcodereview.appspot.com/10837145/diff/3002/ppapi/proxy/resourc...
ppapi/proxy/resource_creation_proxy.cc:214:
printf("ResourceCreationProxy::CreateGraphics3D\n");
What's with this printf() and the one below (l.223)?  Did you mean to drop them
before checking in?

[bbudge, 2012-08-28 18:30:51]

On 2012/08/28 18:24:38, Ami Fischman wrote:
> post-commit drive-by
> 
>
https://chromiumcodereview.appspot.com/10837145/diff/3002/ppapi/proxy/resourc...
> File ppapi/proxy/resource_creation_proxy.cc (right):
> 
>
https://chromiumcodereview.appspot.com/10837145/diff/3002/ppapi/proxy/resourc...
> ppapi/proxy/resource_creation_proxy.cc:214:
> printf("ResourceCreationProxy::CreateGraphics3D\n");
> What's with this printf() and the one below (l.223)?  Did you mean to drop
them
> before checking in?

Yes indeed. Thanks. Fixing.