Split generate_global_constructors.py from compute_interfaces_info.py

Split generate_global_constructors.py from compute_interfaces_info.py

Just split generate_global_constructors.py from compute_interfaces_info.py,
with common functions (file handling and regex IDL parsing) in utilities.py

No logic changes, just split.
(Logic changes in followup.)

Rationale (for reference):
Currently compute_dependencies.py is very complex, doing 3 tasks:
1. Computing interfaces info (main function)
2. Computing global constructors
3. Computing event interfaces

This splits out 2 (global constructors); event interfaces will be a followup.

The complexity makes the code hard to understand and edit (previous CLs have
required extensive review), and makes code reuse hard (e.g., in
run-bindings-tests we need to use dummy files b/c we don't care about global
constructors). 

Most seriously, this makes it hard to use compute_dependencies to compute
dependencies at gyp time (needed to avoid full rebuilds), since it generates
excess files that we only want to generate at build time.

Followup to:
Rename compute_dependencies.py => compute_interfaces_info.py
https://codereview.chromium.org/171373006/

Split from:
Split generate_global_constructors.py out of compute_dependencies.py
https://codereview.chromium.org/173803006/

BUG=341748
R=haraken

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

[Nils Barth (inactive), 2014-02-21 13:06:21]

#2/3

[haraken, 2014-02-21 13:28:20]

LGTM

[Nils Barth (inactive), 2014-02-25 01:39:09]

The CQ bit was checked by nbarth@chromium.org

[commit-bot: I haz the power, 2014-02-25 01:41:04]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nbarth@chromium.org/173503009/70001

[Nils Barth (inactive), 2014-02-25 04:02:06]

The CQ bit was unchecked by nbarth@chromium.org

[Nils Barth (inactive), 2014-02-28 02:08:53]

The CQ bit was checked by nbarth@chromium.org

[commit-bot: I haz the power, 2014-02-28 02:09:07]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nbarth@chromium.org/173503009/70001

[commit-bot: I haz the power, 2014-02-28 02:09:08]

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

[commit-bot: I haz the power, 2014-02-28 02:09:09]

Failed to apply patch for Tools/Scripts/webkitpy/bindings/main.py:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Tools/Scripts/webkitpy/bindings/main.py
  Hunk #1 FAILED at 164.
  1 out of 1 hunk FAILED -- saving rejects to file
Tools/Scripts/webkitpy/bindings/main.py.rej

Patch:       Tools/Scripts/webkitpy/bindings/main.py
Index: Tools/Scripts/webkitpy/bindings/main.py
diff --git a/Tools/Scripts/webkitpy/bindings/main.py
b/Tools/Scripts/webkitpy/bindings/main.py
index
7396fa3ee3ee29a4dbcbbd9b304c67fcd3d6d5d6..37cbe1f0f052c958220f9856c4561d10550b00f0
100644
--- a/Tools/Scripts/webkitpy/bindings/main.py
+++ b/Tools/Scripts/webkitpy/bindings/main.py
@@ -164,22 +164,11 @@ class BindingsTests(object):
 
         def compute_interfaces_info(idl_files_list_filename,
                                     event_names_filename):
-            # Dummy files, required by compute_interfaces_info but not checked
-            _, window_constructors_file = provider.newtempfile()
-            _, workerglobalscope_constructors_file = provider.newtempfile()
-            _, sharedworkerglobalscope_constructors_file =
provider.newtempfile()
-            _, dedicatedworkerglobalscope_constructors_file =
provider.newtempfile()
-            _, serviceworkersglobalscope_constructors_file =
provider.newtempfile()
             cmd = ['python',
                    'bindings/scripts/compute_interfaces_info.py',
                    '--idl-files-list', idl_files_list_filename,
                    '--interface-dependencies-file',
self.interface_dependencies_filename,
                    '--interfaces-info-file', self.interfaces_info_filename,
-                   '--window-constructors-file', window_constructors_file,
-                   '--workerglobalscope-constructors-file',
workerglobalscope_constructors_file,
-                   '--sharedworkerglobalscope-constructors-file',
sharedworkerglobalscope_constructors_file,
-                   '--dedicatedworkerglobalscope-constructors-file',
dedicatedworkerglobalscope_constructors_file,
-                   '--serviceworkerglobalscope-constructors-file',
serviceworkersglobalscope_constructors_file,
                    '--event-names-file', event_names_filename,
                    '--write-file-only-if-changed', '0']
             self.run_command(cmd)

[Nils Barth (inactive), 2014-02-28 03:02:36]

BTW, need to add [NoHeader] to partial interfaces b/c
we've switched to Python.
(Recall that partial interfaces with only constructor attributes
don't have a header.)
I'll clean this up (generate stub header, remove NoHeader)
in a followup, which will simplify the scripts.

Also, I realized that this would be clearer as a simple Jinja template,
instead of building the file by hand.
I've added a FIXME and will do in a followup.

[Nils Barth (inactive), 2014-02-28 03:03:09]

The CQ bit was checked by nbarth@chromium.org

[commit-bot: I haz the power, 2014-02-28 03:03:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nbarth@chromium.org/173503009/90001

[commit-bot: I haz the power, 2014-02-28 04:49:00]

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

[commit-bot: I haz the power, 2014-02-28 04:49:01]

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

[Nils Barth (inactive), 2014-02-28 05:27:47]

The CQ bit was checked by nbarth@chromium.org

[commit-bot: I haz the power, 2014-02-28 05:28:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nbarth@chromium.org/173503009/90001

[commit-bot: I haz the power, 2014-02-28 08:21:01]

Change committed as 168094