Bug 1316290 - Replace default bodies of special member functions with = default, in layout. r=dholbert
authorAndi-Bogdan Postelnicu <bpostelnicu@mozilla.com>
Wed, 09 Nov 2016 15:35:40 +0200
changeset 352276 19f65d152d88b2db84174fffce1d822435c08c21
parent 352275 b1e344ef212dbd4ba46c082d7840babc9a55f685
child 352277 c7054d08b878904bc413750fc7a70d6ea649c9e8
push id6795
push userjlund@mozilla.com
push dateMon, 23 Jan 2017 14:19:46 +0000
treeherdermozilla-esr52@76101b503191 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdholbert
bugs1316290
milestone52.0a1
Bug 1316290 - Replace default bodies of special member functions with = default, in layout. r=dholbert MozReview-Commit-ID: LJiBYdYD1S5
layout/base/nsRefreshDriver.cpp
layout/style/nsCSSRuleProcessor.cpp
--- a/layout/base/nsRefreshDriver.cpp
+++ b/layout/base/nsRefreshDriver.cpp
@@ -512,17 +512,17 @@ private:
 
     void OnTimerStart()
     {
       if (!XRE_IsParentProcess()) {
         mLastChildTick = TimeStamp::Now();
       }
     }
   private:
-    ~RefreshDriverVsyncObserver() {}
+    ~RefreshDriverVsyncObserver() = default;
 
     void RecordTelemetryProbes(TimeStamp aVsyncTimestamp)
     {
       MOZ_ASSERT(NS_IsMainThread());
     #ifndef ANDROID  /* bug 1142079 */
       if (XRE_IsParentProcess()) {
         TimeDuration vsyncLatency = TimeStamp::Now() - aVsyncTimestamp;
         uint32_t sample = (uint32_t)vsyncLatency.ToMilliseconds();
@@ -861,17 +861,17 @@ public:
     MOZ_ASSERT(aPBackgroundChild);
 
     layout::PVsyncChild* actor = aPBackgroundChild->SendPVsyncConstructor();
     layout::VsyncChild* child = static_cast<layout::VsyncChild*>(actor);
     nsRefreshDriver::PVsyncActorCreated(child);
   }
 
 private:
-  virtual ~VsyncChildCreateCallback() {}
+  virtual ~VsyncChildCreateCallback() = default;
 
   void ActorCreated(PBackgroundChild* aPBackgroundChild) override
   {
     MOZ_ASSERT(NS_IsMainThread());
     MOZ_ASSERT(aPBackgroundChild);
     CreateVsyncActor(aPBackgroundChild);
   }
 
--- a/layout/style/nsCSSRuleProcessor.cpp
+++ b/layout/style/nsCSSRuleProcessor.cpp
@@ -765,19 +765,17 @@ RuleHash::SizeOfIncludingThis(MallocSize
 struct SelectorPair
 {
   SelectorPair(nsCSSSelector* aSelector, nsCSSSelector* aRightmostSelector)
     : mSelector(aSelector), mRightmostSelector(aRightmostSelector)
   {
     MOZ_ASSERT(aSelector);
     MOZ_ASSERT(mRightmostSelector);
   }
-  SelectorPair(const SelectorPair& aOther)
-    : mSelector(aOther.mSelector)
-    , mRightmostSelector(aOther.mRightmostSelector) {}
+  SelectorPair(const SelectorPair& aOther) = default;
   nsCSSSelector* const mSelector;
   nsCSSSelector* const mRightmostSelector;
 };
 
 // A hash table mapping atoms to lists of selectors
 struct AtomSelectorEntry : public PLDHashEntryHdr {
   nsIAtom *mAtom;
   // Auto length 2, because a decent fraction of these arrays ends up