Default to the same Cargo features in all build commands

python/servo/build_commands.py

@@ -147,7 +147,7 @@ def build(self, target=None, release=False, dev=False, jobs=None,
               features=None, android=None, verbose=False, debug_mozjs=False, params=None):
         if android is None:
             android = self.config["build"]["android"]
-        features = features or []
+        features = features or self.servo_features()
 
         opts = params or []
 
@@ -194,7 +194,7 @@ def build(self, target=None, release=False, dev=False, jobs=None,
 
         self.ensure_bootstrapped(target=target)
 
-        if debug_mozjs or self.config["build"]["debug-mozjs"]:
+        if debug_mozjs:
             features += ["script/debugmozjs"]
 
         if features:
@@ -265,6 +265,10 @@ def build_cef(self, jobs=None, verbose=False, release=False):
         if release:
             opts += ["--release"]
 
+        servo_features = self.servo_features()
+        if servo_features:
+            opts += ["--features", "%s" % ' '.join("servo/" + x for x in servo_features)]
+
         build_start = time()
         with cd(path.join("ports", "cef")):
             ret = call(["cargo", "build"] + opts,

python/servo/command_base.py

@@ -415,6 +415,13 @@ def build_env(self, gonk=False, hosts_file_path=None, target=None):
     def servo_crate(self):
         return path.join(self.context.topdir, "components", "servo")
 
+    def servo_features(self):
+        """Return a list of optional features to enable for the Servo crate"""
+        features = []
+        if self.config["build"]["debug-mozjs"]:
+            features += ["script/debugmozjs"]
+        return features
+
     def android_support_dir(self):
         return path.join(self.context.topdir, "support", "android")
 

python/servo/testing_commands.py

@@ -178,6 +178,10 @@ def test_unit(self, test_name=None, package=None):
             args += ["-p", "%s_tests" % crate]
         args += test_patterns
 
+        features = self.servo_features()
+        if features:
+            args += ["--features", "%s" % ' '.join(features)]
+
         env = self.build_env()
         env["RUST_BACKTRACE"] = "1"