Add support for Chrome Apps to IDL lexer/parser

This adds support for a number of features in the current WebIDL spec (http://dev.w3.org/2006/webapi/WebIDL/):
-Callbacks
-Dictionaries
-static functions in interfaces
-optional parameters and dictionary members

It also introduces a "namespace" production to the grammar, which is just a named scope surrounding a list of other IDL fragments. 

Finally, there are a couple of random cleanups.


BUG=116636
TEST=existing tests should still work

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

[noelallen1, 2012-02-28 22:31:32]

http://codereview.chromium.org/9388002/diff/4005/ppapi/generators/idl_lexer.py
File ppapi/generators/idl_lexer.py (right):

http://codereview.chromium.org/9388002/diff/4005/ppapi/generators/idl_lexer.p...
ppapi/generators/idl_lexer.py:135: r'(/\*(.|\n)*?\*/)|(//.*(\n//.*)*)'
Isn't this missing indentation/whitespace between the >>*(\n<< and the >>//.*)*
to consume whitespace on any following lines?

http://codereview.chromium.org/9388002/diff/4005/ppapi/generators/idl_parser.py
File ppapi/generators/idl_parser.py (right):

http://codereview.chromium.org/9388002/diff/4005/ppapi/generators/idl_parser....
ppapi/generators/idl_parser.py:217: | namespace"""
I would encourage us to use the copyright notice consistently, however I agree
with the rest.

It would be nice if we can find a way to keep the ability of associating a
comment and attributes with the entire file.   One possibility is a specialized
Copyright regex in the lexer for that particular case.

http://codereview.chromium.org/9388002/diff/4005/ppapi/generators/idl_parser....
ppapi/generators/idl_parser.py:221: # Hack to pull out the first two comments
from the children of |top_list|
Lets find a better solution for this hack.

[asargent_no_longer_on_chrome, 2012-03-03 01:12:02]

http://codereview.chromium.org/9388002/diff/4005/ppapi/generators/idl_lexer.py
File ppapi/generators/idl_lexer.py (right):

http://codereview.chromium.org/9388002/diff/4005/ppapi/generators/idl_lexer.p...
ppapi/generators/idl_lexer.py:135: r'(/\*(.|\n)*?\*/)|(//.*(\n//.*)*)'
On 2012/02/28 22:31:33, noelallen1 wrote:
> Isn't this missing indentation/whitespace between the >>*(\n<< and the
>>//.*)*
> to consume whitespace on any following lines?

Done.

http://codereview.chromium.org/9388002/diff/4005/ppapi/generators/idl_parser.py
File ppapi/generators/idl_parser.py (right):

http://codereview.chromium.org/9388002/diff/4005/ppapi/generators/idl_parser....
ppapi/generators/idl_parser.py:217: | namespace"""
On 2012/02/28 22:31:33, noelallen1 wrote:
> I would encourage us to use the copyright notice consistently, however I agree
> with the rest.
> 
> It would be nice if we can find a way to keep the ability of associating a
> comment and attributes with the entire file.   One possibility is a
specialized
> Copyright regex in the lexer for that particular case.

I fell back to leaving this as-is for now, and will look into future changes to
allow a file-level comment but still require a copyright.

http://codereview.chromium.org/9388002/diff/4005/ppapi/generators/idl_parser....
ppapi/generators/idl_parser.py:221: # Hack to pull out the first two comments
from the children of |top_list|
On 2012/02/28 22:31:33, noelallen1 wrote:
> Lets find a better solution for this hack.

Removed it per above comment.

[noelallen1, 2012-03-03 01:25:29]

lgtm

[commit-bot: I haz the power, 2012-03-03 03:57:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/asargent@chromium.org/9388002/20001

[commit-bot: I haz the power, 2012-03-03 03:57:40]

Presubmit check for 9388002-20001 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit Warnings **
Missing matching PPAPI definition:

***************
  ppapi/c/dev/pp_cursor_type_dev.h
***************

** Presubmit ERRORS **
Missing LGTM from an OWNER for:
ppapi/apps_tests/test1.idl,ppapi/apps_tests/test3.idl,ppapi/apps_tests/experimental.socket.idl,ppapi/apps_tests/test2.idl

Presubmit checks took 2.0s to calculate.