Use Blink IDL parser for dart libraries.

Begin hookup of Blink IDL parser for dart:* libraries.

BUG=
R=vsm@google.com

Committed: https://code.google.com/p/dart/source/detail?r=39948

Committed: https://code.google.com/p/dart/source/detail?r=40026

[terry, 2014-09-03 19:16:50]

terry@google.com changed reviewers:
+ vsm@google.com

[vsm, 2014-09-03 22:23:41]

https://codereview.chromium.org/444743002/diff/200001/tools/dom/idl/dart/new_...
File tools/dom/idl/dart/new_dart.idl (right):

https://codereview.chromium.org/444743002/diff/200001/tools/dom/idl/dart/new_...
tools/dom/idl/dart/new_dart.idl:1: // This file introduces / supplements and
forces Dart declarations.
Are you deleting the old dart.idl?  Might just replace that.

https://codereview.chromium.org/444743002/diff/200001/tools/dom/idl/dart/new_...
tools/dom/idl/dart/new_dart.idl:421: // Operation no longer exists no need to
suppress.
Delete?

https://codereview.chromium.org/444743002/diff/200001/tools/dom/new_scripts/c...
File tools/dom/new_scripts/code_generator_dart.py (right):

https://codereview.chromium.org/444743002/diff/200001/tools/dom/new_scripts/c...
tools/dom/new_scripts/code_generator_dart.py:1: # Copyright (C) 2013 Google Inc.
All rights reserved.
Do you need these files in new_scripts?  They look like copies of what we're
pulling into third_party/WebCore/bindings.

https://codereview.chromium.org/444743002/diff/200001/tools/dom/scripts/dartd...
File tools/dom/scripts/dartdomgenerator.py (right):

https://codereview.chromium.org/444743002/diff/200001/tools/dom/scripts/dartd...
tools/dom/scripts/dartdomgenerator.py:11: # Setup all paths to find our PYTHON
code
You'll need to update these paths.

https://codereview.chromium.org/444743002/diff/200001/tools/dom/scripts/fremo...
File tools/dom/scripts/fremontcutbuilder.py (right):

https://codereview.chromium.org/444743002/diff/200001/tools/dom/scripts/fremo...
tools/dom/scripts/fremontcutbuilder.py:130: webcore_dir =
os.path.join(current_dir, '..', '..', '..', 'third_party',
You'll need to update this path.

https://codereview.chromium.org/444743002/diff/200001/tools/dom/scripts/fremo...
tools/dom/scripts/fremontcutbuilder.py:152: FILES_TO_IGNORE = [
We've got some logic for this in htmlrenamer.py - e.g., search for
WebkitMediaSource.  We drop that in favor of MediaSource.

Perhaps this could be added there?

https://codereview.chromium.org/444743002/diff/200001/tools/dom/scripts/go.sh
File tools/dom/scripts/go.sh (right):

https://codereview.chromium.org/444743002/diff/200001/tools/dom/scripts/go.sh...
tools/dom/scripts/go.sh:59: #  ./dartdomgenerator.py --rebuild --parallel
--systems="$SYSTEMS" \
delete?

[terry, 2014-09-05 20:22:56]

PTAL

https://codereview.chromium.org/444743002/diff/200001/tools/dom/idl/dart/new_...
File tools/dom/idl/dart/new_dart.idl (right):

https://codereview.chromium.org/444743002/diff/200001/tools/dom/idl/dart/new_...
tools/dom/idl/dart/new_dart.idl:1: // This file introduces / supplements and
forces Dart declarations.
No new_dart.idl will replace dart.idl when I checkin (kept old for testing).
On 2014/09/03 22:23:41, vsm wrote:
> Are you deleting the old dart.idl?  Might just replace that.

https://codereview.chromium.org/444743002/diff/200001/tools/dom/idl/dart/new_...
tools/dom/idl/dart/new_dart.idl:421: // Operation no longer exists no need to
suppress.
On 2014/09/03 22:23:41, vsm wrote:
> Delete?

Done.

https://codereview.chromium.org/444743002/diff/200001/tools/dom/new_scripts/c...
File tools/dom/new_scripts/code_generator_dart.py (right):

https://codereview.chromium.org/444743002/diff/200001/tools/dom/new_scripts/c...
tools/dom/new_scripts/code_generator_dart.py:1: # Copyright (C) 2013 Google Inc.
All rights reserved.
They are a littler different because of pickling and Jinja2 support not
happening - yet.  I'm hoping that the dart generator will be able to be replaced
with a general purpose generator using Jinja templates.

Terry.

On 2014/09/03 22:23:41, vsm wrote:
> Do you need these files in new_scripts?  They look like copies of what we're
> pulling into third_party/WebCore/bindings.

https://codereview.chromium.org/444743002/diff/200001/tools/dom/scripts/dartd...
File tools/dom/scripts/dartdomgenerator.py (right):

https://codereview.chromium.org/444743002/diff/200001/tools/dom/scripts/dartd...
tools/dom/scripts/dartdomgenerator.py:11: # Setup all paths to find our PYTHON
code
Right, once I sync'd to you CLs.

On 2014/09/03 22:23:41, vsm wrote:
> You'll need to update these paths.

https://codereview.chromium.org/444743002/diff/200001/tools/dom/scripts/fremo...
File tools/dom/scripts/fremontcutbuilder.py (right):

https://codereview.chromium.org/444743002/diff/200001/tools/dom/scripts/fremo...
tools/dom/scripts/fremontcutbuilder.py:130: webcore_dir =
os.path.join(current_dir, '..', '..', '..', 'third_party',
On 2014/09/03 22:23:41, vsm wrote:
> You'll need to update this path.

The below old webcore_dir will be used.

https://codereview.chromium.org/444743002/diff/200001/tools/dom/scripts/fremo...
tools/dom/scripts/fremontcutbuilder.py:152: FILES_TO_IGNORE = [
Quickly tried it and it didn't work.  I'll look at finding out why post this
checkin and try to integrate with htmlrenamer (added TODO).
On 2014/09/03 22:23:41, vsm wrote:
> We've got some logic for this in htmlrenamer.py - e.g., search for
> WebkitMediaSource.  We drop that in favor of MediaSource.
> 
> Perhaps this could be added there?

https://codereview.chromium.org/444743002/diff/200001/tools/dom/scripts/go.sh
File tools/dom/scripts/go.sh (right):

https://codereview.chromium.org/444743002/diff/200001/tools/dom/scripts/go.sh...
tools/dom/scripts/go.sh:59: #  ./dartdomgenerator.py --rebuild --parallel
--systems="$SYSTEMS" \
On 2014/09/03 22:23:41, vsm wrote:
> delete?

Done.

[vsm, 2014-09-05 20:33:26]

lgtm

https://codereview.chromium.org/444743002/diff/200001/tools/dom/new_scripts/c...
File tools/dom/new_scripts/code_generator_dart.py (right):

https://codereview.chromium.org/444743002/diff/200001/tools/dom/new_scripts/c...
tools/dom/new_scripts/code_generator_dart.py:1: # Copyright (C) 2013 Google Inc.
All rights reserved.
Do you think we'll just be able to use the upstream code once pickling and jinja
are supported?

On 2014/09/05 20:22:55, terry wrote:
> They are a littler different because of pickling and Jinja2 support not
> happening - yet.  I'm hoping that the dart generator will be able to be
replaced
> with a general purpose generator using Jinja templates.
> 
> Terry.
> 
> On 2014/09/03 22:23:41, vsm wrote:
> > Do you need these files in new_scripts?  They look like copies of what we're
> > pulling into third_party/WebCore/bindings.
>

[terry, 2014-09-05 20:35:56]

https://codereview.chromium.org/444743002/diff/200001/tools/dom/new_scripts/c...
File tools/dom/new_scripts/code_generator_dart.py (right):

https://codereview.chromium.org/444743002/diff/200001/tools/dom/new_scripts/c...
tools/dom/new_scripts/code_generator_dart.py:1: # Copyright (C) 2013 Google Inc.
All rights reserved.
Yes, I don't think there's anything that will prevent that.

Terry.

On 2014/09/05 20:33:26, vsm wrote:
> Do you think we'll just be able to use the upstream code once pickling and
jinja
> are supported?
> 
> On 2014/09/05 20:22:55, terry wrote:
> > They are a littler different because of pickling and Jinja2 support not
> > happening - yet.  I'm hoping that the dart generator will be able to be
> replaced
> > with a general purpose generator using Jinja templates.
> > 
> > Terry.
> > 
> > On 2014/09/03 22:23:41, vsm wrote:
> > > Do you need these files in new_scripts?  They look like copies of what
we're
> > > pulling into third_party/WebCore/bindings.
> > 
>

[terry, 2014-09-07 18:40:21]

Changed dart.idl to support __propertyQuery__ exposing what's needed in
CSSStyleDeclaration using existing web idl mechanism for handling
'__operation_name__'.

Blink's IDL parser doesn't allow any operation with a double underscore (e.g.,
__getter__, __propertyQuery__, etc) to be specified as an operation without
explicit special op handling (getter, deleter, setter, etc.).

[terry, 2014-09-08 02:34:10]

Committed patchset #15 (id:280001) manually as 39948 (presubmit successful).

[terry, 2014-09-09 13:18:39]

Committed patchset #17 (id:320001) manually as 40026 (presubmit successful).