Pepper: Don't emit structs with no wrapping.

Pepper: Don't emit structs with no wrapping.

The behavior in the PNaCl shim layer is the same if there's a struct for an
interface with no wrapper or no struct at all.

This change removes structs where no wrapping is necessary.

BUG=251460
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=207839

[jvoung (off chromium), 2013-06-20 19:46:57]

https://codereview.chromium.org/17153012/diff/2001/ppapi/generators/idl_gen_w...
File ppapi/generators/idl_gen_wrapper.py (right):

https://codereview.chromium.org/17153012/diff/2001/ppapi/generators/idl_gen_w...
ppapi/generators/idl_gen_wrapper.py:160: /* We don't have an IDL for this, for
some reason. Take our chances. */
Can you update the emitted comment?

This is now not about taking chances, but is necessary.
Something like "We did not generate a wrapper for this, so return the real
interface."

https://codereview.chromium.org/17153012/diff/2001/ppapi/generators/idl_gen_w...
ppapi/generators/idl_gen_wrapper.py:181: /* We don't have an IDL for this, for
some reason. Take our chances. */
same update for comment.

[teravest, 2013-06-20 19:51:09]

On Thu, Jun 20, 2013 at 1:46 PM,  <jvoung@chromium.org> wrote:
>
>
https://codereview.chromium.org/17153012/diff/2001/ppapi/generators/idl_gen_w...
> File ppapi/generators/idl_gen_wrapper.py (right):
>
>
https://codereview.chromium.org/17153012/diff/2001/ppapi/generators/idl_gen_w...
> ppapi/generators/idl_gen_wrapper.py:160: /* We don't have an IDL for
> this, for some reason. Take our chances. */
> Can you update the emitted comment?
>
> This is now not about taking chances, but is necessary.
> Something like "We did not generate a wrapper for this, so return the
> real interface."

Good call. Done.

>
>
https://codereview.chromium.org/17153012/diff/2001/ppapi/generators/idl_gen_w...
> ppapi/generators/idl_gen_wrapper.py:181: /* We don't have an IDL for
> this, for some reason. Take our chances. */
> same update for comment.

Done.

>
> https://codereview.chromium.org/17153012/

[jvoung (off chromium), 2013-06-20 19:53:49]

LGTM thanks!

[dmichael (off chromium), 2013-06-20 19:55:45]

rubber stamp lgtm

[commit-bot: I haz the power, 2013-06-20 19:56:44]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/teravest@chromium.org/17153012/7001

[commit-bot: I haz the power, 2013-06-21 09:00:41]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[commit-bot: I haz the power, 2013-06-21 14:07:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/teravest@chromium.org/17153012/13003

[commit-bot: I haz the power, 2013-06-21 15:58:52]

Change committed as 207839