Merge "Fix a memory leak with pending jank classifications" into sc-dev

void BufferStateLayer::onSurfaceFrameCreated(

        const std::shared_ptr<frametimeline::SurfaceFrame>& surfaceFrame) {

    while (mPendingJankClassifications.size() >= kPendingClassificationMaxSurfaceFrames) {

        // Too many SurfaceFrames pending classification. The front of the deque is probably not

        // tracked by FrameTimeline and will never be presented. This will only result in a memory

        // leak.

        ALOGW("Removing the front of pending jank deque from layer - %s to prevent memory leak",

              mName.c_str());

        mPendingJankClassifications.pop_front();

    }

    mPendingJankClassifications.emplace_back(surfaceFrame);

}

    nsecs_t mCallbackHandleAcquireTime = -1;

    std::deque<std::shared_ptr<android::frametimeline::SurfaceFrame>> mPendingJankClassifications;

    // An upper bound on the number of SurfaceFrames in the pending classifications deque.

    static constexpr int kPendingClassificationMaxSurfaceFrames = 25;

    const std::string mBlastTransactionName{"BufferTX - " + mName};

    // This integer is incremented everytime a buffer arrives at the server for this layer,