Bug 1109697 - Fix more bad implicit constructors in netwerk; r=mcmanus

netwerk/base/src/OfflineObserver.h

@@ -48,7 +48,7 @@ class OfflineObserver
 public:
   // A nsRefPtr to this object should be kept by the disconnectable parent.
 
-  OfflineObserver(DisconnectableParent * parent);
+  explicit OfflineObserver(DisconnectableParent * parent);
   // This method needs to be called in the destructor of the parent
   // It removes the observer from the nsObserverService list, and it clears
   // the pointer it holds to the disconnectable parent.

netwerk/base/src/ProxyAutoConfig.cpp

@@ -559,7 +559,7 @@ private:
 
   static const JSClass sGlobalClass;
 
-  JSRuntimeWrapper(JSRuntime* rt)
+  explicit JSRuntimeWrapper(JSRuntime* rt)
      : mRuntime(rt), mContext(nullptr), mGlobal(rt, nullptr), mOK(false)
   {
       MOZ_COUNT_CTOR(JSRuntimeWrapper);

netwerk/protocol/http/AlternateServices.h

@@ -101,7 +101,7 @@ public:
     NS_DECL_NSISPECULATIVECONNECTIONOVERRIDER
     NS_DECL_NSIINTERFACEREQUESTOR
 
-    AltSvcOverride(nsIInterfaceRequestor *aRequestor)
+    explicit AltSvcOverride(nsIInterfaceRequestor *aRequestor)
       : mCallbacks(aRequestor) {}
 
 private:

netwerk/protocol/http/InterceptedChannel.h

@@ -39,7 +39,7 @@ protected:
 
   virtual ~InterceptedChannelBase();
 public:
-  InterceptedChannelBase(nsINetworkInterceptController* aController);
+  explicit InterceptedChannelBase(nsINetworkInterceptController* aController);
 
   // Notify the interception controller that the channel has been intercepted
   // and prepare the response body output stream.

netwerk/protocol/websocket/WebSocketChannel.cpp

@@ -1969,7 +1969,7 @@ class RemoveObserverRunnable : public nsRunnable
   nsRefPtr<WebSocketChannel> mChannel;
 
 public:
-  RemoveObserverRunnable(WebSocketChannel* aChannel)
+  explicit RemoveObserverRunnable(WebSocketChannel* aChannel)
     : mChannel(aChannel)
   {}