OLD | NEW |
1 # Copyright 2016 The Chromium Authors. All rights reserved. | 1 # Copyright 2016 The Chromium Authors. All rights reserved. |
2 # Use of this source code is governed by a BSD-style license that can be | 2 # Use of this source code is governed by a BSD-style license that can be |
3 # found in the LICENSE file. | 3 # found in the LICENSE file. |
4 | 4 |
5 """Identifies Polymer elements that downloaded but not used by Chrome. | 5 """Identifies Polymer elements that downloaded but not used by Chrome. |
6 | 6 |
7 Only finds "first-order" unused elements; re-run after removing unused elements | 7 Only finds "first-order" unused elements; re-run after removing unused elements |
8 to check if other elements have become unused. | 8 to check if other elements have become unused. |
9 """ | 9 """ |
10 | 10 |
11 import os | 11 import os |
12 import re | 12 import re |
13 import subprocess | 13 import subprocess |
| 14 import sys |
14 | 15 |
| 16 _HERE_PATH = os.path.dirname(__file__) |
| 17 _SRC_PATH = os.path.normpath(os.path.join(_HERE_PATH, '..', '..', '..')) |
| 18 sys.path.append(os.path.join(_SRC_PATH, 'third_party', 'node')) |
| 19 import node |
| 20 import node_modules |
15 | 21 |
16 class UnusedElementsDetector(object): | 22 class UnusedElementsDetector(object): |
17 """Finds unused Polymer elements.""" | 23 """Finds unused Polymer elements.""" |
18 | 24 |
19 # Unused elements to ignore because we plan to use them soon. | 25 # Unused elements to ignore because we plan to use them soon. |
20 __WHITELIST = ( | 26 __WHITELIST = ( |
21 # Necessary for closure. | 27 # Necessary for closure. |
22 'polymer-externs', | 28 'polymer-externs', |
23 ) | 29 ) |
24 | 30 |
(...skipping 26 matching lines...) Expand all Loading... |
51 | 57 |
52 Returns: | 58 Returns: |
53 A string consisting of the minified file contents with comments and grit | 59 A string consisting of the minified file contents with comments and grit |
54 directives removed. | 60 directives removed. |
55 """ | 61 """ |
56 with open(filename) as f: | 62 with open(filename) as f: |
57 text = f.read() | 63 text = f.read() |
58 text = re.sub('<if .*?>', '', text, flags=re.IGNORECASE) | 64 text = re.sub('<if .*?>', '', text, flags=re.IGNORECASE) |
59 text = re.sub('</if>', '', text, flags=re.IGNORECASE) | 65 text = re.sub('</if>', '', text, flags=re.IGNORECASE) |
60 | 66 |
61 proc = subprocess.Popen(['uglifyjs', filename], stdout=subprocess.PIPE) | 67 return node.RunNode([node_modules.PathToUglify(), filename]) |
62 return proc.stdout.read() | |
63 | 68 |
64 @staticmethod | 69 @staticmethod |
65 def __StripComments(filename): | 70 def __StripComments(filename): |
66 """Returns the contents of a JavaScript or HTML file with comments removed. | 71 """Returns the contents of a JavaScript or HTML file with comments removed. |
67 | 72 |
68 Args: | 73 Args: |
69 filename: The name of the file to read. | 74 filename: The name of the file to read. |
70 | 75 |
71 Returns: | 76 Returns: |
72 A string consisting of the file contents processed via | 77 A string consisting of the file contents processed via |
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
126 for (dirpath, _, filenames) in os.walk(path): | 131 for (dirpath, _, filenames) in os.walk(path): |
127 # Ignore the element's own files. | 132 # Ignore the element's own files. |
128 if dirpath.startswith(os.path.join( | 133 if dirpath.startswith(os.path.join( |
129 self.__COMPONENTS_DIR, element_dir)): | 134 self.__COMPONENTS_DIR, element_dir)): |
130 continue | 135 continue |
131 | 136 |
132 for filename in filenames: | 137 for filename in filenames: |
133 if not filename.endswith('.html') and not filename.endswith('.js'): | 138 if not filename.endswith('.html') and not filename.endswith('.js'): |
134 continue | 139 continue |
135 | 140 |
136 # Skip generated files that may include the element source. | |
137 if filename in ('crisper.js', 'vulcanized.html', | |
138 'app.crisper.js', 'app.vulcanized.html'): | |
139 continue | |
140 | |
141 with open(os.path.join(dirpath, filename)) as f: | 141 with open(os.path.join(dirpath, filename)) as f: |
142 text = f.read() | 142 text = f.read() |
143 if not re.search('/%s/' % element_dir, text): | 143 if not re.search('/%s/' % element_dir, text): |
144 continue | 144 continue |
145 | 145 |
146 # Check the file again, ignoring comments (e.g. example imports and | 146 # Check the file again, ignoring comments (e.g. example imports and |
147 # scripts). | 147 # scripts). |
148 if re.search('/%s' % element_dir, | 148 if re.search('/%s' % element_dir, |
149 self.__StripComments( | 149 self.__StripComments( |
150 os.path.join(dirpath, filename))): | 150 os.path.join(dirpath, filename))): |
151 return True | 151 return True |
152 return False | 152 return False |
153 | 153 |
154 | 154 |
155 if __name__ == '__main__': | 155 if __name__ == '__main__': |
156 UnusedElementsDetector().Run() | 156 UnusedElementsDetector().Run() |
OLD | NEW |