Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(195)

Unified Diff: Source/bindings/scripts/code_generator_v8.py

Issue 18190004: Add Python flow to bindings generation, move dummy-generating IDL files over (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Revised Created 7 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/bindings/scripts/code_generator_v8.py
diff --git a/Source/bindings/scripts/code_generator_v8.py b/Source/bindings/scripts/code_generator_v8.py
new file mode 100644
index 0000000000000000000000000000000000000000..a9bad2c57ad0f8abbc3b4db471b2a80c4ff3b6f8
--- /dev/null
+++ b/Source/bindings/scripts/code_generator_v8.py
@@ -0,0 +1,70 @@
+# Copyright (C) 2013 Google Inc. All rights reserved.
+#
+# Redistribution and use in source and binary forms, with or without
+# modification, are permitted provided that the following conditions are
+# met:
+#
+# * Redistributions of source code must retain the above copyright
+# notice, this list of conditions and the following disclaimer.
+# * Redistributions in binary form must reproduce the above
+# copyright notice, this list of conditions and the following disclaimer
+# in the documentation and/or other materials provided with the
+# distribution.
+# * Neither the name of Google Inc. nor the names of its
+# contributors may be used to endorse or promote products derived from
+# this software without specific prior written permission.
+#
+# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+
+"""Generate Blink V8 bindings (.h and .cpp files).
+
+Input: An object of class IdlDocument, representing an IDL interface X
+Output: V8X.h and V8X.cpp
+
+FIXME: Currently a stub, as part of landing the parser and code generator
+incrementally. Only implements generation of dummy .cpp and .h files.
+
+The plan is as follows.
haraken 2013/07/09 11:16:48 Shall we put this comment in idl_compiler.py since
Nils Barth (inactive) 2013/07/10 02:38:32 Good point (overview in main file); done!
+We will temporarily have two build flows (see ../derived_sources.gyp):
+[1] Perl: deprecated_generate_bindings.pl, which calls:
+ deprecated_idl_parser.pm => deprecated_code_generator_v8.pm
+[2] Python: idl_compiler.py, which calls:
+ frontend (lexer => parser => build_ir.py) => code_generator_v8.py
+
+We will move IDL files from the Perl build flow [1] to the Python build flow [2]
+incrementally.
+First we will land the build changes and idl_compiler.py (in one CL).
+Next we will land the full frontend (in one CL), which parses all IDL files.
+Finally we will incrementally land the code generator (in several CLs),
+moving IDL files to the Python build flow as the code generator is completed.
+"""
+
+import os.path
+
+
+def generate_dummy_header_and_cpp(target_interface_name, output_directory):
+ header_basename = 'V8%s.h' % target_interface_name
+ cpp_basename = 'V8%s.cpp' % target_interface_name
+ header_fullname = os.path.join(output_directory, header_basename)
+ cpp_fullname = os.path.join(output_directory, cpp_basename)
+ contents = """/*
+ This file is generated just to tell build scripts that {header_basename} and
+ {cpp_basename} are created for {target_interface_name}.idl, and thus
+ prevent the build scripts from trying to generate {header_basename} and
+ {cpp_basename} at every build. This file must not be tried to compile.
+*/
+""".format(**locals())
+ with open(header_fullname, 'w') as header_file:
+ header_file.write(contents)
+ with open(cpp_fullname, 'w') as cpp_file:
+ cpp_file.write(contents)

Powered by Google App Engine
This is Rietveld 408576698