Bug 1109702 - Fix more bad implicit constructors in js; r=Waldo

js/src/asmjs/AsmJSValidate.cpp

@@ -5376,7 +5376,7 @@ class CheckSimdScalarArgs
     Type formalType_;
 
   public:
-    CheckSimdScalarArgs(AsmJSSimdType simdType)
+    explicit CheckSimdScalarArgs(AsmJSSimdType simdType)
       : simdType_(simdType), formalType_(SimdToCoercedScalarType(simdType))
     {}
 

js/src/builtin/AtomicsObject.cpp

@@ -728,7 +728,7 @@ class AutoLockFutexAPI
 {
     JS::PerRuntimeFutexAPI * const fx;
   public:
-    AutoLockFutexAPI(JS::PerRuntimeFutexAPI *fx) : fx(fx) {
+    explicit AutoLockFutexAPI(JS::PerRuntimeFutexAPI *fx) : fx(fx) {
         fx->lock();
     }
     ~AutoLockFutexAPI() {

js/src/builtin/TypedObject.h

@@ -1060,7 +1060,7 @@ class LazyArrayBufferTable
     Map map;
 
   public:
-    LazyArrayBufferTable(JSContext *cx);
+    explicit LazyArrayBufferTable(JSContext *cx);
     ~LazyArrayBufferTable();
 
     ArrayBufferObject *maybeBuffer(InlineTransparentTypedObject *obj);

js/src/jit/JitCompartment.h

@@ -287,7 +287,7 @@ class JitRuntime
     {
         JitRuntime *jrt_;
       public:
-        AutoMutateBackedges(JitRuntime *jrt) : jrt_(jrt) {
+        explicit AutoMutateBackedges(JitRuntime *jrt) : jrt_(jrt) {
             MOZ_ASSERT(!jrt->mutatingBackedgeList_);
             jrt->mutatingBackedgeList_ = true;
         }

js/src/jit/JitFrames.cpp

@@ -553,7 +553,7 @@ HandleClosingGeneratorReturn(JSContext *cx, const JitFrameIterator &frame, jsbyt
 struct AutoDebuggerHandlingException
 {
     BaselineFrame *frame;
-    AutoDebuggerHandlingException(BaselineFrame *frame)
+    explicit AutoDebuggerHandlingException(BaselineFrame *frame)
       : frame(frame)
     {
         frame->setIsDebuggerHandlingException();

js/src/jit/LIR-Common.h

@@ -4301,7 +4301,7 @@ class LTypedObjectDescr : public LInstructionHelper<1, 1, 0>
   public:
     LIR_HEADER(TypedObjectDescr)
 
-    LTypedObjectDescr(const LAllocation &object) {
+    explicit LTypedObjectDescr(const LAllocation &object) {
         setOperand(0, object);
     }
     const LAllocation *object() {

js/src/jit/MIR.h

@@ -10507,7 +10507,7 @@ class MGetDOMProperty
     const JSJitInfo *info_;
 
   protected:
-    MGetDOMProperty(const JSJitInfo *jitinfo)
+    explicit MGetDOMProperty(const JSJitInfo *jitinfo)
       : info_(jitinfo)
     {
         MOZ_ASSERT(jitinfo);
@@ -10624,7 +10624,7 @@ class MGetDOMProperty
 class MGetDOMMember : public MGetDOMProperty
 {
     // We inherit everything from MGetDOMProperty except our possiblyCalls value
-    MGetDOMMember(const JSJitInfo *jitinfo)
+    explicit MGetDOMMember(const JSJitInfo *jitinfo)
         : MGetDOMProperty(jitinfo)
     {
     }

js/src/jsgc.h

@@ -638,7 +638,7 @@ class ArenaLists
     ArenaHeader *savedEmptyObjectArenas;
 
   public:
-    ArenaLists(JSRuntime *rt) : runtime_(rt) {
+    explicit ArenaLists(JSRuntime *rt) : runtime_(rt) {
         for (size_t i = 0; i != FINALIZE_LIMIT; ++i)
             freeLists[i].initAsEmpty();
         for (size_t i = 0; i != FINALIZE_LIMIT; ++i)

js/src/jsinfer.h

@@ -679,7 +679,7 @@ class AutoClearTypeInferenceStateOnOOM
     bool oom;
 
   public:
-    AutoClearTypeInferenceStateOnOOM(Zone *zone)
+    explicit AutoClearTypeInferenceStateOnOOM(Zone *zone)
       : zone(zone), oom(false)
     {}
 

js/xpconnect/src/XPCWrappedJSClass.cpp

@@ -86,7 +86,7 @@ bool xpc_IsReportableErrorCode(nsresult code)
 // PendingResult.
 class MOZ_STACK_CLASS AutoSavePendingResult {
 public:
-    AutoSavePendingResult(XPCContext *xpcc) :
+    explicit AutoSavePendingResult(XPCContext *xpcc) :
         mXPCContext(xpcc)
     {
         // Save any existing pending result and reset to NS_OK for this invocation.