Bug 1598709 - Fix up the Windows platforms that were overriding the variant settings r=ahal

Differential Revision: https://phabricator.services.mozilla.com/D54335

--HG--
extra : moz-landing-system : lando

js/src/devtools/automation/autospider.py

@@ -189,9 +189,6 @@ if opt is not None:
     CONFIGURE_ARGS += (" --enable-optimize" if opt else " --disable-optimize")
 
 opt = args.debug
-if opt is None and args.platform:
-    # Override variant['debug'].
-    opt = ('-debug' in args.platform)
 if opt is None:
     opt = variant.get('debug')
 if opt is not None:

js/src/vm/BytecodeLocation-inl.h

@@ -14,7 +14,7 @@
 
 namespace js {
 
-inline bool BytecodeLocation::isValid(const JSScript* script) const {
+inline JS_PUBLIC_API bool BytecodeLocation::isValid(const JSScript* script) const {
   // Note: Don't create a new BytecodeLocation during the implementation of
   // this, as it is used in the constructor, and will recurse forever.
   return script->contains(*this) || toRawBytecode() == script->codeEnd();

js/src/vm/BytecodeLocation.h

@@ -67,7 +67,7 @@ class BytecodeLocation {
 
   // Return true if this bytecode location is valid for the given script.
   // This includes the location 1-past the end of the bytecode.
-  bool isValid(const JSScript* script) const;
+  JS_PUBLIC_API bool isValid(const JSScript* script) const;
 
   // Return true if this bytecode location is within the bounds of the
   // bytecode for a given script.