From 2de9ed313032f725b59a6e07aa192e06b67af048 Mon Sep 17 00:00:00 2001 From: Edgar Arriaga Date: Wed, 3 Nov 2021 18:31:54 -0700 Subject: [PATCH] Remove old and unnecessary log Bug: 205056348 Test: Manual Change-Id: I74e470cd8dfd9cdeb2bbc9e56af2c7e48f631acd --- core/java/android/database/CursorWindow.java | 60 -------------------- 1 file changed, 60 deletions(-) diff --git a/core/java/android/database/CursorWindow.java b/core/java/android/database/CursorWindow.java index 1db948aba7c69..ccb7cf19d0b14 100644 --- a/core/java/android/database/CursorWindow.java +++ b/core/java/android/database/CursorWindow.java @@ -143,7 +143,6 @@ public class CursorWindow extends SQLiteClosable implements Parcelable { throw new AssertionError(); // Not possible, the native code won't return it. } mCloseGuard.open("close"); - recordNewWindow(Binder.getCallingPid(), mWindowPtr); } /** @@ -191,7 +190,6 @@ public class CursorWindow extends SQLiteClosable implements Parcelable { mCloseGuard.close(); } if (mWindowPtr != 0) { - recordClosingOfWindow(mWindowPtr); nativeDispose(mWindowPtr); mWindowPtr = 0; } @@ -746,64 +744,6 @@ public class CursorWindow extends SQLiteClosable implements Parcelable { dispose(); } - @UnsupportedAppUsage - private static final LongSparseArray sWindowToPidMap = new LongSparseArray(); - - private void recordNewWindow(int pid, long window) { - synchronized (sWindowToPidMap) { - sWindowToPidMap.put(window, pid); - if (Log.isLoggable(STATS_TAG, Log.VERBOSE)) { - Log.i(STATS_TAG, "Created a new Cursor. " + printStats()); - } - } - } - - private void recordClosingOfWindow(long window) { - synchronized (sWindowToPidMap) { - if (sWindowToPidMap.size() == 0) { - // this means we are not in the ContentProvider. - return; - } - sWindowToPidMap.delete(window); - } - } - - @UnsupportedAppUsage(maxTargetSdk = Build.VERSION_CODES.R, trackingBug = 170729553) - private String printStats() { - StringBuilder buff = new StringBuilder(); - int myPid = Process.myPid(); - int total = 0; - SparseIntArray pidCounts = new SparseIntArray(); - synchronized (sWindowToPidMap) { - int size = sWindowToPidMap.size(); - if (size == 0) { - // this means we are not in the ContentProvider. - return ""; - } - for (int indx = 0; indx < size; indx++) { - int pid = sWindowToPidMap.valueAt(indx); - int value = pidCounts.get(pid); - pidCounts.put(pid, ++value); - } - } - int numPids = pidCounts.size(); - for (int i = 0; i < numPids;i++) { - buff.append(" (# cursors opened by "); - int pid = pidCounts.keyAt(i); - if (pid == myPid) { - buff.append("this proc="); - } else { - buff.append("pid ").append(pid).append('='); - } - int num = pidCounts.get(pid); - buff.append(num).append(')'); - total += num; - } - // limit the returned string size to 1000 - String s = (buff.length() > 980) ? buff.substring(0, 980) : buff.toString(); - return "# Open Cursors=" + total + s; - } - private static int getCursorWindowSize() { if (sCursorWindowSize < 0) { // The cursor window size. resource xml file specifies the value in kB.