Bug 1613985 - Use default for equivalent-to-default constructors/destructors in gfx. r=jrmuizel

Differential Revision: https://phabricator.services.mozilla.com/D65288

--HG--
extra : moz-landing-system : lando

gfx/2d/2D.h

@@ -203,7 +203,7 @@ class GradientStops : public external::AtomicRefCounted<GradientStops> {
   virtual bool IsValid() const { return true; }
 
  protected:
-  GradientStops() {}
+  GradientStops() = default;
 };
 
 /**
@@ -929,7 +929,7 @@ class UnscaledFont : public SupportsThreadSafeWeakPtr<UnscaledFont> {
   }
 
  protected:
-  UnscaledFont() {}
+  UnscaledFont() = default;
 
  private:
   static Atomic<uint32_t> sDeletionCounter;

gfx/2d/Blur.cpp

@@ -545,7 +545,7 @@ AlphaBoxBlur::AlphaBoxBlur(const Rect& aRect, int32_t aStride, float aSigmaX,
   }
 }
 
-AlphaBoxBlur::~AlphaBoxBlur() {}
+AlphaBoxBlur::~AlphaBoxBlur() = default;
 
 IntSize AlphaBoxBlur::GetSize() const {
   IntSize size(mRect.Width(), mRect.Height());

gfx/2d/ConvolutionFilter.cpp

@@ -18,7 +18,7 @@ namespace mozilla::gfx {
 ConvolutionFilter::ConvolutionFilter()
     : mFilter(MakeUnique<SkConvolutionFilter1D>()) {}
 
-ConvolutionFilter::~ConvolutionFilter() {}
+ConvolutionFilter::~ConvolutionFilter() = default;
 
 int32_t ConvolutionFilter::MaxFilter() const { return mFilter->maxFilter(); }
 

gfx/2d/DrawCommands.h

@@ -821,7 +821,7 @@ class PushLayerCommand : public DrawingCommand {
 
 class PopClipCommand : public DrawingCommand {
  public:
-  PopClipCommand() {}
+  PopClipCommand() = default;
 
   CommandType GetType() const override { return PopClipCommand::Type; }
 
@@ -841,7 +841,7 @@ class PopClipCommand : public DrawingCommand {
 
 class PopLayerCommand : public DrawingCommand {
  public:
-  PopLayerCommand() {}
+  PopLayerCommand() = default;
 
   CommandType GetType() const override { return PopLayerCommand::Type; }
 

gfx/2d/DrawEventRecorder.h

@@ -227,7 +227,7 @@ class DrawEventRecorderMemory : public DrawEventRecorderPrivate {
   MemStream mIndex;
 
  protected:
-  virtual ~DrawEventRecorderMemory(){};
+  virtual ~DrawEventRecorderMemory() = default;
 
  private:
   SerializeResourcesFn mSerializeCallback;

gfx/2d/DrawTargetOffset.cpp

@@ -11,7 +11,7 @@
 namespace mozilla {
 namespace gfx {
 
-DrawTargetOffset::DrawTargetOffset() {}
+DrawTargetOffset::DrawTargetOffset() = default;
 
 bool DrawTargetOffset::Init(DrawTarget* aDrawTarget, IntPoint aOrigin) {
   mDrawTarget = aDrawTarget;

gfx/2d/DrawTargetTiled.cpp

@@ -11,7 +11,7 @@
 namespace mozilla {
 namespace gfx {
 
-DrawTargetTiled::DrawTargetTiled() {}
+DrawTargetTiled::DrawTargetTiled() = default;
 
 bool DrawTargetTiled::Init(const TileSet& aTiles) {
   if (!aTiles.mTileCount) {

gfx/2d/DrawingJob.cpp

@@ -11,7 +11,7 @@
 namespace mozilla {
 namespace gfx {
 
-DrawingJobBuilder::DrawingJobBuilder() {}
+DrawingJobBuilder::DrawingJobBuilder() = default;
 
 DrawingJobBuilder::~DrawingJobBuilder() { MOZ_ASSERT(!mDrawTarget); }
 

gfx/2d/Filters.h

@@ -436,7 +436,7 @@ class FilterNode : public external::AtomicRefCounted<FilterNode> {
  protected:
   friend class Factory;
 
-  FilterNode() {}
+  FilterNode() = default;
 };
 
 }  // namespace gfx

gfx/2d/GenericRefCounted.h

@@ -25,7 +25,7 @@ namespace mozilla {
  */
 class GenericRefCountedBase {
  protected:
-  virtual ~GenericRefCountedBase(){};
+  virtual ~GenericRefCountedBase() = default;
 
  public:
   // AddRef() and Release() method names are for compatibility with nsRefPtr.

gfx/2d/JobScheduler.cpp

@@ -114,7 +114,7 @@ SetEventJob::SetEventJob(EventObject* aEvent, SyncObject* aStart,
                          SyncObject* aCompletion, WorkerThread* aWorker)
     : Job(aStart, aCompletion, aWorker), mEvent(aEvent) {}
 
-SetEventJob::~SetEventJob() {}
+SetEventJob::~SetEventJob() = default;
 
 SyncObject::SyncObject(uint32_t aNumPrerequisites)
     : mSignals(aNumPrerequisites),

gfx/2d/Logging.h

@@ -216,11 +216,11 @@ class LogForwarder {
 
 class NoLog {
  public:
-  NoLog() {}
-  ~NoLog() {}
+  NoLog() = default;
+  ~NoLog() = default;
 
   // No-op
-  MOZ_IMPLICIT NoLog(const NoLog&) {}
+  MOZ_IMPLICIT NoLog(const NoLog&) = default;
 
   template <typename T>
   NoLog& operator<<(const T& aLogText) {

gfx/2d/NativeFontResourceFreeType.cpp

@@ -17,7 +17,7 @@ NativeFontResourceFreeType::NativeFontResourceFreeType(
       mDataLength(aDataLength),
       mFTLibrary(aFTLibrary) {}
 
-NativeFontResourceFreeType::~NativeFontResourceFreeType() {}
+NativeFontResourceFreeType::~NativeFontResourceFreeType() = default;
 
 template <class T>
 already_AddRefed<T> NativeFontResourceFreeType::CreateInternal(

gfx/2d/Path.cpp

@@ -32,7 +32,7 @@ struct PointD : public BasePoint<double, PointD> {
 };
 
 struct BezierControlPoints {
-  BezierControlPoints() {}
+  BezierControlPoints() = default;
   BezierControlPoints(const PointD& aCP1, const PointD& aCP2,
                       const PointD& aCP3, const PointD& aCP4)
       : mCP1(aCP1), mCP2(aCP2), mCP3(aCP3), mCP4(aCP4) {}
@@ -43,9 +43,9 @@ struct BezierControlPoints {
 void FlattenBezier(const BezierControlPoints& aPoints, PathSink* aSink,
                    double aTolerance);
 
-Path::Path() {}
+Path::Path() = default;
 
-Path::~Path() {}
+Path::~Path() = default;
 
 Float Path::ComputeLength() {
   EnsureFlattenedPath();

gfx/2d/PathHelpers.h

@@ -16,8 +16,8 @@ namespace mozilla {
 namespace gfx {
 
 struct PathOp {
-  PathOp() {}
-  ~PathOp() {}
+  PathOp() = default;
+  ~PathOp() = default;
 
   enum OpType {
     OP_MOVETO = 0,

gfx/2d/PathRecording.h

@@ -19,19 +19,16 @@ namespace gfx {
 
 class PathOps {
  public:
-  PathOps() {}
+  PathOps() = default;
 
   template <class S>
   explicit PathOps(S& aStream);
 
-  PathOps(PathOps&& aOther) : mPathData(std::move(aOther.mPathData)) {}
+  PathOps(const PathOps& aOther) = default;
+  PathOps& operator=(const PathOps&) = delete;  // assign using std::move()!
 
-  PathOps(const PathOps& aOther) : mPathData(aOther.mPathData) {}
-
-  PathOps& operator=(PathOps&& aOther) {
-    mPathData = std::move(aOther.mPathData);
-    return *this;
-  }
+  PathOps(PathOps&& aOther) = default;
+  PathOps& operator=(PathOps&& aOther) = default;
 
   template <class S>
   void Record(S& aStream) const;
@@ -67,8 +64,6 @@ class PathOps {
   }
 
  private:
-  void operator=(const PathOps&) = delete;  // assign using std::move()!
-
   enum class OpType : uint32_t {
     OP_MOVETO = 0,
     OP_LINETO,

gfx/2d/RecordedEventImpl.h

@@ -948,7 +948,7 @@ class RecordedExternalSurfaceCreation
         mRefPtr(aRefPtr),
         mKey(aKey) {}
 
-  ~RecordedExternalSurfaceCreation() {}
+  ~RecordedExternalSurfaceCreation() = default;
 
   virtual bool PlayEvent(Translator* aTranslator) const;
 
@@ -3141,7 +3141,7 @@ inline void RecordedExternalSurfaceCreation::OutputSimpleEventInfo(
                 << "] SourceSurfaceSharedData created (Key: " << mKey << ")";
 }
 
-inline RecordedFilterNodeCreation::~RecordedFilterNodeCreation() {}
+inline RecordedFilterNodeCreation::~RecordedFilterNodeCreation() = default;
 
 inline bool RecordedFilterNodeCreation::PlayEvent(
     Translator* aTranslator) const {
@@ -3444,7 +3444,7 @@ RecordedFontData::RecordedFontData(S& aStream)
   }
 }
 
-inline RecordedFontDescriptor::~RecordedFontDescriptor() {}
+inline RecordedFontDescriptor::~RecordedFontDescriptor() = default;
 
 inline bool RecordedFontDescriptor::PlayEvent(Translator* aTranslator) const {
   RefPtr<UnscaledFont> font = Factory::CreateUnscaledFontFromFontDescriptor(

gfx/2d/SFNTData.h

@@ -76,7 +76,7 @@ class SFNTData final {
                           uint32_t* aIndex, size_t aTruncatedLen = 0);
 
  private:
-  SFNTData() {}
+  SFNTData() = default;
 
   bool AddFont(const uint8_t* aFontData, uint32_t aDataLength,
                uint32_t aOffset);

gfx/2d/SourceSurfaceCapture.cpp

@@ -62,7 +62,7 @@ SourceSurfaceCapture::SourceSurfaceCapture(DrawTargetCaptureImpl* aOwner,
   mRefDT = mOwner->mRefDT;
 }
 
-SourceSurfaceCapture::~SourceSurfaceCapture() {}
+SourceSurfaceCapture::~SourceSurfaceCapture() = default;
 
 void SourceSurfaceCapture::SizeOfExcludingThis(MallocSizeOf aMallocSizeOf,
                                                SizeOfInfo& aInfo) const {

gfx/2d/SourceSurfaceRawData.h

@@ -22,7 +22,7 @@ class SourceSurfaceMappedData final : public DataSourceSurface {
                           SurfaceFormat aFormat)
       : mMap(std::move(aMap)), mSize(aSize), mFormat(aFormat) {}
 
-  ~SourceSurfaceMappedData() final {}
+  ~SourceSurfaceMappedData() final = default;
 
   uint8_t* GetData() final { return mMap.GetData(); }
   int32_t Stride() final { return mMap.GetStride(); }
@@ -104,7 +104,7 @@ class SourceSurfaceAlignedRawData : public DataSourceSurface {
   MOZ_DECLARE_REFCOUNTED_VIRTUAL_TYPENAME(DataSourceSurfaceAlignedRawData,
                                           override)
   SourceSurfaceAlignedRawData() : mStride(0), mFormat(SurfaceFormat::UNKNOWN) {}
-  ~SourceSurfaceAlignedRawData() override {}
+  ~SourceSurfaceAlignedRawData() override = default;
 
   bool Init(const IntSize& aSize, SurfaceFormat aFormat, bool aClearMem,
             uint8_t aClearValue, int32_t aStride = 0);

gfx/2d/unittest/TestBase.h

@@ -27,7 +27,7 @@
 
 class TestBase {
  public:
-  TestBase() {}
+  TestBase() = default;
 
   typedef void (TestBase::*TestCall)();
 

gfx/config/gfxVars.cpp

@@ -65,7 +65,7 @@ void gfxVars::Initialize() {
   }
 }
 
-gfxVars::gfxVars() {}
+gfxVars::gfxVars() = default;
 
 void gfxVars::Shutdown() {
   sInstance = nullptr;

gfx/gl/GLContextProviderEGL.cpp

@@ -238,8 +238,8 @@ class GLContextEGLFactory {
                                             bool aWebRender);
 
  private:
-  GLContextEGLFactory() {}
-  ~GLContextEGLFactory() {}
+  GLContextEGLFactory() = default;
+  ~GLContextEGLFactory() = default;
 };
 
 already_AddRefed<GLContext> GLContextEGLFactory::Create(

gfx/gl/GLLibraryEGL.h

@@ -45,7 +45,7 @@ void AfterEGLCall(const char* funcName);
 
 class GLLibraryEGL final {
  protected:
-  ~GLLibraryEGL() {}
+  ~GLLibraryEGL() = default;
 
  public:
   NS_INLINE_DECL_THREADSAFE_REFCOUNTING(GLLibraryEGL)

gfx/gl/GLTextureImage.cpp

@@ -220,7 +220,7 @@ TiledTextureImage::TiledTextureImage(GLContext* aGL, gfx::IntSize aSize,
   }
 }
 
-TiledTextureImage::~TiledTextureImage() {}
+TiledTextureImage::~TiledTextureImage() = default;
 
 bool TiledTextureImage::DirectUpdate(
     gfx::DataSourceSurface* aSurf, const nsIntRegion& aRegion,

gfx/gl/GfxTexturesReporter.h

@@ -15,7 +15,7 @@ namespace mozilla {
 namespace gl {
 
 class GfxTexturesReporter final : public nsIMemoryReporter {
-  ~GfxTexturesReporter() {}
+  ~GfxTexturesReporter() = default;
 
  public:
   NS_DECL_ISUPPORTS

gfx/ipc/CompositorSession.cpp

@@ -30,7 +30,7 @@ CompositorSession::CompositorSession(nsBaseWidget* aWidget,
       mCompositorBridgeChild(aChild),
       mRootLayerTreeId(aRootLayerTreeId) {}
 
-CompositorSession::~CompositorSession() {}
+CompositorSession::~CompositorSession() = default;
 
 CompositorBridgeChild* CompositorSession::GetCompositorBridgeChild() {
   return mCompositorBridgeChild;

gfx/ipc/CrossProcessPaint.cpp

@@ -220,7 +220,7 @@ CrossProcessPaint::CrossProcessPaint(dom::Promise* aPromise, float aScale,
                                      dom::WindowGlobalParent* aRoot)
     : mPromise{aPromise}, mRoot{aRoot}, mScale{aScale}, mPendingFragments{0} {}
 
-CrossProcessPaint::~CrossProcessPaint() {}
+CrossProcessPaint::~CrossProcessPaint() = default;
 
 static dom::TabId GetTabId(dom::WindowGlobalParent* aWGP) {
   // There is no unique TabId for a given WindowGlobalParent, as multiple

gfx/ipc/GPUProcessImpl.cpp

@@ -22,7 +22,7 @@ using namespace ipc;
 GPUProcessImpl::GPUProcessImpl(ProcessId aParentPid)
     : ProcessChild(aParentPid) {}
 
-GPUProcessImpl::~GPUProcessImpl() {}
+GPUProcessImpl::~GPUProcessImpl() = default;
 
 bool GPUProcessImpl::Init(int aArgc, char* aArgv[]) {
 #if defined(MOZ_SANDBOX) && defined(OS_WIN)

gfx/ipc/VsyncBridgeChild.cpp

@@ -14,7 +14,7 @@ VsyncBridgeChild::VsyncBridgeChild(RefPtr<VsyncIOThreadHolder> aThread,
                                    const uint64_t& aProcessToken)
     : mThread(aThread), mProcessToken(aProcessToken) {}
 
-VsyncBridgeChild::~VsyncBridgeChild() {}
+VsyncBridgeChild::~VsyncBridgeChild() = default;
 
 /* static */
 RefPtr<VsyncBridgeChild> VsyncBridgeChild::Create(

gfx/layers/AnimationInfo.cpp

@@ -18,7 +18,7 @@ namespace layers {
 
 AnimationInfo::AnimationInfo() : mCompositorAnimationsId(0), mMutated(false) {}
 
-AnimationInfo::~AnimationInfo() {}
+AnimationInfo::~AnimationInfo() = default;
 
 void AnimationInfo::EnsureAnimationsId() {
   if (!mCompositorAnimationsId) {

gfx/layers/AxisPhysicsMSDModel.cpp

@@ -38,7 +38,7 @@ AxisPhysicsMSDModel::AxisPhysicsMSDModel(double aInitialPosition,
       mSpringConstantSqrtXTwo(sqrt(mSpringConstant) * 2.0),
       mDampingRatio(aDampingRatio) {}
 
-AxisPhysicsMSDModel::~AxisPhysicsMSDModel() {}
+AxisPhysicsMSDModel::~AxisPhysicsMSDModel() = default;
 
 double AxisPhysicsMSDModel::Acceleration(const State& aState) {
   // Simulate a Mass-Damper-Spring Model; assume a unit mass

gfx/layers/AxisPhysicsModel.cpp

@@ -38,7 +38,7 @@ AxisPhysicsModel::AxisPhysicsModel(double aInitialPosition,
       mPrevState(aInitialPosition, aInitialVelocity),
       mNextState(aInitialPosition, aInitialVelocity) {}
 
-AxisPhysicsModel::~AxisPhysicsModel() {}
+AxisPhysicsModel::~AxisPhysicsModel() = default;
 
 double AxisPhysicsModel::GetVelocity() const {
   return LinearInterpolate(mPrevState.v, mNextState.v, mProgress);

gfx/layers/GPUVideoImage.h

@@ -21,7 +21,7 @@ namespace layers {
 
 class IGPUVideoSurfaceManager {
  protected:
-  virtual ~IGPUVideoSurfaceManager(){};
+  virtual ~IGPUVideoSurfaceManager() = default;
 
  public:
   NS_INLINE_DECL_PURE_VIRTUAL_REFCOUNTING

gfx/layers/ImageContainer.cpp

@@ -102,7 +102,7 @@ ImageContainerListener::ImageContainerListener(ImageContainer* aImageContainer)
     : mLock("mozilla.layers.ImageContainerListener.mLock"),
       mImageContainer(aImageContainer) {}
 
-ImageContainerListener::~ImageContainerListener() {}
+ImageContainerListener::~ImageContainerListener() = default;
 
 void ImageContainerListener::NotifyComposite(
     const ImageCompositeNotification& aNotification) {

gfx/layers/ImageContainer.h

@@ -157,7 +157,7 @@ struct ImageBackendData {
   virtual ~ImageBackendData() = default;
 
  protected:
-  ImageBackendData() {}
+  ImageBackendData() = default;
 };
 
 /* Forward declarations for Image derivatives. */
@@ -280,7 +280,7 @@ class BufferRecycleBin final {
   typedef mozilla::Mutex Mutex;
 
   // Private destructor, to discourage deletion outside of Release():
-  ~BufferRecycleBin() {}
+  ~BufferRecycleBin() = default;
 
   // This protects mRecycledBuffers, mRecycledBufferSize, mRecycledTextures
   // and mRecycledTextureSizes
@@ -314,7 +314,7 @@ class ImageFactory {
  protected:
   friend class ImageContainer;
 
-  ImageFactory() {}
+  ImageFactory() = default;
   virtual ~ImageFactory() = default;
 
   virtual RefPtr<PlanarYCbCrImage> CreatePlanarYCbCrImage(

gfx/layers/ImageLayers.cpp

@@ -19,7 +19,7 @@ ImageLayer::ImageLayer(LayerManager* aManager, void* aImplData)
       mSamplingFilter(gfx::SamplingFilter::GOOD),
       mScaleMode(ScaleMode::SCALE_NONE) {}
 
-ImageLayer::~ImageLayer() {}
+ImageLayer::~ImageLayer() = default;
 
 void ImageLayer::SetContainer(ImageContainer* aContainer) {
   mContainer = aContainer;

gfx/layers/LayerScope.cpp

@@ -133,7 +133,7 @@ class LayerScopeWebSocketManager {
    public:
     NS_DECL_THREADSAFE_ISUPPORTS
 
-    SocketListener() {}
+    SocketListener() = default;
 
     /* nsIServerSocketListener */
     NS_IMETHOD OnSocketAccepted(nsIServerSocket* aServ,

gfx/layers/Layers.cpp

@@ -185,7 +185,7 @@ Layer::Layer(LayerManager* aManager, void* aImplData)
 {
 }
 
-Layer::~Layer() {}
+Layer::~Layer() = default;
 
 void Layer::SetCompositorAnimations(
     const CompositorAnimations& aCompositorAnimations) {
@@ -770,7 +770,7 @@ ContainerLayer::ContainerLayer(LayerManager* aManager, void* aImplData)
       mMayHaveReadbackChild(false),
       mChildrenChanged(false) {}
 
-ContainerLayer::~ContainerLayer() {}
+ContainerLayer::~ContainerLayer() = default;
 
 bool ContainerLayer::InsertAfter(Layer* aChild, Layer* aAfter) {
   if (aChild->Manager() != Manager()) {

gfx/layers/LayersTypes.h

@@ -381,7 +381,7 @@ class LayerHandle final {
 
  public:
   LayerHandle() : mHandle(0) {}
-  LayerHandle(const LayerHandle& aOther) : mHandle(aOther.mHandle) {}
+  LayerHandle(const LayerHandle& aOther) = default;
   explicit LayerHandle(uint64_t aHandle) : mHandle(aHandle) {}
   bool IsValid() const { return mHandle != 0; }
   explicit operator bool() const { return IsValid(); }
@@ -403,8 +403,7 @@ class CompositableHandle final {
 
  public:
   CompositableHandle() : mHandle(0) {}
-  CompositableHandle(const CompositableHandle& aOther)
-      : mHandle(aOther.mHandle) {}
+  CompositableHandle(const CompositableHandle& aOther) = default;
   explicit CompositableHandle(uint64_t aHandle) : mHandle(aHandle) {}
   bool IsValid() const { return mHandle != 0; }
   explicit operator bool() const { return IsValid(); }

gfx/layers/NativeLayer.h

@@ -59,7 +59,7 @@ class NativeLayerRoot {
   }
 
  protected:
-  virtual ~NativeLayerRoot() {}
+  virtual ~NativeLayerRoot() = default;
 };
 
 // Allows reading back the visual output of a NativeLayerRoot.
@@ -69,7 +69,7 @@ class NativeLayerRoot {
 // same thread.
 class NativeLayerRootSnapshotter {
  public:
-  virtual ~NativeLayerRootSnapshotter() {}
+  virtual ~NativeLayerRootSnapshotter() = default;
 
   // Reads the composited result of the NativeLayer tree into aReadbackBuffer,
   // synchronously. Should only be called right after a call to CommitToScreen()
@@ -185,7 +185,7 @@ class NativeLayer {
   virtual void DiscardBackbuffers() = 0;
 
  protected:
-  virtual ~NativeLayer() {}
+  virtual ~NativeLayer() = default;
 };
 
 }  // namespace layers

gfx/layers/OOPCanvasRenderer.h

@@ -44,7 +44,7 @@ class OOPCanvasRenderer final {
   CanvasClient* mCanvasClient = nullptr;
 
  private:
-  ~OOPCanvasRenderer() {}
+  ~OOPCanvasRenderer() = default;
 };
 
 }  // namespace layers

gfx/layers/PaintThread.cpp

@@ -42,7 +42,7 @@ StaticAutoPtr<PaintThread> PaintThread::sSingleton;
 StaticRefPtr<nsIThread> PaintThread::sThread;
 PlatformThreadId PaintThread::sThreadId;
 
-PaintThread::PaintThread() {}
+PaintThread::PaintThread() = default;
 
 void PaintThread::Release() {}
 

gfx/layers/PaintThread.h

@@ -35,8 +35,8 @@ namespace layers {
 //    task's duration, and then destroyed on the main thread
 class PaintTask {
  public:
-  PaintTask() {}
-  ~PaintTask() {}
+  PaintTask() = default;
+  ~PaintTask() = default;
 
   void DropTextureClients();
 

gfx/layers/ProfilerScreenshots.cpp

@@ -23,7 +23,7 @@ using namespace mozilla::layers;
 ProfilerScreenshots::ProfilerScreenshots()
     : mMutex("ProfilerScreenshots::mMutex"), mLiveSurfaceCount(0) {}
 
-ProfilerScreenshots::~ProfilerScreenshots() {}
+ProfilerScreenshots::~ProfilerScreenshots() = default;
 
 /* static */
 bool ProfilerScreenshots::IsEnabled() {

gfx/layers/ReadbackLayer.h

@@ -35,7 +35,7 @@ class LayersPacket;
  */
 class ReadbackSink {
  public:
-  ReadbackSink() {}
+  ReadbackSink() = default;
   virtual ~ReadbackSink() = default;
 
   /**

gfx/layers/ScrollableLayerGuid.h

@@ -45,12 +45,9 @@ struct ScrollableLayerGuid {
         mPresShellId(aPresShellId),
         mScrollId(aScrollId) {}
 
-  ScrollableLayerGuid(const ScrollableLayerGuid& other)
-      : mLayersId(other.mLayersId),
-        mPresShellId(other.mPresShellId),
-        mScrollId(other.mScrollId) {}
+  ScrollableLayerGuid(const ScrollableLayerGuid& other) = default;
 
-  ~ScrollableLayerGuid() {}
+  ~ScrollableLayerGuid() = default;
 
   bool operator==(const ScrollableLayerGuid& other) const {
     return mLayersId == other.mLayersId && mPresShellId == other.mPresShellId &&

gfx/layers/SurfacePool.h

@@ -40,7 +40,7 @@ class SurfacePool {
   virtual void DestroyGLResourcesForContext(gl::GLContext* aGL) = 0;
 
  protected:
-  virtual ~SurfacePool() {}
+  virtual ~SurfacePool() = default;
 };
 
 class SurfacePoolHandleCA;
@@ -63,7 +63,7 @@ class SurfacePoolHandle {
   virtual void OnEndFrame() = 0;
 
  protected:
-  virtual ~SurfacePoolHandle() {}
+  virtual ~SurfacePoolHandle() = default;
 };
 
 }  // namespace layers

gfx/layers/SyncObject.h

@@ -39,7 +39,7 @@ class SyncObjectHost : public RefCounted<SyncObjectHost> {
   virtual bool Synchronize(bool aFallible = false) = 0;
 
  protected:
-  SyncObjectHost() {}
+  SyncObjectHost() = default;
 };
 
 class SyncObjectClient : public external::AtomicRefCounted<SyncObjectClient> {
@@ -67,7 +67,7 @@ class SyncObjectClient : public external::AtomicRefCounted<SyncObjectClient> {
   virtual bool IsSyncObjectValid() = 0;
 
  protected:
-  SyncObjectClient() {}
+  SyncObjectClient() = default;
 };
 
 }  // namespace layers

gfx/layers/TextureWrapperImage.cpp

@@ -16,7 +16,7 @@ TextureWrapperImage::TextureWrapperImage(TextureClient* aClient,
       mPictureRect(aPictureRect),
       mTextureClient(aClient) {}
 
-TextureWrapperImage::~TextureWrapperImage() {}
+TextureWrapperImage::~TextureWrapperImage() = default;
 
 gfx::IntSize TextureWrapperImage::GetSize() const {
   return mTextureClient->GetSize();

gfx/layers/TiledLayerBuffer.h

@@ -137,7 +137,7 @@ class TiledLayerBuffer {
         mResolution(1),
         mTileSize(mozilla::gfx::gfxVars::TileSize()) {}
 
-  ~TiledLayerBuffer() {}
+  ~TiledLayerBuffer() = default;
 
   gfx::IntPoint GetTileOffset(TileCoordIntPoint aPosition) const {
     gfx::IntSize scaledTileSize = GetScaledTileSize();

gfx/layers/apz/public/GeckoContentController.h

@@ -195,7 +195,7 @@ class GeckoContentController {
   virtual void UpdateOverscrollOffset(float aX, float aY, bool aIsRootContent) {
   }
 
-  GeckoContentController() {}
+  GeckoContentController() = default;
 
   /**
    * Needs to be called on the main thread.

gfx/layers/apz/public/MatrixMessage.h

@@ -17,7 +17,7 @@ namespace layers {
 class MatrixMessage {
  public:
   // Don't use this one directly
-  MatrixMessage() {}
+  MatrixMessage() = default;
 
   MatrixMessage(const Maybe<LayerToScreenMatrix4x4>& aMatrix,
                 const ScreenRect& aTopLevelViewportVisibleRectInBrowserCoords,

gfx/layers/apz/util/APZCCallbackHelper.cpp

@@ -783,7 +783,7 @@ DisplayportSetListener::DisplayportSetListener(
       mInputBlockId(aInputBlockId),
       mTargets(aTargets) {}
 
-DisplayportSetListener::~DisplayportSetListener() {}
+DisplayportSetListener::~DisplayportSetListener() = default;
 
 bool DisplayportSetListener::Register() {
   if (mPresShell->AddPostRefreshObserver(this)) {

gfx/layers/apz/util/APZEventState.cpp

@@ -110,7 +110,7 @@ APZEventState::APZEventState(nsIWidget* aWidget,
              " does not support weak references. APZ will NOT work!");
 }
 
-APZEventState::~APZEventState() {}
+APZEventState::~APZEventState() = default;
 
 class DelayedFireSingleTapEvent final : public nsITimerCallback,
                                         public nsINamed {
@@ -148,7 +148,7 @@ class DelayedFireSingleTapEvent final : public nsITimerCallback,
   void ClearTimer() { mTimer = nullptr; }
 
  private:
-  ~DelayedFireSingleTapEvent() {}
+  ~DelayedFireSingleTapEvent() = default;
 
   nsWeakPtr mWidget;
   LayoutDevicePoint mPoint;

gfx/layers/apz/util/ActiveElementManager.cpp

@@ -23,7 +23,7 @@ namespace layers {
 ActiveElementManager::ActiveElementManager()
     : mCanBePan(false), mCanBePanSet(false), mSetActiveTask(nullptr) {}
 
-ActiveElementManager::~ActiveElementManager() {}
+ActiveElementManager::~ActiveElementManager() = default;
 
 void ActiveElementManager::SetTargetElement(dom::EventTarget* aTarget) {
   if (mTarget) {

gfx/layers/apz/util/CheckerboardReportService.h

@@ -50,7 +50,7 @@ class CheckerboardEventStorage {
 
  private:
   /* Stuff for refcounted singleton */
-  CheckerboardEventStorage() {}
+  CheckerboardEventStorage() = default;
   virtual ~CheckerboardEventStorage() = default;
 
   static StaticRefPtr<CheckerboardEventStorage> sInstance;

gfx/layers/apz/util/ChromeProcessController.cpp

@@ -44,7 +44,7 @@ ChromeProcessController::ChromeProcessController(
                         &ChromeProcessController::InitializeRoot));
 }
 
-ChromeProcessController::~ChromeProcessController() {}
+ChromeProcessController::~ChromeProcessController() = default;
 
 void ChromeProcessController::InitializeRoot() {
   APZCCallbackHelper::InitializeRootDisplayport(GetPresShell());

gfx/layers/basic/AutoMaskData.h

@@ -19,8 +19,8 @@ namespace layers {
  */
 class MOZ_STACK_CLASS AutoMoz2DMaskData {
  public:
-  AutoMoz2DMaskData() {}
-  ~AutoMoz2DMaskData() {}
+  AutoMoz2DMaskData() = default;
+  ~AutoMoz2DMaskData() = default;
 
   void Construct(const gfx::Matrix& aTransform, gfx::SourceSurface* aSurface) {
     MOZ_ASSERT(!IsConstructed());

gfx/layers/basic/BasicImages.cpp

@@ -78,7 +78,7 @@ class BasicPlanarYCbCrImage : public RecyclingPlanarYCbCrImage {
 
 class BasicImageFactory : public ImageFactory {
  public:
-  BasicImageFactory() {}
+  BasicImageFactory() = default;
 
   virtual RefPtr<PlanarYCbCrImage> CreatePlanarYCbCrImage(
       const gfx::IntSize& aScaleHint, BufferRecycleBin* aRecycleBin) override {

gfx/layers/basic/X11BasicCompositor.h

@@ -20,7 +20,7 @@ namespace layers {
 class X11DataTextureSourceBasic : public DataTextureSource,
                                   public TextureSourceBasic {
  public:
-  X11DataTextureSourceBasic(){};
+  X11DataTextureSourceBasic() = default;
 
   const char* Name() const override { return "X11DataTextureSourceBasic"; }
 

gfx/layers/client/CanvasClient.cpp

@@ -528,7 +528,7 @@ CanvasClientOOP::CanvasClientOOP(CompositableForwarder* aLayerForwarder,
                                  TextureFlags aFlags)
     : CanvasClient(aLayerForwarder, aFlags) {}
 
-CanvasClientOOP::~CanvasClientOOP() {}
+CanvasClientOOP::~CanvasClientOOP() = default;
 
 void CanvasClientOOP::SetLayer(ShadowableLayer* aLayer,
                                OOPCanvasRenderer* aRenderer) {

gfx/layers/client/GPUVideoTextureClient.cpp

@@ -18,7 +18,7 @@ GPUVideoTextureData::GPUVideoTextureData(IGPUVideoSurfaceManager* aManager,
                                          const gfx::IntSize& aSize)
     : mManager(aManager), mSD(aSD), mSize(aSize) {}
 
-GPUVideoTextureData::~GPUVideoTextureData() {}
+GPUVideoTextureData::~GPUVideoTextureData() = default;
 
 bool GPUVideoTextureData::Serialize(SurfaceDescriptor& aOutDescriptor) {
   aOutDescriptor = mSD;

gfx/layers/client/TextureClientSharedSurface.cpp

@@ -23,7 +23,7 @@ SharedSurfaceTextureData::SharedSurfaceTextureData(
     UniquePtr<gl::SharedSurface> surf)
     : mSurf(std::move(surf)) {}
 
-SharedSurfaceTextureData::~SharedSurfaceTextureData() {}
+SharedSurfaceTextureData::~SharedSurfaceTextureData() = default;
 
 void SharedSurfaceTextureData::Deallocate(LayersIPCChannel*) {}
 

gfx/layers/client/TextureRecorded.cpp

@@ -18,7 +18,7 @@ RecordedTextureData::RecordedTextureData(
   mCanvasChild->EnsureRecorder(aTextureType);
 }
 
-RecordedTextureData::~RecordedTextureData() {}
+RecordedTextureData::~RecordedTextureData() = default;
 
 void RecordedTextureData::FillInfo(TextureData::Info& aInfo) const {
   aInfo.size = mSize;

gfx/layers/composite/AsyncCompositionManager.cpp

@@ -74,7 +74,7 @@ AsyncCompositionManager::AsyncCompositionManager(
   MOZ_ASSERT(mCompositorBridge);
 }
 
-AsyncCompositionManager::~AsyncCompositionManager() {}
+AsyncCompositionManager::~AsyncCompositionManager() = default;
 
 void AsyncCompositionManager::ResolveRefLayers(
     CompositorBridgeParent* aCompositor, bool* aHasRemoteContent,

gfx/layers/composite/CompositorScreenshotGrabber.cpp

@@ -59,9 +59,9 @@ class CompositorScreenshotGrabberImpl final {
   const IntSize mBufferSize;
 };
 
-CompositorScreenshotGrabber::CompositorScreenshotGrabber() {}
+CompositorScreenshotGrabber::CompositorScreenshotGrabber() = default;
 
-CompositorScreenshotGrabber::~CompositorScreenshotGrabber() {}
+CompositorScreenshotGrabber::~CompositorScreenshotGrabber() = default;
 
 void CompositorScreenshotGrabber::MaybeGrabScreenshot(Compositor* aCompositor) {
   if (ProfilerScreenshots::IsEnabled()) {

gfx/layers/composite/ContentHost.cpp

@@ -26,7 +26,7 @@ namespace layers {
 ContentHostBase::ContentHostBase(const TextureInfo& aTextureInfo)
     : ContentHost(aTextureInfo), mInitialised(false) {}
 
-ContentHostBase::~ContentHostBase() {}
+ContentHostBase::~ContentHostBase() = default;
 
 void ContentHostTexture::Composite(
     Compositor* aCompositor, LayerComposite* aLayer, EffectChain& aEffectChain,

gfx/layers/composite/FPSCounter.cpp

@@ -32,7 +32,7 @@ FPSCounter::FPSCounter(const char* aName)
   Init();
 }
 
-FPSCounter::~FPSCounter() {}
+FPSCounter::~FPSCounter() = default;
 
 void FPSCounter::Init() {
   for (int i = 0; i < kMaxFrames; i++) {

gfx/layers/composite/FrameUniformityData.h

@@ -25,7 +25,7 @@ class FrameUniformityData {
 };
 
 struct LayerTransforms {
-  LayerTransforms() {}
+  LayerTransforms() = default;
 
   gfx::Point GetAverage();
   gfx::Point GetStdDev();
@@ -36,7 +36,7 @@ struct LayerTransforms {
 
 class LayerTransformRecorder {
  public:
-  LayerTransformRecorder() {}
+  LayerTransformRecorder() = default;
   ~LayerTransformRecorder();
 
   void RecordTransform(Layer* aLayer, const gfx::Point& aTransform);

gfx/layers/composite/ImageComposite.cpp

@@ -23,7 +23,7 @@ ImageComposite::ImageComposite()
       mDroppedFrames(0),
       mLastChosenImageIndex(0) {}
 
-ImageComposite::~ImageComposite() {}
+ImageComposite::~ImageComposite() = default;
 
 TimeStamp ImageComposite::GetBiasedTime(const TimeStamp& aInput) const {
   switch (mBias) {

gfx/layers/composite/ImageHost.cpp

@@ -30,7 +30,7 @@ class ISurfaceAllocator;
 ImageHost::ImageHost(const TextureInfo& aTextureInfo)
     : CompositableHost(aTextureInfo), ImageComposite(), mLocked(false) {}
 
-ImageHost::~ImageHost() {}
+ImageHost::~ImageHost() = default;
 
 void ImageHost::UseTextureHost(const nsTArray<TimedTexture>& aTextures) {
   MOZ_ASSERT(!mLocked);

gfx/layers/composite/LayerManagerComposite.cpp

@@ -119,7 +119,7 @@ HostLayerManager::HostLayerManager()
       mLastPaintTime(TimeDuration::Forever()),
       mRenderStartTime(TimeStamp::Now()) {}
 
-HostLayerManager::~HostLayerManager() {}
+HostLayerManager::~HostLayerManager() = default;
 
 void HostLayerManager::RecordPaintTimes(const PaintTiming& aTiming) {
   mDiagnostics->RecordPaintTimes(aTiming);
@@ -1638,7 +1638,7 @@ LayerComposite::LayerComposite(LayerManagerComposite* aManager)
       mDestroyed(false),
       mLayerComposited(false) {}
 
-LayerComposite::~LayerComposite() {}
+LayerComposite::~LayerComposite() = default;
 
 void LayerComposite::Destroy() {
   if (!mDestroyed) {

gfx/layers/composite/TextRenderer.cpp

@@ -46,7 +46,7 @@ static void PNGAPI row_callback(png_structp png_ptr, png_bytep new_row,
   }
 }
 
-TextRenderer::~TextRenderer() {}
+TextRenderer::~TextRenderer() = default;
 
 TextRenderer::FontCache::~FontCache() { mGlyphBitmaps->Unmap(); }
 

gfx/layers/composite/TextureHost.cpp

@@ -484,7 +484,7 @@ void TextureHost::Updated(const nsIntRegion* aRegion) {
 
 TextureSource::TextureSource() : mCompositableCount(0) {}
 
-TextureSource::~TextureSource() {}
+TextureSource::~TextureSource() = default;
 BufferTextureHost::BufferTextureHost(const BufferDescriptor& aDesc,
                                      TextureFlags aFlags)
     : TextureHost(aFlags),
@@ -520,7 +520,7 @@ BufferTextureHost::BufferTextureHost(const BufferDescriptor& aDesc,
   }
 }
 
-BufferTextureHost::~BufferTextureHost() {}
+BufferTextureHost::~BufferTextureHost() = default;
 
 void BufferTextureHost::UpdatedInternal(const nsIntRegion* aRegion) {
   ++mUpdateSerial;

gfx/layers/composite/TextureHost.h

@@ -232,7 +232,7 @@ class TextureSource : public RefCounted<TextureSource> {
 template <typename T>
 class CompositableTextureRef {
  public:
-  CompositableTextureRef() {}
+  CompositableTextureRef() = default;
 
   explicit CompositableTextureRef(const CompositableTextureRef& aOther) {
     *this = aOther;

gfx/layers/composite/TiledContentHost.h

@@ -46,7 +46,7 @@ class TileHost {
   // TiledLayerBuffer for more information on what this is used for;
   // essentially, this is a sentinel used to represent an invalid or blank
   // tile.
-  TileHost() {}
+  TileHost() = default;
 
   // Constructs a TileHost from a TextureReadLock and TextureHost.
   TileHost(TextureReadLock* aSharedLock, TextureHost* aTextureHost,

gfx/layers/ipc/APZCTreeManagerChild.cpp

@@ -18,7 +18,7 @@ namespace layers {
 APZCTreeManagerChild::APZCTreeManagerChild()
     : mCompositorSession(nullptr), mIPCOpen(false) {}
 
-APZCTreeManagerChild::~APZCTreeManagerChild() {}
+APZCTreeManagerChild::~APZCTreeManagerChild() = default;
 
 void APZCTreeManagerChild::SetCompositorSession(
     RemoteCompositorSession* aSession) {

gfx/layers/ipc/APZCTreeManagerParent.cpp

@@ -24,7 +24,7 @@ APZCTreeManagerParent::APZCTreeManagerParent(
   MOZ_ASSERT(mUpdater->HasTreeManager(mTreeManager));
 }
 
-APZCTreeManagerParent::~APZCTreeManagerParent() {}
+APZCTreeManagerParent::~APZCTreeManagerParent() = default;
 
 void APZCTreeManagerParent::ChildAdopted(
     RefPtr<APZCTreeManager> aAPZCTreeManager, RefPtr<APZUpdater> aAPZUpdater) {

gfx/layers/ipc/APZInputBridgeChild.cpp

@@ -16,7 +16,7 @@ APZInputBridgeChild::APZInputBridgeChild() : mDestroyed(false) {
   MOZ_ASSERT(NS_IsMainThread());
 }
 
-APZInputBridgeChild::~APZInputBridgeChild() {}
+APZInputBridgeChild::~APZInputBridgeChild() = default;
 
 void APZInputBridgeChild::Destroy() {
   MOZ_ASSERT(XRE_IsParentProcess());

gfx/layers/ipc/APZInputBridgeParent.cpp

@@ -21,7 +21,7 @@ APZInputBridgeParent::APZInputBridgeParent(const LayersId& aLayersId) {
   MOZ_ASSERT(mTreeManager);
 }
 
-APZInputBridgeParent::~APZInputBridgeParent() {}
+APZInputBridgeParent::~APZInputBridgeParent() = default;
 
 mozilla::ipc::IPCResult APZInputBridgeParent::RecvReceiveMultiTouchInputEvent(
     const MultiTouchInput& aEvent, APZEventResult* aOutResult,

gfx/layers/ipc/CanvasChild.cpp

@@ -111,7 +111,7 @@ CanvasChild::CanvasChild(Endpoint<PCanvasChild>&& aEndpoint) {
   aEndpoint.Bind(this);
 }
 
-CanvasChild::~CanvasChild() {}
+CanvasChild::~CanvasChild() = default;
 
 ipc::IPCResult CanvasChild::RecvNotifyDeviceChanged() {
   nsCOMPtr<nsIObserverService> obs = mozilla::services::GetObserverService();

gfx/layers/ipc/CompositableTransactionParent.h

@@ -24,7 +24,7 @@ namespace layers {
 // through this interface.
 class CompositableParentManager : public HostIPCAllocator {
  public:
-  CompositableParentManager() {}
+  CompositableParentManager() = default;
 
   void DestroyActor(const OpDestroy& aOp);
 

gfx/layers/ipc/CompositorBridgeParent.cpp

@@ -134,7 +134,7 @@ CompositorBridgeParentBase::CompositorBridgeParentBase(
     CompositorManagerParent* aManager)
     : mCanSend(true), mCompositorManager(aManager) {}
 
-CompositorBridgeParentBase::~CompositorBridgeParentBase() {}
+CompositorBridgeParentBase::~CompositorBridgeParentBase() = default;
 
 ProcessId CompositorBridgeParentBase::GetChildProcessId() { return OtherPid(); }
 

gfx/layers/ipc/CompositorManagerParent.cpp

@@ -116,7 +116,7 @@ CompositorManagerParent::CreateSameProcessWidgetCompositorBridge(
 CompositorManagerParent::CompositorManagerParent()
     : mCompositorThreadHolder(CompositorThreadHolder::GetSingleton()) {}
 
-CompositorManagerParent::~CompositorManagerParent() {}
+CompositorManagerParent::~CompositorManagerParent() = default;
 
 void CompositorManagerParent::Bind(
     Endpoint<PCompositorManagerParent>&& aEndpoint, bool aIsRoot) {

gfx/layers/ipc/ISurfaceAllocator.h

@@ -69,7 +69,7 @@ class ISurfaceAllocator {
   MOZ_DECLARE_REFCOUNTED_TYPENAME(ISurfaceAllocator)
   NS_INLINE_DECL_THREADSAFE_REFCOUNTING(ISurfaceAllocator)
 
-  ISurfaceAllocator() {}
+  ISurfaceAllocator() = default;
 
   // down-casting
 
@@ -113,7 +113,7 @@ class ISurfaceAllocator {
 /// Methods that are specific to the client/child side.
 class ClientIPCAllocator : public ISurfaceAllocator {
  public:
-  ClientIPCAllocator() {}
+  ClientIPCAllocator() = default;
 
   ClientIPCAllocator* AsClientAllocator() override { return this; }
 
@@ -127,7 +127,7 @@ class ClientIPCAllocator : public ISurfaceAllocator {
 /// Methods that are specific to the host/parent side.
 class HostIPCAllocator : public ISurfaceAllocator {
  public:
-  HostIPCAllocator() {}
+  HostIPCAllocator() = default;
 
   HostIPCAllocator* AsHostIPCAllocator() override { return this; }
 
@@ -200,7 +200,7 @@ void DestroySurfaceDescriptor(mozilla::ipc::IShmemAllocator* aAllocator,
                               SurfaceDescriptor* aSurface);
 
 class GfxMemoryImageReporter final : public nsIMemoryReporter {
-  ~GfxMemoryImageReporter() {}
+  ~GfxMemoryImageReporter() = default;
 
  public:
   NS_DECL_ISUPPORTS

gfx/layers/ipc/ImageBridgeParent.cpp

@@ -71,7 +71,7 @@ ImageBridgeParent::ImageBridgeParent(MessageLoop* aLoop,
   SetOtherProcessId(aChildProcessId);
 }
 
-ImageBridgeParent::~ImageBridgeParent() {}
+ImageBridgeParent::~ImageBridgeParent() = default;
 
 /* static */
 ImageBridgeParent* ImageBridgeParent::CreateSameProcess() {

gfx/layers/ipc/LayerTransactionParent.cpp

@@ -65,7 +65,7 @@ LayerTransactionParent::LayerTransactionParent(
   MOZ_ASSERT(mId.IsValid());
 }
 
-LayerTransactionParent::~LayerTransactionParent() {}
+LayerTransactionParent::~LayerTransactionParent() = default;
 
 void LayerTransactionParent::SetLayerManager(
     HostLayerManager* aLayerManager, CompositorAnimationStorage* aAnimStorage) {

gfx/layers/ipc/RemoteContentController.cpp

@@ -31,7 +31,7 @@ using namespace mozilla::gfx;
 RemoteContentController::RemoteContentController()
     : mCompositorThread(MessageLoop::current()), mCanSend(true) {}
 
-RemoteContentController::~RemoteContentController() {}
+RemoteContentController::~RemoteContentController() = default;
 
 void RemoteContentController::NotifyLayerTransforms(
     const nsTArray<MatrixMessage>& aTransforms) {

gfx/layers/ipc/ShadowLayerUtilsX11.h

@@ -27,7 +27,7 @@ namespace mozilla {
 namespace layers {
 
 struct SurfaceDescriptorX11 {
-  SurfaceDescriptorX11() {}
+  SurfaceDescriptorX11() = default;
 
   explicit SurfaceDescriptorX11(gfxXlibSurface* aSurf,
                                 bool aForwardGLX = false);

gfx/layers/ipc/ShadowLayers.cpp

@@ -152,7 +152,7 @@ void KnowsCompositor::IdentifyTextureHost(
 
 KnowsCompositor::KnowsCompositor() : mSerial(++sSerialCounter) {}
 
-KnowsCompositor::~KnowsCompositor() {}
+KnowsCompositor::~KnowsCompositor() = default;
 
 KnowsCompositorMediaProxy::KnowsCompositorMediaProxy(
     const TextureFactoryIdentifier& aIdentifier) {
@@ -163,7 +163,7 @@ KnowsCompositorMediaProxy::KnowsCompositorMediaProxy(
   mSyncObject = mThreadSafeAllocator->GetSyncObject();
 }
 
-KnowsCompositorMediaProxy::~KnowsCompositorMediaProxy() {}
+KnowsCompositorMediaProxy::~KnowsCompositorMediaProxy() = default;
 
 TextureForwarder* KnowsCompositorMediaProxy::GetTextureForwarder() {
   return mThreadSafeAllocator->GetTextureForwarder();

gfx/layers/ipc/ShadowLayers.h

@@ -470,7 +470,7 @@ class ShadowableLayer {
   virtual CompositableClient* GetCompositableClient() { return nullptr; }
 
  protected:
-  ShadowableLayer() {}
+  ShadowableLayer() = default;
 
  private:
   RefPtr<ShadowLayerForwarder> mForwarder;

gfx/layers/ipc/SharedSurfacesChild.cpp

@@ -42,7 +42,7 @@ SharedSurfacesChild::ImageKeyData& SharedSurfacesChild::ImageKeyData::operator=(
   return *this;
 }
 
-SharedSurfacesChild::ImageKeyData::~ImageKeyData() {}
+SharedSurfacesChild::ImageKeyData::~ImageKeyData() = default;
 
 void SharedSurfacesChild::ImageKeyData::MergeDirtyRect(
     const Maybe<IntRect>& aDirtyRect) {

gfx/layers/ipc/SharedSurfacesParent.cpp

@@ -21,7 +21,7 @@ using namespace mozilla::gfx;
 StaticMutex SharedSurfacesParent::sMutex;
 StaticAutoPtr<SharedSurfacesParent> SharedSurfacesParent::sInstance;
 
-SharedSurfacesParent::SharedSurfacesParent() {}
+SharedSurfacesParent::SharedSurfacesParent() = default;
 
 SharedSurfacesParent::~SharedSurfacesParent() {
   for (auto i = mSurfaces.Iter(); !i.Done(); i.Next()) {

gfx/layers/ipc/UiCompositorControllerChild.cpp

@@ -281,7 +281,7 @@ UiCompositorControllerChild::UiCompositorControllerChild(
     const uint64_t& aProcessToken)
     : mIsOpen(false), mProcessToken(aProcessToken), mWidget(nullptr) {}
 
-UiCompositorControllerChild::~UiCompositorControllerChild() {}
+UiCompositorControllerChild::~UiCompositorControllerChild() = default;
 
 void UiCompositorControllerChild::OpenForSameProcess() {
   MOZ_ASSERT(IsOnUiThread());

gfx/layers/ipc/VideoBridgeChild.cpp

@@ -50,7 +50,7 @@ VideoBridgeChild::VideoBridgeChild()
       mMessageLoop(MessageLoop::current()),
       mCanSend(true) {}
 
-VideoBridgeChild::~VideoBridgeChild() {}
+VideoBridgeChild::~VideoBridgeChild() = default;
 
 VideoBridgeChild* VideoBridgeChild::GetSingleton() { return sVideoBridge; }
 

gfx/layers/mlgpu/BufferCache.cpp

@@ -30,7 +30,7 @@ BufferCache::BufferCache(MLGDevice* aDevice)
   mCaches.resize(lastSizeClass - mFirstSizeClass + 1);
 }
 
-BufferCache::~BufferCache() {}
+BufferCache::~BufferCache() = default;
 
 RefPtr<MLGBuffer> BufferCache::GetOrCreateBuffer(size_t aBytes) {
   size_t sizeClass = CeilingLog2(aBytes);

gfx/layers/mlgpu/BufferCache.h

@@ -51,11 +51,10 @@ class BufferCache final {
   // while drawing, it shouldn't be a big deal.
   struct CacheEntry {
     CacheEntry() : mLastUsedFrame(0) {}
-    CacheEntry(const CacheEntry& aEntry)
-        : mLastUsedFrame(aEntry.mLastUsedFrame), mBuffer(aEntry.mBuffer) {}
-    CacheEntry(CacheEntry&& aEntry)
-        : mLastUsedFrame(aEntry.mLastUsedFrame),
-          mBuffer(std::move(aEntry.mBuffer)) {}
+    // XXX The copy constructor can be deleted once RefPtr's move constructor is
+    // declared noexcept, see Bug 1612680.
+    CacheEntry(const CacheEntry& aEntry) = default;
+    CacheEntry(CacheEntry&& aEntry) = default;
     CacheEntry(size_t aLastUsedFrame, MLGBuffer* aBuffer)
         : mLastUsedFrame(aLastUsedFrame), mBuffer(aBuffer) {}
 

gfx/layers/mlgpu/FrameBuilder.cpp

@@ -33,7 +33,7 @@ FrameBuilder::FrameBuilder(LayerManagerMLGPU* aManager,
   mRoot = mManager->GetRoot()->AsHostLayer()->AsLayerMLGPU();
 }
 
-FrameBuilder::~FrameBuilder() {}
+FrameBuilder::~FrameBuilder() = default;
 
 bool FrameBuilder::Build() {
   AUTO_PROFILER_LABEL("FrameBuilder::Build", GRAPHICS);

gfx/layers/mlgpu/LayerMLGPU.cpp

@@ -126,13 +126,13 @@ void LayerMLGPU::SetLayerManager(HostLayerManager* aManager) {
 RefLayerMLGPU::RefLayerMLGPU(LayerManagerMLGPU* aManager)
     : RefLayer(aManager, static_cast<HostLayer*>(this)), LayerMLGPU(aManager) {}
 
-RefLayerMLGPU::~RefLayerMLGPU() {}
+RefLayerMLGPU::~RefLayerMLGPU() = default;
 
 ColorLayerMLGPU::ColorLayerMLGPU(LayerManagerMLGPU* aManager)
     : ColorLayer(aManager, static_cast<HostLayer*>(this)),
       LayerMLGPU(aManager) {}
 
-ColorLayerMLGPU::~ColorLayerMLGPU() {}
+ColorLayerMLGPU::~ColorLayerMLGPU() = default;
 
 }  // namespace layers
 }  // namespace mozilla