Commit 3e90a3f5 authored by Gregory Pappas's avatar Gregory Pappas
Browse files

Bug 1754851 - Remove Compositor::RecordFrame r=mstange 

Differential Revision: https://phabricator.services.mozilla.com/D179304
parent 17acee2c

gfx/layers/Compositor.cpp

+0 −17
Original line number Diff line number Diff line
@@ -233,23 +233,6 @@ size_t DecomposeIntoNoRepeatRects(const gfx::Rect& aRect, 
  return 4;

}



already_AddRefed<RecordedFrame> Compositor::RecordFrame(

    const TimeStamp& aTimeStamp) {

  RefPtr<CompositingRenderTarget> renderTarget = GetWindowRenderTarget();

  if (!renderTarget) {

    return nullptr;

  }



  RefPtr<AsyncReadbackBuffer> buffer =

      CreateAsyncReadbackBuffer(renderTarget->GetSize());



  if (!ReadbackRenderTarget(renderTarget, buffer)) {

    return nullptr;

  }



  return MakeAndAddRef<CompositorRecordedFrame>(aTimeStamp, std::move(buffer));

}



bool Compositor::ShouldRecordFrames() const {

  return profiler_feature_active(ProfilerFeature::Screenshots) || mRecordFrames;

}

gfx/layers/Compositor.h

+0 −8
Original line number Diff line number Diff line
@@ -327,14 +327,6 @@ class Compositor : public TextureSourceProvider { 
    mRecordFrames = aWillRecord;

  }



  /**

   * Record the current frame for readback by the |CompositionRecorder|.

   *

   * If this compositor does not support this feature, a null pointer is

   * returned instead.

   */

  already_AddRefed<RecordedFrame> RecordFrame(const TimeStamp& aTimeStamp);



 protected:

  /**

   * Whether or not the compositor should be prepared to record frames. While