Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit fb06bfd7 authored by Adithya Srinivasan's avatar Adithya Srinivasan Committed by Automerger Merge Worker
Browse files

Merge "Send isBuffer to perfetto trace" into sc-dev am: 5c6d16d6 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/native/+/14937651

Change-Id: I7c6fc4a57a63ffe3ad4ced006a5808060b029cf1
parents 2a8a57f7 5c6d16d6

services/surfaceflinger/FrameTimeline/FrameTimeline.cpp

+1 −0
Original line number Diff line number Diff line
@@ -693,6 +693,7 @@ void SurfaceFrame::traceActuals(int64_t displayFrameToken) const { 
        actualSurfaceFrameStartEvent->set_gpu_composition(mGpuComposition);

        actualSurfaceFrameStartEvent->set_jank_type(jankTypeBitmaskToProto(mJankType));

        actualSurfaceFrameStartEvent->set_prediction_type(toProto(mPredictionState));

        actualSurfaceFrameStartEvent->set_is_buffer(mIsBuffer);

    });



    // Actual timeline end

services/surfaceflinger/tests/unittests/FrameTimelineTest.cpp

+8 −5
Original line number Diff line number Diff line
@@ -877,7 +877,7 @@ ProtoExpectedSurfaceFrameStart createProtoExpectedSurfaceFrameStart(int64_t cook 
ProtoActualSurfaceFrameStart createProtoActualSurfaceFrameStart(

        int64_t cookie, int64_t token, int64_t displayFrameToken, pid_t pid, std::string layerName,

        ProtoPresentType presentType, bool onTimeFinish, bool gpuComposition,

        ProtoJankType jankType, ProtoPredictionType predictionType) {

        ProtoJankType jankType, ProtoPredictionType predictionType, bool isBuffer) {

    ProtoActualSurfaceFrameStart proto;

    proto.set_cookie(cookie);

    proto.set_token(token);
@@ -889,6 +889,7 @@ ProtoActualSurfaceFrameStart createProtoActualSurfaceFrameStart( 
    proto.set_gpu_composition(gpuComposition);

    proto.set_jank_type(jankType);

    proto.set_prediction_type(predictionType);

    proto.set_is_buffer(isBuffer);

    return proto;

}
@@ -978,6 +979,8 @@ void validateTraceEvent(const ProtoActualSurfaceFrameStart& received, 
    EXPECT_EQ(received.jank_type(), source.jank_type());

    ASSERT_TRUE(received.has_prediction_type());

    EXPECT_EQ(received.prediction_type(), source.prediction_type());

    ASSERT_TRUE(received.has_is_buffer());

    EXPECT_EQ(received.is_buffer(), source.is_buffer());

}



void validateTraceEvent(const ProtoFrameEnd& received, const ProtoFrameEnd& source) {
@@ -1154,7 +1157,7 @@ TEST_F(FrameTimelineTest, traceSurfaceFrame_emitsValidTracePacket) { 
                                               displayFrameToken1, sPidOne, sLayerNameOne,

                                               FrameTimelineEvent::PRESENT_DROPPED, false, false,

                                               FrameTimelineEvent::JANK_NONE,

                                               FrameTimelineEvent::PREDICTION_VALID);

                                               FrameTimelineEvent::PREDICTION_VALID, true);

    auto protoDroppedSurfaceFrameActualEnd = createProtoFrameEnd(traceCookie + 2);



    auto protoPresentedSurfaceFrameExpectedStart =
@@ -1166,7 +1169,7 @@ TEST_F(FrameTimelineTest, traceSurfaceFrame_emitsValidTracePacket) { 
                                               displayFrameToken1, sPidOne, sLayerNameOne,

                                               FrameTimelineEvent::PRESENT_ON_TIME, true, false,

                                               FrameTimelineEvent::JANK_NONE,

                                               FrameTimelineEvent::PREDICTION_VALID);

                                               FrameTimelineEvent::PREDICTION_VALID, true);

    auto protoPresentedSurfaceFrameActualEnd = createProtoFrameEnd(traceCookie + 4);



    // Set up the display frame
@@ -1309,7 +1312,7 @@ TEST_F(FrameTimelineTest, traceSurfaceFrame_predictionExpiredDoesNotTraceExpecte 
                                               displayFrameToken, sPidOne, sLayerNameOne,

                                               FrameTimelineEvent::PRESENT_UNSPECIFIED, false,

                                               false, FrameTimelineEvent::JANK_UNKNOWN,

                                               FrameTimelineEvent::PREDICTION_EXPIRED);

                                               FrameTimelineEvent::PREDICTION_EXPIRED, true);

    auto protoActualSurfaceFrameEnd = createProtoFrameEnd(traceCookie + 1);



    // Set up the display frame
@@ -1383,7 +1386,7 @@ TEST_F(FrameTimelineTest, traceSurfaceFrame_predictionExpiredDroppedFramesTraced 
                                               displayFrameToken, sPidOne, sLayerNameOne,

                                               FrameTimelineEvent::PRESENT_DROPPED, false, false,

                                               FrameTimelineEvent::JANK_NONE,

                                               FrameTimelineEvent::PREDICTION_EXPIRED);

                                               FrameTimelineEvent::PREDICTION_EXPIRED, true);

    auto protoActualSurfaceFrameEnd = createProtoFrameEnd(traceCookie + 1);



    // Set up the display frame