This is wave 2 - for Bug 1282408 As for some member variables we want to ignore it's initialisation check performed by clang-plugin - https://bugzilla.mozilla.org/show_bug.cgi?id=525063 We've added a flag for the declaration of each variable that we want to be ignored. On this bug there will be posted patches of this kind.

Comment on attachment 8799765 [details] Bug 1308868 - Add ignore-initialization-check annotation to mPendingTextTrackChange from TextTrackList. https://reviewboard.mozilla.org/r/84888/#review83506 ::: dom/media/TextTrackList.h:70 (Diff revision 1) > > IMPL_EVENT_HANDLER(change) > IMPL_EVENT_HANDLER(addtrack) > IMPL_EVENT_HANDLER(removetrack) > > - bool mPendingTextTrackChange; > + MOZ_INIT_OUTSIDE_CTOR bool mPendingTextTrackChange; Isn't the warning telling us that on some constructor paths mPendingTrackChange isn't initialized? How about we just go: bool mPendingTextTrackChange = false; here, so it's alway initialized?

MozReview-Commit-ID: FNTRHFpTx2E

MozReview-Commit-ID: 2dLsmPWeqSk

MozReview-Commit-ID: GT8oz6sCxaX

Pushed by bpostelnicu@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/2b08765d5eb6 Add ignore-initialization-check annotation to mWorkerPrivate from ClientNavigateRunnable. r=baku

Pushed by bpostelnicu@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/ba0be6c3f68a Add ignore-initialization-check annotation to variables from TelephonyCall and TelephonyCallGroup. r=echen

https://hg.mozilla.org/mozilla-central/rev/2b08765d5eb6 https://hg.mozilla.org/mozilla-central/rev/ba0be6c3f68a

MozReview-Commit-ID: 1dIKcbbeStW

Pushed by bpostelnicu@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/f65cc8c9aace Add ignore-initialization-check annotation to mErrorCode from U2FStatus. r=baku

MozReview-Commit-ID: ZDCY7i19x5

MozReview-Commit-ID: AYPnCUEtMrE

Comment on attachment 8802097 [details] [diff] [review] Add ignore-initialization-check annotation to mPosition and mLine from TextTrackCue Review of attachment 8802097 [details] [diff] [review]: ----------------------------------------------------------------- forward review to Ralph.

Comment on attachment 8802097 [details] [diff] [review] Add ignore-initialization-check annotation to mPosition and mLine from TextTrackCue Review of attachment 8802097 [details] [diff] [review]: ----------------------------------------------------------------- This is fine; the accesses are protected by the `IsAutoKeyword` tags. It would be better to convert these to a Variant type so the compiler checks for us, though.

Pushed by bpostelnicu@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/40a7a506f5bb Add ignore-initialization-check annotation to time from RecentURIKey. r=adw

Pushed by bpostelnicu@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/37e81a1c152c Add ignore-initialization-check annotation to mPosition and mLine from TextTrackCue. r=giles

Comment on attachment 8799342 [details] Bug 1308868 - Add ignore-initialization-check annotation to members from TriangleTyped. https://reviewboard.mozilla.org/r/84544/#review87488

Too late for firefox 52, mass-wontfix.

The leave-open keyword is there and there is no activity for 6 months. :andi, maybe it's time to close this bug?

Most of the work for this bug has already landed in m-c, an since now we will have coverity during review phase this is no longer necessary, so closing it.