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

Commit fd37d2a4 authored by Leon Scroggins III's avatar Leon Scroggins III
Browse files

sf-latency: Explicitly time SKIA_GL_THREADED 

Bug: 193240340
Test: This

My original intent for this benchmark was to time drawLayers, without
worrying about threading overhead. But using SKIA_GL means that we do
not execute mapExternalTextureBuffer ahead of time, so we have to do
this work each frame. mapExternalTextureBuffer should be tracked by a
separate benchmark.

Change-Id: I0d58fdf67a6322f0bbc04835ab1cdb08b73bcf7b
parent b76f17d5

libs/renderengine/benchmark/RenderEngineBench.cpp

+6 −6
Original line number Diff line number Diff line
@@ -46,8 +46,8 @@ std::string RenderEngineTypeName(RenderEngine::RenderEngineType type) { 
}



/**

 * Passed (indirectly - see RunSkiaGL) to Benchmark::Apply to create a Benchmark

 * which specifies which RenderEngineType it uses.

 * Passed (indirectly - see RunSkiaGLThreaded) to Benchmark::Apply to create a

 * Benchmark which specifies which RenderEngineType it uses.

 *

 * This simplifies calling ->Arg(type)->Arg(type) and provides strings to make

 * it obvious which version is being run.
@@ -62,10 +62,10 @@ static void AddRenderEngineType(benchmark::internal::Benchmark* b, 
}



/**

 * Run a benchmark once using SKIA_GL.

 * Run a benchmark once using SKIA_GL_THREADED.

 */

static void RunSkiaGL(benchmark::internal::Benchmark* b) {

    AddRenderEngineType(b, RenderEngine::RenderEngineType::SKIA_GL);

static void RunSkiaGLThreaded(benchmark::internal::Benchmark* b) {

    AddRenderEngineType(b, RenderEngine::RenderEngineType::SKIA_GL_THREADED);

}



///////////////////////////////////////////////////////////////////////////////
@@ -256,4 +256,4 @@ void BM_blur(benchmark::State& benchState) { 
    benchDrawLayers(*re, layers, benchState, "blurred");

}



BENCHMARK(BM_blur)->Apply(RunSkiaGL);
 
BENCHMARK(BM_blur)->Apply(RunSkiaGLThreaded);