Skip to content

Commit 5944038

Browse files
Ying WeiAndroid (Google) Code Review
authored andcommitted
Merge "Revert "FrameTimeline: Optimize tracing information about frame start time"" into main
2 parents f08666b + 8d2aede commit 5944038

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

services/surfaceflinger/FrameTimeline/FrameTimeline.cpp

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -801,9 +801,6 @@ void SurfaceFrame::traceActuals(int64_t displayFrameToken, nsecs_t monoBootOffse
801801
FrameTimelineDataSource::Trace([&](FrameTimelineDataSource::TraceContext ctx) {
802802
const auto timestamp = [&]() {
803803
std::scoped_lock lock(mMutex);
804-
if (mActuals.startTime != 0) {
805-
return mActuals.startTime;
806-
}
807804
// Actual start time is not yet available, so use expected start instead
808805
if (mPredictionState == PredictionState::Expired) {
809806
// If prediction is expired, we can't use the predicted start time. Instead, just
@@ -814,7 +811,7 @@ void SurfaceFrame::traceActuals(int64_t displayFrameToken, nsecs_t monoBootOffse
814811
return endTime - kPredictionExpiredStartTimeDelta;
815812
}
816813

817-
return mPredictions.startTime;
814+
return mActuals.startTime == 0 ? mPredictions.startTime : mActuals.startTime;
818815
}();
819816

820817
if (filterFramesBeforeTraceStarts && !shouldTraceForDataSource(ctx, timestamp)) {

0 commit comments

Comments
 (0)