Bug 1231378 - part 2 - Fix uninitialized members of classes in dom/*, r=smaug

dom/alarm/AlarmHalService.h

@@ -33,6 +33,10 @@ public:
   NS_DECL_ISUPPORTS
   NS_DECL_NSIALARMHALSERVICE
 
+  AlarmHalService()
+    : mAlarmEnabled(false)
+  {}
+
   void Init();
 
   static already_AddRefed<AlarmHalService> GetInstance();

dom/broadcastchannel/BroadcastChannelChild.cpp

@@ -27,7 +27,8 @@ namespace dom {
 using namespace workers;
 
 BroadcastChannelChild::BroadcastChannelChild(const nsACString& aOrigin)
-  : mActorDestroyed(false)
+  : mBC(nullptr)
+  , mActorDestroyed(false)
 {
   CopyUTF8toUTF16(aOrigin, mOrigin);
 }

dom/datastore/DataStoreService.cpp

@@ -79,6 +79,8 @@ public:
                 const nsAString& aManifestURL,
                 bool aReadOnly,
                 bool aEnabled)
+    : mReadOnly(true)
+    , mEnabled(false)
   {
     Init(aName, aOriginURL, aManifestURL, aReadOnly, aEnabled);
   }

dom/fetch/InternalRequest.cpp

@@ -86,7 +86,6 @@ InternalRequest::InternalRequest(const InternalRequest& aOther)
   , mForceOriginHeader(aOther.mForceOriginHeader)
   , mPreserveContentCodings(aOther.mPreserveContentCodings)
   , mSameOriginDataURL(aOther.mSameOriginDataURL)
-  , mSandboxedStorageAreaURLs(aOther.mSandboxedStorageAreaURLs)
   , mSkipServiceWorker(aOther.mSkipServiceWorker)
   , mSynchronous(aOther.mSynchronous)
   , mUnsafeRequest(aOther.mUnsafeRequest)

dom/fetch/InternalRequest.h

@@ -91,6 +91,7 @@ public:
   explicit InternalRequest()
     : mMethod("GET")
     , mHeaders(new InternalHeaders(HeadersGuardEnum::None))
+    , mContentPolicyType(nsIContentPolicy::TYPE_FETCH)
     , mReferrer(NS_LITERAL_STRING(kFETCH_CLIENT_REFERRER_STR))
     , mMode(RequestMode::No_cors)
     , mCredentialsMode(RequestCredentials::Omit)
@@ -413,7 +414,6 @@ private:
   bool mForceOriginHeader;
   bool mPreserveContentCodings;
   bool mSameOriginDataURL;
-  bool mSandboxedStorageAreaURLs;
   bool mSkipServiceWorker;
   bool mSynchronous;
   bool mUnsafeRequest;

dom/geolocation/nsGeolocationSettings.h

@@ -102,7 +102,14 @@ public:
   inline const nsString& GetOrigin() const { return mOrigin; }
 
 private:
-  GeolocationSetting() {} // can't default construct
+  GeolocationSetting() :
+#ifdef MOZ_APPROX_LOCATION
+    mDistance(0),
+#endif
+    mLatitude(0),
+    mLongitude(0)
+  {} // can't default construct
+
   GeolocationFuzzMethod mFuzzMethod;
 #ifdef MOZ_APPROX_LOCATION
   int32_t         mDistance;

dom/indexedDB/IDBRequest.cpp

@@ -51,6 +51,13 @@ NS_DEFINE_IID(kIDBRequestIID, PRIVATE_IDBREQUEST_IID);
 
 IDBRequest::IDBRequest(IDBDatabase* aDatabase)
   : IDBWrapperCache(aDatabase)
+#ifdef DEBUG
+  , mOwningThread(nullptr)
+#endif
+  , mLoggingSerialNumber(0)
+  , mLineNo(0)
+  , mColumn(0)
+  , mHaveResultOrErrorCode(false)
 {
   MOZ_ASSERT(aDatabase);
   aDatabase->AssertIsOnOwningThread();
@@ -60,6 +67,13 @@ IDBRequest::IDBRequest(IDBDatabase* aDatabase)
 
 IDBRequest::IDBRequest(nsPIDOMWindow* aOwner)
   : IDBWrapperCache(aOwner)
+#ifdef DEBUG
+  , mOwningThread(nullptr)
+#endif
+  , mLoggingSerialNumber(0)
+  , mLineNo(0)
+  , mColumn(0)
+  , mHaveResultOrErrorCode(false)
 {
   InitMembers();
 }

dom/messagechannel/MessagePortChild.h

@@ -19,7 +19,7 @@ class MessagePortChild final : public PMessagePortChild
 public:
   NS_INLINE_DECL_REFCOUNTING(MessagePortChild)
 
-  MessagePortChild() {}
+  MessagePortChild() : mPort(nullptr) {}
 
   void SetPort(MessagePort* aPort)
   {

dom/mobileconnection/MobileCallForwardingOptions.h

@@ -27,7 +27,13 @@ public:
 
 private:
   // Don't try to use the default constructor.
-  MobileCallForwardingOptions() {}
+  MobileCallForwardingOptions()
+    : mActive(false)
+    , mAction(0)
+    , mReason(0)
+    , mTimeSeconds(0)
+    , mServiceClass(0)
+  {}
 
   ~MobileCallForwardingOptions() {}
 

dom/mobileconnection/ipc/MobileConnectionChild.cpp

@@ -18,6 +18,8 @@ NS_IMPL_ISUPPORTS(MobileConnectionChild, nsIMobileConnection)
 MobileConnectionChild::MobileConnectionChild(uint32_t aServiceId)
   : mServiceId(aServiceId)
   , mLive(true)
+  , mRadioState(0)
+  , mNetworkSelectionMode(0)
 {
   MOZ_COUNT_CTOR(MobileConnectionChild);
 }

dom/network/UDPSocket.cpp

@@ -118,6 +118,8 @@ UDPSocket::UDPSocket(nsPIDOMWindow* aOwner,
   : DOMEventTargetHelper(aOwner)
   , mRemoteAddress(aRemoteAddress)
   , mRemotePort(aRemotePort)
+  , mAddressReuse(false)
+  , mLoopback(false)
   , mReadyState(SocketReadyState::Opening)
 {
   MOZ_ASSERT(aOwner);

dom/power/PowerManagerService.h

@@ -31,6 +31,10 @@ public:
   NS_DECL_ISUPPORTS
   NS_DECL_NSIPOWERMANAGERSERVICE
 
+  PowerManagerService()
+    : mWatchdogTimeoutSecs(0)
+  {}
+
   static already_AddRefed<PowerManagerService> GetInstance();
 
   void Init();

dom/tv/TVTuner.cpp

@@ -32,6 +32,7 @@ NS_INTERFACE_MAP_END_INHERITING(DOMEventTargetHelper)
 
 TVTuner::TVTuner(nsPIDOMWindow* aWindow)
   : DOMEventTargetHelper(aWindow)
+  , mStreamType(0)
 {
 }
 

dom/tv/TVTypes.cpp

@@ -20,6 +20,7 @@ NS_IMPL_ISUPPORTS(TVTunerData, nsITVTunerData)
 TVTunerData::TVTunerData()
   : mSupportedSourceTypes(nullptr)
   , mCount(0)
+  , mStreamType(0)
 {
 }
 
@@ -275,7 +276,9 @@ TVChannelData::SetIsFree(bool aIsFree)
 NS_IMPL_ISUPPORTS(TVProgramData, nsITVProgramData)
 
 TVProgramData::TVProgramData()
-  : mAudioLanguages(nullptr)
+  : mStartTime(0)
+  , mDuration(0)
+  , mAudioLanguages(nullptr)
   , mAudioLanguageCount(0)
   , mSubtitleLanguages(nullptr)
   , mSubtitleLanguageCount(0)

editor/libeditor/SetDocTitleTxn.cpp

@@ -25,6 +25,7 @@ using namespace mozilla;
 // note that aEditor is not refcounted
 SetDocTitleTxn::SetDocTitleTxn()
   : EditTxn()
+  , mEditor(nullptr)
   , mIsTransient(false)
 {
 }

editor/libeditor/nsHTMLEditRules.cpp

@@ -170,6 +170,13 @@ class nsEditableTextFunctor : public nsBoolDomIterFunctor
  ********************************************************/
 
 nsHTMLEditRules::nsHTMLEditRules()
+  : mHTMLEditor(nullptr)
+  , mListenerEnabled(false)
+  , mReturnInEmptyLIKillsList(false)
+  , mDidDeleteSelection(false)
+  , mDidRangedDelete(false)
+  , mRestoreContentEditableCount(false)
+  , mJoinOffset(0)
 {
   InitFields();
 }

editor/libeditor/nsHTMLEditor.cpp

@@ -96,17 +96,42 @@ IsNamedAnchorTag(const nsString& s)
 nsHTMLEditor::nsHTMLEditor()
 : nsPlaintextEditor()
 , mCRInParagraphCreatesParagraph(false)
+, mCSSAware(false)
 , mSelectedCellIndex(0)
 , mIsObjectResizingEnabled(true)
 , mIsResizing(false)
+, mPreserveRatio(false)
+, mResizedObjectIsAnImage(false)
 , mIsAbsolutelyPositioningEnabled(true)
 , mResizedObjectIsAbsolutelyPositioned(false)
 , mGrabberClicked(false)
 , mIsMoving(false)
 , mSnapToGridEnabled(false)
 , mIsInlineTableEditingEnabled(true)
+, mOriginalX(0)
+, mOriginalY(0)
+, mResizedObjectX(0)
+, mResizedObjectY(0)
+, mResizedObjectWidth(0)
+, mResizedObjectHeight(0)
+, mResizedObjectMarginLeft(0)
+, mResizedObjectMarginTop(0)
+, mResizedObjectBorderLeft(0)
+, mResizedObjectBorderTop(0)
+, mXIncrementFactor(0)
+, mYIncrementFactor(0)
+, mWidthIncrementFactor(0)
+, mHeightIncrementFactor(0)
 , mInfoXIncrement(20)
 , mInfoYIncrement(20)
+, mPositionedObjectX(0)
+, mPositionedObjectY(0)
+, mPositionedObjectWidth(0)
+, mPositionedObjectHeight(0)
+, mPositionedObjectMarginLeft(0)
+, mPositionedObjectMarginTop(0)
+, mPositionedObjectBorderLeft(0)
+, mPositionedObjectBorderTop(0)
 , mGridSize(0)
 {
 }

editor/libeditor/nsHTMLURIRefObject.cpp

@@ -59,8 +59,8 @@
 #define MATCHES(tagName, str) tagName.EqualsIgnoreCase(str)
 
 nsHTMLURIRefObject::nsHTMLURIRefObject()
+  : mCurAttrIndex(0), mAttributeCnt(0)
 {
-  mCurAttrIndex = mAttributeCnt = 0;
 }
 
 nsHTMLURIRefObject::~nsHTMLURIRefObject()

editor/libeditor/nsTextEditRules.cpp

@@ -57,6 +57,15 @@ using namespace mozilla::dom;
  ********************************************************/
 
 nsTextEditRules::nsTextEditRules()
+  : mEditor(nullptr)
+  , mPasswordIMEIndex(0)
+  , mCachedSelectionOffset(0)
+  , mActionNesting(0)
+  , mLockRulesSniffing(false)
+  , mDidExplicitlySetInterline(false)
+  , mDeleteBidiImmediately(false)
+  , mLastStart(0)
+  , mLastLength(0)
 {
   InitFields();
 }

editor/libeditor/nsTextEditRules.h

@@ -335,10 +335,21 @@ class nsAutoLockListener
 {
 public:
 
-  explicit nsAutoLockListener(bool *enabled) : mEnabled(enabled)
+  explicit nsAutoLockListener(bool *enabled)
-                 {if (mEnabled) { mOldState=*mEnabled; *mEnabled = false;}}
+    : mEnabled(enabled), mOldState(false)
+  {
+    if (mEnabled) {
+      mOldState = *mEnabled;
+      *mEnabled = false;
+    }
+  }
+
   ~nsAutoLockListener()
-                 {if (mEnabled) *mEnabled = mOldState;}
+  {
+    if (mEnabled) {
+      *mEnabled = mOldState;
+    }
+  }
 
 protected:
   bool *mEnabled;

editor/txtsvc/nsFilteredContentIterator.h

@@ -45,7 +45,7 @@ public:
   void         ClearDidSkip() {  mDidSkip = false; }
 
 protected:
-  nsFilteredContentIterator() { }
+  nsFilteredContentIterator() : mDidSkip(false), mIsOutOfRange(false) { }
 
   virtual ~nsFilteredContentIterator();
 

gfx/2d/Logging.h

@@ -260,7 +260,10 @@ public:
   // version of that method for different loggers, this is OK. Once we do,
   // change BasicLogger::ShouldOutputMessage to Logger::ShouldOutputMessage.
   explicit Log(int aOptions = Log::DefaultOptions(L == LOG_CRITICAL),
-               LogReason aReason = LogReason::MustBeMoreThanThis) {
+               LogReason aReason = LogReason::MustBeMoreThanThis)
+  : mOptions(0)
+  , mLogIt(false)
+  {
     Init(aOptions, BasicLogger::ShouldOutputMessage(L), aReason);
   }