geckodriver: marionette: append X11 style -marionette flag when starting browser
authorAndreas Tolfsen <>
Wed, 12 Apr 2017 14:51:16 +0100
changeset 581790 0663affcced9ec002c219ec077d62817c6a4ae9a
parent 581789 44089bdc00398d1ab45759c955b346242a0201a3
child 581791 d62542532dc1b6a196b6f24a821e1fa08de6fe27
push id59896
push dateFri, 19 May 2017 23:30:01 +0000
geckodriver: marionette: append X11 style -marionette flag when starting browser was recently submitted to make mozrunner not imply starting the Marionette server by passing the --marionette flag. This patch appends -marionette, with a single dash, so that it will be accepted on Windows systems. More discussion around this in Fixes: Source-Repo: Source-Revision: 7c577c04176c1cc7b5bd45928b3a36bd1818c5ae committer: jgraham <>
--- a/testing/geckodriver/src/
+++ b/testing/geckodriver/src/
@@ -309,16 +309,20 @@ impl MarionetteHandler {
                                         'moz:firefoxOptions.binary' capability provided, and \
                                         no binary flag set on the command line")));
         let custom_profile = options.profile.is_some();
         let mut runner = try!(FirefoxRunner::new(&binary, options.profile.take())
                               .map_err(|e| WebDriverError::new(ErrorStatus::SessionNotCreated,
+        // double-dashed flags are not accepted on Windows systems
+        runner.args().push("-marionette");
         if let Some(args) = options.args.take() {
         try!(self.set_prefs(port, &mut runner.profile, custom_profile, options.prefs)
             .map_err(|e| {
                                     format!("Failed to set preferences: {}", e))