servo: Merge #10892 - Pass --no-patch by default to update_css (from CorcovadoMing:patch-1); r=Wafflespeanut
authorCorcovadoMing <rf37535@gmail.com>
Thu, 28 Apr 2016 16:19:35 -0700
changeset 338657 01dd12c29942287b4a5e2a9cfb61b8b9799b5586
parent 338656 7c3e985976dfda875f212d307f3b7bd4a5aaab60
child 338658 39d0a18a46cb5ef347cae7a2bac84e14dcdd295d
push id31307
push usergszorc@mozilla.com
push dateSat, 04 Feb 2017 00:59:06 +0000
treeherdermozilla-central@94079d43835f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersWafflespeanut
servo: Merge #10892 - Pass --no-patch by default to update_css (from CorcovadoMing:patch-1); r=Wafflespeanut fix #10884 according to #9666 Source-Repo: https://github.com/servo/servo Source-Revision: f932db34c85dcd65a46ddde3ce0d8f6c92c28b7b
servo/python/servo/testing_commands.py
--- a/servo/python/servo/testing_commands.py
+++ b/servo/python/servo/testing_commands.py
@@ -380,19 +380,22 @@ class MachCommands(CommandBase):
         self.ensure_bootstrapped()
         run_file = path.abspath(path.join("tests", "wpt", "run_css.py"))
         return self.wptrunner(run_file, **kwargs)
 
     @Command('update-css',
              description='Update the web platform tests',
              category='testing',
              parser=updatecommandline.create_parser())
-    def update_css(self, **kwargs):
+    @CommandArgument('--patch', action='store_true', default=False,
+                     help='Create an mq patch or git commit containing the changes')
+    def update_css(self, patch, **kwargs):
         self.ensure_bootstrapped()
         run_file = path.abspath(path.join("tests", "wpt", "update_css.py"))
+        kwargs["no_patch"] = not patch
         run_globals = {"__file__": run_file}
         execfile(run_file, run_globals)
         return run_globals["update_tests"](**kwargs)
 
     @Command('compare_dromaeo',
              description='compare outputs of two runs of ./mach test-dromaeo command',
              category='testing')
     @CommandArgument('params', default=None, nargs="...",