Create run_javascript_tests.py

testing/tools/fixup_pdf_template.py

 #!/usr/bin/env python
 # Copyright 2014 The PDFium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Expands a hand-written PDF testcase (template) into a valid PDF file.
 
 There are several places in a PDF file where byte-offsets are required. This
 script replaces {{name}}-style variables in the input with calculated results
 
@@ skipping 52 matching lines @@
     if self.STARTXREF_TOKEN in line:
       replacement = self.STARTXREF_REPLACEMENT % self.xref_offset
       line = line.replace(self.STARTXREF_TOKEN, replacement)
     match = re.match(self.OBJECT_PATTERN, line)
     if match:
       self.insert_xref_entry(int(match.group(1)), int(match.group(2)))
       line = re.sub(self.OBJECT_PATTERN, self.OBJECT_REPLACEMENT, line)
     self.offset += len(line)
     return line
 
-def expand_file(input_filename):
+def expand_file(input_filename, output_dir):

[Lei Zhang, 2015/02/10 23:09:29]

Two blank lines between top-level definitions:
https://google-styleguide.googlecode.com/svn/trunk/pyguide.html?showone=Blank...

I admit to not always following this when I write Python code too.

[Tom Sepez, 2015/02/10 23:33:31]

Done.

   (input_root, extension) = os.path.splitext(input_filename)

[Lei Zhang, 2015/02/10 23:09:29]

BTW, if you don't need |extension|, you can just put _ in its place. You also
don't need the parenthesis.

[Tom Sepez, 2015/02/10 23:33:31]

Done.

-  output_filename = input_root + '.pdf'
+  output_filename = os.path.join(output_dir, input_root + '.pdf')
+  output_dirname = os.path.dirname(output_filename)
+  if not os.path.exists(output_dirname):
+    os.makedirs(output_dirname)
   processor = TemplateProcessor()
   try:
     with open(input_filename, 'r') as infile:
       with open(output_filename, 'w') as outfile:
         for line in infile:
            outfile.write(processor.process_line(line))
   except IOError:
     print >> sys.stderr, 'failed to process %s' % input_filename
 
 def main():
-  for arg in sys.argv[1:]:
-    expand_file(arg)
+  parser = optparse.OptionParser()
+  parser.add_option('--build-dir', default='')
+  (options, args) = parser.parse_args()
+  working_dir = options.build_dir
+  if working_dir:
+    working_dir = os.path.join(options.build_dir, 'gen', 'pdfium')
+  for arg in args:
+    expand_file(arg, working_dir)
   return 0
 
 if __name__ == '__main__':
   sys.exit(main())