diff options
| author | Florian Mayer <fmayer@google.com> | 2021-02-12 16:50:14 +0000 |
|---|---|---|
| committer | Florian Mayer <fmayer@google.com> | 2021-02-12 20:27:59 +0000 |
| commit | f287f6900c7212596f626a6fefbd2f16a7698882 (patch) | |
| tree | 14986e853e1346001b49f2cb3d5bc2598aa4f7ff | |
| parent | a8a2198ddc1a6bc4936a54a7df338fe374d943b2 (diff) | |
Do not check for debuggable / profilable for app tracing.
This is publicly undocumented and surprising behaviour.
Bug: 179883548
Change-Id: Iceee54c63dc9f5d61e9dcc76a47e11cc4088ed19
| -rw-r--r-- | core/java/android/os/Trace.java | 6 | ||||
| -rw-r--r-- | core/jni/android_os_Trace.cpp | 2 |
2 files changed, 5 insertions, 3 deletions
diff --git a/core/java/android/os/Trace.java b/core/java/android/os/Trace.java index 9c9e4995d673..c8cbc517b226 100644 --- a/core/java/android/os/Trace.java +++ b/core/java/android/os/Trace.java @@ -168,8 +168,10 @@ public final class Trace { } /** - * Set whether application tracing is allowed for this process. This is intended to be set - * once at application start-up time based on whether the application is debuggable. + * From Android S, this is no-op. + * + * Before, set whether application tracing is allowed for this process. This is intended to be + * set once at application start-up time based on whether the application is debuggable. * * @hide */ diff --git a/core/jni/android_os_Trace.cpp b/core/jni/android_os_Trace.cpp index 0f7611a8ead1..f67007cda209 100644 --- a/core/jni/android_os_Trace.cpp +++ b/core/jni/android_os_Trace.cpp @@ -83,7 +83,7 @@ static void android_os_Trace_nativeAsyncTraceEnd(JNIEnv* env, jclass, } static void android_os_Trace_nativeSetAppTracingAllowed(JNIEnv*, jclass, jboolean allowed) { - atrace_set_debuggable(allowed); + atrace_update_tags(); } static void android_os_Trace_nativeSetTracingEnabled(JNIEnv*, jclass, jboolean enabled) { |