Bug 1079317 - Fix some more bad implicit constructors in DOM; r=baku

content/base/src/ImportManager.h

@@ -70,7 +70,7 @@ class ImportLoader MOZ_FINAL : public nsIStreamListener
   class Updater {
 
   public:
-    Updater(ImportLoader* aLoader) : mLoader(aLoader)
+    explicit Updater(ImportLoader* aLoader) : mLoader(aLoader)
     {}
 
     // After a new link is added that refers to this import, we

content/base/src/nsDocument.cpp

@@ -555,7 +555,7 @@ namespace {
 struct PositionComparator
 {
   Element* const mElement;
-  PositionComparator(Element* const aElement) : mElement(aElement) {}
+  explicit PositionComparator(Element* const aElement) : mElement(aElement) {}
 
   int operator()(void* aElement) const {
     Element* curElement = static_cast<Element*>(aElement);

content/base/src/nsPlainTextSerializer.cpp

@@ -1873,7 +1873,7 @@ struct interval
 struct CombiningComparator
 {
   const char16_t mUcs;
-  CombiningComparator(char16_t ucs) : mUcs(ucs) {}
+  explicit CombiningComparator(char16_t aUcs) : mUcs(aUcs) {}
   int operator()(const interval& combining) const {
     if (mUcs > combining.last)
       return 1;

content/html/content/src/HTMLFormElement.cpp

@@ -2244,7 +2244,7 @@ namespace {
 struct PositionComparator
 {
   nsIContent* const mElement;
-  PositionComparator(nsIContent* const element) : mElement(element) {}
+  explicit PositionComparator(nsIContent* const aElement) : mElement(aElement) {}
 
   int operator()(nsIContent* aElement) const {
     if (mElement == aElement) {
@@ -2260,7 +2260,7 @@ struct PositionComparator
 struct NodeListAdaptor
 {
   nsINodeList* const mList;
-  NodeListAdaptor(nsINodeList* aList) : mList(aList) {}
+  explicit NodeListAdaptor(nsINodeList* aList) : mList(aList) {}
   nsIContent* operator[](size_t aIdx) const {
     return mList->Item(aIdx);
   }

dom/canvas/WebGLContext.h

@@ -1363,7 +1363,7 @@ protected:
                    webgl.mColorWriteMask[3] != false;
         }
 
-        ScopedMaskWorkaround(WebGLContext& webgl);
+        explicit ScopedMaskWorkaround(WebGLContext& aWebgl);
 
         ~ScopedMaskWorkaround();
     };

dom/cellbroadcast/CellBroadcast.cpp

@@ -32,7 +32,7 @@ public:
   NS_DECL_ISUPPORTS
   NS_FORWARD_SAFE_NSICELLBROADCASTLISTENER(mCellBroadcast)
 
-  Listener(CellBroadcast* aCellBroadcast)
+  explicit Listener(CellBroadcast* aCellBroadcast)
     : mCellBroadcast(aCellBroadcast)
   {
     MOZ_ASSERT(mCellBroadcast);

dom/fetch/Response.h

@@ -27,7 +27,7 @@ class Response MOZ_FINAL : public nsISupports
   NS_DECL_CYCLE_COLLECTION_SCRIPT_HOLDER_CLASS(Response)
 
 public:
-  Response(nsISupports* aOwner);
+  explicit Response(nsISupports* aOwner);
 
   JSObject*
   WrapObject(JSContext* aCx)

dom/ipc/Blob.cpp

@@ -329,7 +329,7 @@ class RemoteInputStream MOZ_FINAL
 public:
   NS_DECL_THREADSAFE_ISUPPORTS
 
-  RemoteInputStream(DOMFileImpl* aBlobImpl)
+  explicit RemoteInputStream(DOMFileImpl* aBlobImpl)
     : mMonitor("RemoteInputStream.mMonitor")
     , mBlobImpl(aBlobImpl)
     , mWeakSeekableStream(nullptr)

dom/voicemail/Voicemail.cpp

@@ -33,7 +33,7 @@ public:
   NS_DECL_ISUPPORTS
   NS_FORWARD_SAFE_NSIVOICEMAILLISTENER(mVoicemail)
 
-  Listener(Voicemail* aVoicemail)
+  explicit Listener(Voicemail* aVoicemail)
     : mVoicemail(aVoicemail)
   {
     MOZ_ASSERT(mVoicemail);

dom/voicemail/ipc/VoicemailIPCService.cpp

@@ -23,7 +23,7 @@ public:
   NS_DECL_ISUPPORTS
   NS_DECL_NSIVOICEMAILPROVIDER
 
-  VoicemailIPCProvider(uint32_t aServiceId);
+  explicit VoicemailIPCProvider(uint32_t aServiceId);
 
 private:
   // MOZ_FINAL suppresses -Werror,-Wdelete-non-virtual-dtor