Commit f9510e15 authored by sotaro's avatar sotaro
Browse files

Bug 1834948 - Remove PendingFrameInfo::mFrameNeedsRender r=gfx-reviewers,lsalzman 

PendingFrameInfo::mFrameNeedsRender is no longer needed. It was originally added for document splitting by Bug 1441308 and modified by Bug 1570869.

Differential Revision: https://phabricator.services.mozilla.com/D179032
parent 791826bf

gfx/webrender_bindings/RenderThread.cpp

+2 −5
Original line number Diff line number Diff line
@@ -604,13 +604,11 @@ void RenderThread::HandleFrameOneDocInner(wr::WindowId aWindowId, bool aRender, 


    WindowInfo* info = it->second.get();

    PendingFrameInfo& frameInfo = info->mPendingFrames.front();

    frameInfo.mFrameNeedsRender |= aRender;

    render = frameInfo.mFrameNeedsRender;



    frame = frameInfo;

  } else {

    // Just give the frame info default values.

    frame = {TimeStamp::Now(), VsyncId(), aRender};

    frame = {TimeStamp::Now(), VsyncId()};

  }



  // Sadly this doesn't include the lock, since we don't have the frame there
@@ -925,8 +923,7 @@ void RenderThread::IncPendingFrameCount(wr::WindowId aWindowId, 
    return;

  }

  it->second->mPendingFrameBuild++;

  it->second->mPendingFrames.push(

      PendingFrameInfo{aStartTime, aStartId, false});

  it->second->mPendingFrames.push(PendingFrameInfo{aStartTime, aStartId});

}



void RenderThread::DecPendingFrameBuildCount(wr::WindowId aWindowId) {

gfx/webrender_bindings/RenderThread.h

+0 −1
Original line number Diff line number Diff line
@@ -437,7 +437,6 @@ class RenderThread final { 
  struct PendingFrameInfo {

    TimeStamp mStartTime;

    VsyncId mStartId;

    bool mFrameNeedsRender = false;

  };



  struct WindowInfo {