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

Commit aa74a663 authored by Robert Delgado's avatar Robert Delgado
Browse files

Changed target path for transaction writing. 

Previously, transactions were written into a directory (data/SurfaceTrace.dat)
that required SELinux write permissions. Changing it to the new path now allows for it to
be accessed with no permission changes needed.

Test: adb shell su root service call SurfaceFlinger 1020 i32 1 then
verify that the transaction file is in
/data/misc/wmtrace/transaction_trace.pb

Change-Id: I1fc30bca93e7f2a064db856be223ddd64b53fbda
parent d82627cd

services/surfaceflinger/SurfaceInterceptor.h

+1 −1
Original line number Diff line number Diff line
@@ -46,7 +46,7 @@ using SurfaceChange = surfaceflinger::SurfaceChange; 
using Increment = surfaceflinger::Increment;

using DisplayChange = surfaceflinger::DisplayChange;



constexpr auto DEFAULT_FILENAME = "/data/SurfaceTrace.dat";

constexpr auto DEFAULT_FILENAME = "/data/misc/wmtrace/transaction_trace.pb";



class SurfaceInterceptor {

public:

services/surfaceflinger/tests/SurfaceInterceptor_test.cpp

+1 −1
Original line number Diff line number Diff line
@@ -60,7 +60,7 @@ constexpr auto UNIQUE_TEST_FG_SURFACE_NAME = "FG Interceptor Test Surface#0"; 
constexpr auto LAYER_NAME = "Layer Create and Delete Test";

constexpr auto UNIQUE_LAYER_NAME = "Layer Create and Delete Test#0";



constexpr auto DEFAULT_FILENAME = "/data/SurfaceTrace.dat";

constexpr auto DEFAULT_FILENAME = "/data/misc/wmtrace/transaction_trace.pb";



// Fill an RGBA_8888 formatted surface with a single color.

static void fillSurfaceRGBA8(const sp<SurfaceControl>& sc, uint8_t r, uint8_t g, uint8_t b) {