IDL: Defer typedef resolution

IDL: Defer typedef resolution

This is a preparation for typedefs visibility change.
Since some typedefs are used by more than one interface
(e.g. BufferSource), they should be visible from all
interfaces when they satisfy the dependencies.

Currently, typedef resolution takes place when an IDL
file is parsed (in idl_definitions.py). However, not all
typedefs are available at that point. This CL defers
the typedef resolution at the point the code generator
generates bindings code. All typedefs are stored in
component_info so that the code generator can refer them.

This CL doesn't change the generated code.

BUG=425916
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=185769

[bashi, 2014-11-19 11:06:35]

bashi@chromium.org changed reviewers:
+ haraken@chromium.org, tasak@google.com

[bashi, 2014-11-19 11:06:35]

PTAL?

tasak-san, I'd like you to take a look at GYP/GN changes. I want to make sure
that this CL doesn't break partial interface handling.

https://codereview.chromium.org/735983002/diff/1/Source/bindings/core/v8/BUIL...
File Source/bindings/core/v8/BUILD.gn (left):

https://codereview.chromium.org/735983002/diff/1/Source/bindings/core/v8/BUIL...
Source/bindings/core/v8/BUILD.gn:45: component_wide_info =
Dropped "wide" for consistency.

https://codereview.chromium.org/735983002/diff/1/Source/modules/crypto/Subtle...
File Source/modules/crypto/SubtleCrypto.idl (right):

https://codereview.chromium.org/735983002/diff/1/Source/modules/crypto/Subtle...
Source/modules/crypto/SubtleCrypto.idl:37: //typedef (ArrayBuffer or
ArrayBufferView) CryptoOperationData;
Commented out so that we don't generate unused ArrayBufferOrArrayBufferView for
now.

[bashi, 2014-11-20 23:15:30]

tasak-san, ping?

[tasak, 2014-11-21 02:09:01]

lgtm

[haraken, 2014-11-21 02:10:59]

LGTM

https://codereview.chromium.org/735983002/diff/1/Source/modules/crypto/Subtle...
File Source/modules/crypto/SubtleCrypto.idl (right):

https://codereview.chromium.org/735983002/diff/1/Source/modules/crypto/Subtle...
Source/modules/crypto/SubtleCrypto.idl:37: //typedef (ArrayBuffer or
ArrayBufferView) CryptoOperationData;
On 2014/11/19 11:06:35, bashi1 wrote:
> Commented out so that we don't generate unused ArrayBufferOrArrayBufferView
for
> now.

Nit: One space after //.

[bashi, 2014-11-21 03:13:44]

Thank you for reviews!

https://codereview.chromium.org/735983002/diff/1/Source/modules/crypto/Subtle...
File Source/modules/crypto/SubtleCrypto.idl (right):

https://codereview.chromium.org/735983002/diff/1/Source/modules/crypto/Subtle...
Source/modules/crypto/SubtleCrypto.idl:37: //typedef (ArrayBuffer or
ArrayBufferView) CryptoOperationData;
On 2014/11/21 02:10:58, haraken wrote:
> On 2014/11/19 11:06:35, bashi1 wrote:
> > Commented out so that we don't generate unused ArrayBufferOrArrayBufferView
> for
> > now.
> 
> Nit: One space after //.

Done.

[bashi, 2014-11-21 03:13:50]

The CQ bit was checked by bashi@chromium.org

[commit-bot: I haz the power, 2014-11-21 03:14:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/735983002/20001

[commit-bot: I haz the power, 2014-11-21 05:06:42]

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

[commit-bot: I haz the power, 2014-11-21 05:06:43]

Try jobs failed on following builders:
  linux_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/linux_blink_rel/builds/3...)

[bashi, 2014-11-21 05:18:36]

The CQ bit was checked by bashi@chromium.org

[commit-bot: I haz the power, 2014-11-21 05:19:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/735983002/40001

[commit-bot: I haz the power, 2014-11-21 07:24:53]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=185769