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

Unified Diff: Source/devtools/scripts/concatenate_module_descriptors.py

Issue 472903003: DevTools: Get rid of module initializers in the source tree (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Add missing worker module.json's to gypi Created 6 years, 4 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/devtools/scripts/concatenate_module_descriptors.py
diff --git a/Source/devtools/scripts/concatenate_module_descriptors.py b/Source/devtools/scripts/concatenate_module_descriptors.py
index 2745dbda322b2c692b114749f1f529b50979e697..2840af9c7724fa41a6a3465716a1c3e4e7125c97 100755
--- a/Source/devtools/scripts/concatenate_module_descriptors.py
+++ b/Source/devtools/scripts/concatenate_module_descriptors.py
@@ -4,10 +4,12 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
-"""Inlines all module.json files into modules.js."""
+"""Inlines all module.json files into "var allDescriptors" in Runtime.js."""
from os import path
import errno
+import os
+import re
import shutil
import sys
try:
@@ -45,9 +47,11 @@ def main(argv):
output_filename = argv[2]
module_jsons = argv[3:]
+ output_contents = re.sub('var allDescriptors = \[\];', 'var allDescriptors = %s;' % build_modules(module_jsons).replace("\\", "\\\\"), read_file(input_filename), 1)
+ if (path.exists(output_filename)):
+ os.remove(output_filename)
with open(output_filename, 'w') as output_file:
- output_file.write('var allDescriptors=%s;' % build_modules(module_jsons))
-
+ output_file.write(output_contents)
if __name__ == '__main__':
sys.exit(main(sys.argv))

Powered by Google App Engine
This is Rietveld 408576698