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

Commit edaa9132 authored by Josh Chien's avatar Josh Chien Committed by Lin Lili
Browse files

Fix dumpsys SurfaceFlinger --frame-events 

The function “dumpFrameEventsLocked” use incorrect method to traverse
layer hierarchy.

Bug: 176133025
Test: adb shell dumpsys SurfaceFlinger --frame-events
Change-Id: I27431cca559121ac0ed1ae1e1685872516ff2092
parent eed76146

services/surfaceflinger/SurfaceFlinger.cpp

+3 −6
Original line number Original line Diff line number Diff line
@@ -4502,12 +4502,9 @@ void SurfaceFlinger::recordBufferingStats(const std::string& layerName, 




void SurfaceFlinger::dumpFrameEventsLocked(std::string& result) {

void SurfaceFlinger::dumpFrameEventsLocked(std::string& result) {

    result.append("Layer frame timestamps:\n");

    result.append("Layer frame timestamps:\n");



    // Traverse all layers to dump frame-events for each layer

    const LayerVector& currentLayers = mCurrentState.layersSortedByZ;

    mCurrentState.traverseInZOrder(

    const size_t count = currentLayers.size();

        [&] (Layer* layer) { layer->dumpFrameEvents(result); });

    for (size_t i=0 ; i<count ; i++) {

        currentLayers[i]->dumpFrameEvents(result);

    }

}

}





void SurfaceFlinger::dumpBufferingStats(std::string& result) const {

void SurfaceFlinger::dumpBufferingStats(std::string& result) const {