Refactor idlsync.py to update chrome apps idl/tools.

Refactor idlsync.py to update chrome apps idl/tools.

Note, by default, idlsync.py will only update webkit IDL.  Use:
idlsync.py -u chrome
to update chrome only or:
idlsync.py -u all
to update both.

Sasha: can you verify that the WHITELIST (*.py, *.idl, *.json) matches the files you need?  Everything else will be pruned.  Also, note, I've slightly changed the directories to not conflict with the existing ones pulled in by DEPS.  I will remove those existing ones in a separate CL (the DEPS is a slightly different repo path).

Emily/Anton: can you review the rest?  If this looks good, the intent is to run and check-in the files under third_party/chrome.

BUG=http://code.google.com/p/dart/issues/detail?id=8173

[Anton Muhin, 2013-02-13 07:49:36]

lgtm with comments addressed

https://chromiumcodereview.appspot.com/12225151/diff/1/tools/dom/scripts/idls...
File tools/dom/scripts/idlsync.py (right):

https://chromiumcodereview.appspot.com/12225151/diff/1/tools/dom/scripts/idls...
tools/dom/scripts/idlsync.py:18: DEPS =
'http://dart.googlecode.com/svn/branches/bleeding_edge/deps/dartium.deps/DEPS'
nit: maybe split into a couple of lines?

https://chromiumcodereview.appspot.com/12225151/diff/1/tools/dom/scripts/idls...
tools/dom/scripts/idlsync.py:74: update_list = [
make it all upper case (and deps_pattern too) to distinguish from local
variables?

https://chromiumcodereview.appspot.com/12225151/diff/1/tools/dom/scripts/idls...
tools/dom/scripts/idlsync.py:114: """Refreshes the IDL to specific WebKit url /
revision."""
please, adjust the comment

https://chromiumcodereview.appspot.com/12225151/diff/1/tools/dom/scripts/idls...
tools/dom/scripts/idlsync.py:178: if not revision:
if revision is None: ?

[Emily Fortuna, 2013-02-13 18:33:42]

lgtm with anton's comments

https://chromiumcodereview.appspot.com/12225151/diff/1/tools/dom/scripts/idls...
File tools/dom/scripts/idlsync.py (right):

https://chromiumcodereview.appspot.com/12225151/diff/1/tools/dom/scripts/idls...
tools/dom/scripts/idlsync.py:161: help='IDL to update (webkit | chrome | all)',
default='webkit')
long line

https://chromiumcodereview.appspot.com/12225151/diff/1/tools/dom/scripts/idls...
tools/dom/scripts/idlsync.py:178: if not revision:
On 2013/02/13 07:49:36, Anton Muhin wrote:
> if revision is None: ?

I think "if not revision" is the more standard python way to write it

[vsm, 2013-02-13 18:53:27]

Thanks!  Addressed comments and landing.

https://codereview.chromium.org/12225151/diff/1/tools/dom/scripts/idlsync.py
File tools/dom/scripts/idlsync.py (right):

https://codereview.chromium.org/12225151/diff/1/tools/dom/scripts/idlsync.py#...
tools/dom/scripts/idlsync.py:18: DEPS =
'http://dart.googlecode.com/svn/branches/bleeding_edge/deps/dartium.deps/DEPS'
On 2013/02/13 07:49:36, Anton Muhin wrote:
> nit: maybe split into a couple of lines?

Done.

https://codereview.chromium.org/12225151/diff/1/tools/dom/scripts/idlsync.py#...
tools/dom/scripts/idlsync.py:74: update_list = [
On 2013/02/13 07:49:36, Anton Muhin wrote:
> make it all upper case (and deps_pattern too) to distinguish from local
> variables?

Done.

https://codereview.chromium.org/12225151/diff/1/tools/dom/scripts/idlsync.py#...
tools/dom/scripts/idlsync.py:114: """Refreshes the IDL to specific WebKit url /
revision."""
On 2013/02/13 07:49:36, Anton Muhin wrote:
> please, adjust the comment

Done.

https://codereview.chromium.org/12225151/diff/1/tools/dom/scripts/idlsync.py#...
tools/dom/scripts/idlsync.py:161: help='IDL to update (webkit | chrome | all)',
default='webkit')
On 2013/02/13 18:33:42, Emily Fortuna wrote:
> long line

Done.

https://codereview.chromium.org/12225151/diff/1/tools/dom/scripts/idlsync.py#...
tools/dom/scripts/idlsync.py:178: if not revision:
I'll keep this as is.

On 2013/02/13 18:33:42, Emily Fortuna wrote:
> On 2013/02/13 07:49:36, Anton Muhin wrote:
> > if revision is None: ?
> 
> I think "if not revision" is the more standard python way to write it

[Anton Muhin, 2013-02-13 18:59:58]

https://codereview.chromium.org/12225151/diff/1/tools/dom/scripts/idlsync.py
File tools/dom/scripts/idlsync.py (right):

https://codereview.chromium.org/12225151/diff/1/tools/dom/scripts/idlsync.py#...
tools/dom/scripts/idlsync.py:178: if not revision:
There is an important difference: not revision is true if revision is 0.  Not
that I expect it, but given default to None, is None looks more direct.

On 2013/02/13 18:53:27, vsm wrote:
> I'll keep this as is.
> 
> On 2013/02/13 18:33:42, Emily Fortuna wrote:
> > On 2013/02/13 07:49:36, Anton Muhin wrote:
> > > if revision is None: ?
> > 
> > I think "if not revision" is the more standard python way to write it
>

[sashab, 2013-02-14 00:39:59]

lgtm, whitelist seems fine :)

Later, you might want to include api.gyp as well, as that contains a nice list
of all the IDL files to include. But that's later down the track.