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

Commit 1995d22c authored by Rachel Lee's avatar Rachel Lee Committed by Automerger Merge Worker
Browse files

Merge "Add detailed Skia traces sysprop to RenderEngine." into sc-dev am: 2c34aa61 am: a396259d 

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

Change-Id: I879749f313bba89eda94d8795eca549571ab4034
parents 3bb77fc6 a396259d

libs/renderengine/include/renderengine/RenderEngine.h

+9 −4
Original line number Diff line number Diff line
@@ -49,6 +49,11 @@ 
 */

#define PROPERTY_DEBUG_RENDERENGINE_CAPTURE_FILENAME "debug.renderengine.capture_filename"



/**

 * Allows recording of Skia drawing commands with systrace.

 */

#define PROPERTY_SKIA_ATRACE_ENABLED "debug.renderengine.skia_atrace_enabled"



struct ANativeWindowBuffer;



namespace android {
@@ -95,10 +100,6 @@ public: 


    static std::unique_ptr<RenderEngine> create(const RenderEngineCreationArgs& args);



    RenderEngine() : RenderEngine(RenderEngineType::GLES) {}



    RenderEngine(RenderEngineType type) : mRenderEngineType(type) {}



    virtual ~RenderEngine() = 0;



    // ----- BEGIN DEPRECATED INTERFACE -----
@@ -205,6 +206,10 @@ public: 
    static void validateOutputBufferUsage(const sp<GraphicBuffer>&);



protected:

    RenderEngine() : RenderEngine(RenderEngineType::GLES) {}



    RenderEngine(RenderEngineType type) : mRenderEngineType(type) {}



    // Maps GPU resources for this buffer.

    // Note that work may be deferred to an additional thread, i.e. this call

    // is made asynchronously, but the caller can expect that map/unmap calls

libs/renderengine/skia/SkiaRenderEngine.cpp

+12 −3
Original line number Diff line number Diff line
@@ -14,13 +14,22 @@ 
 * limitations under the License.

 */



//#define LOG_NDEBUG 0

#undef LOG_TAG

#define LOG_TAG "RenderEngine"

#define ATRACE_TAG ATRACE_TAG_GRAPHICS



#include "SkiaRenderEngine.h"



#include <android-base/properties.h>

#include <src/core/SkTraceEventCommon.h>



namespace android {

namespace renderengine {

namespace skia {} // namespace skia

namespace skia {

SkiaRenderEngine::SkiaRenderEngine(RenderEngineType type) : RenderEngine(type) {

    SkAndroidFrameworkTraceUtil::setEnableTracing(

            base::GetBoolProperty(PROPERTY_SKIA_ATRACE_ENABLED, false));

}

} // namespace skia

} // namespace renderengine

} // namespace android

libs/renderengine/skia/SkiaRenderEngine.h

+3 −3
Original line number Diff line number Diff line
@@ -36,7 +36,7 @@ class BlurFilter; 
class SkiaRenderEngine : public RenderEngine {

public:

    static std::unique_ptr<SkiaRenderEngine> create(const RenderEngineCreationArgs& args);

    SkiaRenderEngine(RenderEngineType type) : RenderEngine(type) {}

    SkiaRenderEngine(RenderEngineType type);

    ~SkiaRenderEngine() override {}



    virtual std::future<void> primeCache() override { return {}; };
@@ -60,8 +60,8 @@ public: 


protected:

    virtual void mapExternalTextureBuffer(const sp<GraphicBuffer>& /*buffer*/,

                                          bool /*isRenderable*/) override;

    virtual void unmapExternalTextureBuffer(const sp<GraphicBuffer>& /*buffer*/) override;

                                          bool /*isRenderable*/) override = 0;

    virtual void unmapExternalTextureBuffer(const sp<GraphicBuffer>& /*buffer*/) override = 0;

};



} // namespace skia