Trace synchronized blocks in NotifSettingsController
Bug: 295015479
Test: make
Change-Id: I20362f96e51d89adce35f4f651e31d44d74a8e37
diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotificationSettingsController.java b/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotificationSettingsController.java
index 51e4537..4ace194 100644
--- a/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotificationSettingsController.java
+++ b/packages/SystemUI/src/com/android/systemui/statusbar/notification/row/NotificationSettingsController.java
@@ -21,6 +21,7 @@
import android.net.Uri;
import android.os.Handler;
import android.os.HandlerExecutor;
+import android.os.Trace;
import androidx.annotation.MainThread;
import androidx.annotation.NonNull;
@@ -69,6 +70,7 @@
mContentObserver = new ContentObserver(mBackgroundHandler) {
@Override
public void onChange(boolean selfChange, Uri uri) {
+ Trace.traceBegin(Trace.TRACE_TAG_APP, TAG + ".ContentObserver.onChange");
super.onChange(selfChange, uri);
synchronized (mListeners) {
if (mListeners.containsKey(uri)) {
@@ -79,12 +81,15 @@
}
}
}
+ Trace.traceEnd(Trace.TRACE_TAG_APP);
}
};
mCurrentUserTrackerCallback = new UserTracker.Callback() {
@Override
public void onUserChanged(int newUser, Context userContext) {
+ Trace.traceBegin(Trace.TRACE_TAG_APP, TAG + ".UserTracker.Callback.onUserChanged");
+
synchronized (mListeners) {
if (mListeners.size() > 0) {
mSecureSettings.unregisterContentObserver(mContentObserver);
@@ -94,6 +99,7 @@
}
}
}
+ Trace.traceEnd(Trace.TRACE_TAG_APP);
}
};
mUserTracker.addCallback(
@@ -113,6 +119,7 @@
if (uri == null || listener == null) {
return;
}
+ Trace.traceBegin(Trace.TRACE_TAG_APP, TAG + ".addCallback");
synchronized (mListeners) {
ArrayList<Listener> currentListeners = mListeners.get(uri);
if (currentListeners == null) {
@@ -132,10 +139,12 @@
String value = getCurrentSettingValue(uri, userId);
mMainHandler.post(() -> listener.onSettingChanged(uri, userId, value));
});
-
+ Trace.traceEnd(Trace.TRACE_TAG_APP);
}
public void removeCallback(Uri uri, Listener listener) {
+ Trace.traceBegin(Trace.TRACE_TAG_APP, TAG + ".removeCallback");
+
synchronized (mListeners) {
ArrayList<Listener> currentListeners = mListeners.get(uri);
@@ -150,10 +159,13 @@
mSecureSettings.unregisterContentObserver(mContentObserver);
}
}
+ Trace.traceEnd(Trace.TRACE_TAG_APP);
}
@Override
public void dump(@NonNull PrintWriter pw, @NonNull String[] args) {
+ Trace.traceBegin(Trace.TRACE_TAG_APP, TAG + ".dump");
+
synchronized (mListeners) {
pw.println("Settings Uri Listener List:");
for (Uri uri : mListeners.keySet()) {
@@ -163,6 +175,7 @@
}
}
}
+ Trace.traceEnd(Trace.TRACE_TAG_APP);
}
private String getCurrentSettingValue(Uri uri, int userId) {