Collect IDL files and exract interface node object's information

Source/bindings/scripts/interface_node_path.py

+#!/usr/bin/env python
+
+"""
+The goal of this script is to integrate IDL file path under the directory to text file.

[bashi, 2015/09/02 05:17:37]

Please describe a typical usage here. Developers typically want to know the
usage without looking the whole code.

Another option would be to use optparse.OptionParser/argparse.ArgumentParser.

+"""
+import os
+import sys
+
+
+def get_idl_files(path):
+    """

[bashi, 2015/09/02 05:17:37]

Please add one line summary as shiino-san suggested.

For example,

"""Returns a generator which yields absolute paths of idl files.
...
"""

+    Args:
+      path: directory path
+    Return:
+      str, absolute IDL file path

[bashi, 2015/09/02 05:17:37]

This function returns a generator, not a str.

+    """
+    file_type = '.idl'
+    non_idl_set = (
+        'InspectorInstrumentation.idl',
+    )

[bashi, 2015/09/02 05:17:37]

Let move |file_type| and |non_idl_set| out from this function. We don't need to
create them every time get_idl_files() is called.

We usually use CAPITAL_LETTERS for constants, and if they are 'private' (they
aren't intended to be used outside of this script), we add '_' prefix. So, how
about:

_IDL_SUFFIX = '.idl'
_NON_IDL_FILES = frozenset([
    'InspectorInstrumentation.idl',
])

?

+    for dir_path, dir_names, file_names in os.walk(path):
+        for file_name in file_names:
+            if file_name.endswith(file_type) and file_name not in non_idl_set:
+                yield os.path.join(dir_path, file_name)
+
+
+def main(args):
+    path = args[0]
+    filename = args[1]
+    f = open(filename, 'w')

[bashi, 2015/09/02 05:17:37]

How about:

  with open(filename, 'w') as f:
    ...

This way you don't need to close |f|.

+    for filepath in get_idl_files(path):
+        f.write(filepath + '\n')
+    f.close()
+
+
+if __name__ == '__main__':
+    main(sys.argv[1:])