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

Unified Diff: chrome/browser/resources/md_downloads/vulcanize.py

Issue 1846383002: MD Downloads: fix vulcanize issues by excluding higher up in the dependency tree (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix tests Created 4 years, 8 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: chrome/browser/resources/md_downloads/vulcanize.py
diff --git a/chrome/browser/resources/md_downloads/vulcanize.py b/chrome/browser/resources/md_downloads/vulcanize.py
index 3ef22dc316f58735a30feaf0ea2290cc48f8de57..6ef0f5938f9c7a1f3b8eba7e9f4a8089af2193bc 100755
--- a/chrome/browser/resources/md_downloads/vulcanize.py
+++ b/chrome/browser/resources/md_downloads/vulcanize.py
@@ -34,9 +34,13 @@ _VULCANIZE_ARGS = [
'--exclude', 'polymer-extracted.js',
'--exclude', 'polymer-micro-extracted.js',
'--exclude', 'polymer-mini-extracted.js',
- '--exclude', 'polymer_config.js',
'--exclude', 'web-animations-next-lite.min.js',
+ # Because we're excluding polymer-* files, they are run earlier as separate
+ # <script>s. polymer_config.js must run before any polymer-*.js, so it must be
+ # excluded from vulcanization as well to have any effect.
+ '--exclude', 'polymer_config.js',
+
# These files are dynamically created by C++.
'--exclude', 'load_time_data.js',
'--exclude', 'strings.js',

Powered by Google App Engine
This is Rietveld 408576698