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

Unified Diff: third_party/closure_compiler/checker_test.py

Issue 435053002: Remove <if> statements before checking JS code by checker.py (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@B_define
Patch Set: inline replace_if function Created 6 years, 5 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: third_party/closure_compiler/checker_test.py
diff --git a/third_party/closure_compiler/checker_test.py b/third_party/closure_compiler/checker_test.py
index efc9d2f996e4e2fd2e8e9248afe5c655836afe20..4a7aec4d0cebb3c15720e3313469cb68d50460dc 100755
--- a/third_party/closure_compiler/checker_test.py
+++ b/third_party/closure_compiler/checker_test.py
@@ -68,5 +68,35 @@ debug(global);
self.assertLineNumber(11, LineNumber("/checked.js", 8))
+class IfStrippingTest(unittest.TestCase):
+ def __init__(self, *args, **kwargs):
+ unittest.TestCase.__init__(self, *args, **kwargs)
+ self.maxDiff = None
+
+ def setUp(self):
+ FileCache._cache["/century.js"] = """
+ function getCurrentCentury() {
+<if expr="netscape_os">
+ alert("Oh wow!");
+ return "XX";
+</if>
+ return "XXI";
+ }
+""".strip()
+
+ self.flattener_ = Flattener("/century.js")
+
+ def testIfStripping(self):
+ self.assertMultiLineEqual("""
+ function getCurrentCentury() {
+
+ alert("Oh wow!");
+ return "XX";
+
+ return "XXI";
+ }
+""".strip(), self.flattener_.contents)
+
+
if __name__ == '__main__':
unittest.main()
« third_party/closure_compiler/checker.py ('K') | « third_party/closure_compiler/checker.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698