Bug 1317322 - Part 3: Make RemoveTempFileHelper's constructor explicit, a=bustage
☠☠ backed out by d64711bdb426 ☠ ☠
authorMichael Layzell <michael@thelayzells.com>
Wed, 01 Mar 2017 15:48:08 -0500
changeset 394499 ee9b08930b6563a28f621f6220cd2e4bb60f83dd
parent 394498 4048d3a531078eb13aed019c5cc5f68da49967c5
child 394500 053c34ac7631c30954fd5cfabf5d6fcdc28571f0
push id1468
push userasasaki@mozilla.com
push dateMon, 05 Jun 2017 19:31:07 +0000
treeherdermozilla-release@0641fc6ee9d1 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbustage
bugs1317322
milestone54.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1317322 - Part 3: Make RemoveTempFileHelper's constructor explicit, a=bustage MozReview-Commit-ID: 9Fm95kclBWq
widget/windows/nsDataObj.cpp
--- a/widget/windows/nsDataObj.cpp
+++ b/widget/windows/nsDataObj.cpp
@@ -440,17 +440,17 @@ STDMETHODIMP_(ULONG) nsDataObj::AddRef()
 	NS_LOG_ADDREF(this, m_cRef, "nsDataObj", sizeof(*this));
 	return m_cRef;
 }
 
 namespace {
 class RemoveTempFileHelper : public nsIObserver
 {
 public:
-  RemoveTempFileHelper(nsIFile* aTempFile)
+  explicit RemoveTempFileHelper(nsIFile* aTempFile)
     : mTempFile(aTempFile)
   {
     MOZ_ASSERT(mTempFile);
   }
 
   // The attach method is seperate from the constructor as we may be addref-ing
   // ourself, and we want to be sure someone has a strong reference to us.
   void Attach()