Bug 1407432 - Remove unused builtins arg from Sandbox; r=froydnj
authorMike Shal <mshal@mozilla.com>
Wed, 08 Nov 2017 19:58:28 -0500
changeset 444567 94f475532485aacdbb8e2c664a8da4a68e0c9e93
parent 444566 3258f49e8c7378add641f5792fa449a39bc29880
child 444568 ccc34ca6ecfde0b52f0725caeefc03ecf5ec20a5
push id1618
push userCallek@gmail.com
push dateThu, 11 Jan 2018 17:45:48 +0000
treeherdermozilla-release@882ca853e05a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersfroydnj
bugs1407432
milestone58.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1407432 - Remove unused builtins arg from Sandbox; r=froydnj MozReview-Commit-ID: JVukxIg04AT
python/mozbuild/mozbuild/frontend/sandbox.py
--- a/python/mozbuild/mozbuild/frontend/sandbox.py
+++ b/python/mozbuild/mozbuild/frontend/sandbox.py
@@ -108,20 +108,20 @@ class Sandbox(dict):
         'None': None,
         'False': False,
         'True': True,
         'sorted': alphabetical_sorted,
         'int': int,
         'set': set,
     })
 
-    def __init__(self, context, builtins=None, finder=default_finder):
+    def __init__(self, context, finder=default_finder):
         """Initialize a Sandbox ready for execution.
         """
-        self._builtins = builtins or self.BUILTINS
+        self._builtins = self.BUILTINS
         dict.__setitem__(self, '__builtins__', self._builtins)
 
         assert isinstance(self._builtins, ReadOnlyDict)
         assert isinstance(context, Context)
 
         # Contexts are modeled as a stack because multiple context managers
         # may be active.
         self._active_contexts = [context]