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

Unified Diff: tools/polymer/polymer_grdp_to_txt.py

Issue 984553002: Added helper scripts for modifying polymer_resources.grdp. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Comments addressed. Created 5 years, 9 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: tools/polymer/polymer_grdp_to_txt.py
diff --git a/tools/polymer/polymer_grdp_to_txt.py b/tools/polymer/polymer_grdp_to_txt.py
new file mode 100755
index 0000000000000000000000000000000000000000..f9c8735720cc981b8713e8dd2a7dc3402919362d
--- /dev/null
+++ b/tools/polymer/polymer_grdp_to_txt.py
@@ -0,0 +1,34 @@
+#!/usr/bin/env python
+# Copyright 2015 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+import sys
+import xml.sax
+
+
+class PathsExtractor(xml.sax.ContentHandler):
+
+ def __init__(self):
+ self.paths = []
+
+ def startElement(self, name, attrs):
+ if name != 'structure':
+ return
+ path = attrs['file']
+ if path.startswith('../../../third_party/web-animations-js'):
+ return
+ prefix = '../../../third_party/polymer/components-chromium/'
+ if not path.startswith(prefix):
+ raise Exception("Unexpected path %s." % path)
+ self.paths.append(path[len(prefix):])
+
+
+def main(argv):
+ xml_handler = PathsExtractor()
+ xml.sax.parse(sys.argv[1], xml_handler)
dzhioev (left Google) 2015/03/17 18:26:46 Should be just "argv[1]". Fixed in the next patchs
+ print '\n'.join(sorted(xml_handler.paths))
+
+
+if __name__ == '__main__':
+ sys.exit(main(sys.argv))
Dan Beam 2015/03/17 16:37:53 I don't think this is the way you're supposed to u
dzhioev (left Google) 2015/03/17 18:26:46 I don't see what is wrong. main() can optionally r
Dan Beam 2015/03/17 18:37:53 i guess None is 0-ish, yeah, but there's nothing t
dzhioev (left Google) 2015/03/18 16:38:05 Probably such construction is idiomatic in Python.

Powered by Google App Engine
This is Rietveld 408576698